Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dependency on newer TimeZoneConverter #43

Closed
jnyrup opened this issue Apr 28, 2018 · 5 comments
Closed

Dependency on newer TimeZoneConverter #43

jnyrup opened this issue Apr 28, 2018 · 5 comments

Comments

@jnyrup
Copy link
Contributor

jnyrup commented Apr 28, 2018

Hi Matt

I'm sorry to bother you again with this dependency hassle again.
Would you mind upgrading TimeZoneNames to depend on the new TimeZoneConverter?

@mattjohnsonpint
Copy link
Owner

Yeah, will do shortly.

Need to get to the bottom of the dependency problem here though also. They should be able to rev independently if semver compatible.

@mattjohnsonpint
Copy link
Owner

Also, you could add a binding redirect. See my notes in mattjohnsonpint/TimeZoneConverter#16.

The real fix would be to figure out how to force a binding redirect be added automatically when you pull in the package from Nuget. If anyone knows how to do that, please let me know. Thanks.

@mattjohnsonpint
Copy link
Owner

I'm going to leave it as-is for now to help troubleshoot the root cause. Please add the binding redirect to your application to work around for now. Thanks.

@mattjohnsonpint
Copy link
Owner

Still don't have a great answer for the long term, but updated to TimeZoneConverter 2.4.1 in TimeZoneNames 3.3.3.

@mattjohnsonpint
Copy link
Owner

Given we have strong name signing now, I believe this issue is resolved. If not, please let me know. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants