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

Sign NAudio assembly #677

Closed
wants to merge 1 commit into from
Closed

Conversation

ermshiperete
Copy link

This change creates a strong-named version of NAudio.dll. While this
doesn't have any benefits in regards to security it helps those projects
that (for whatever reason) are required to create strong-named assemblies
because all dependencies have to be strong-named as well.

This change creates a strong-named version of NAudio.dll. While this
doesn't have any benefits in regards to security it helps those projects
that (for whatever reason) are required to create strong-named assemblies
because all dependencies have to be strong-named as well.
@markheath
Copy link
Contributor

The new NAudio 2 (currently in beta) will used strong named assemblies. Please let me know if that meets your needs

@ermshiperete
Copy link
Author

Yes, that helps. Thanks!

But it still would be good to have it available in v1 as well - I haven't checked what changed between v1 and v2, but the major version number change suggests that there are breaking changes that not every project can make immediately. And v2 is still in beta, whereas it should be possible to release a new stable v1 package right away (with the only change being that it is now signed).

@markheath
Copy link
Contributor

NAudio v2 will be highly compatible. All the same classes, with same names and same namespaces, just organized into different assembly groupings (which are strongly typed).

@ermshiperete
Copy link
Author

OK, sounds good. I'll close this one then.

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

Successfully merging this pull request may close these issues.

None yet

2 participants