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

Please provide strongly name key #12

Open
docwattsman opened this issue Oct 21, 2018 · 3 comments
Open

Please provide strongly name key #12

docwattsman opened this issue Oct 21, 2018 · 3 comments

Comments

@docwattsman
Copy link

Your code is great, but we were forced to fork it because it was not strongly signed. Could have been leveraged as-is, if only it had SNK in the project. Thank you for your hard work!

@jennings
Copy link
Owner

I've never set up strong naming before, but if you're willing to create a pull request that adds it to the build script, I'm open to merging it in.

The standard advice is for open source projects to check in their strong-naming key, so do include the key in the PR.

@docwattsman
Copy link
Author

docwattsman commented Oct 23, 2018 via email

@jennings
Copy link
Owner

jennings commented Oct 25, 2018

Not at all! The process is:

  1. Fork this repository (there should be a "Fork" button in the upper-right). This puts a copy of the repo into your GitHub account.

  2. Clone that repository to your computer, write and commit the changes you want to make, then push it back to GitHub. (Most of the time you'd want to make a new branch for this work, but if you do it in the master branch I won't be picky.)

  3. Back on GitHub, open a pull request to merge your branch into my master branch.

It looks like there are many ways you can strong-name an assembly; my preference if possible would be for MSBuild to automatically sign the assembly when generating a Release build.

Thanks for looking into this!

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