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

Fixes #3178. v1 - Library has no strong-name. #3191

Merged
merged 3 commits into from
Jan 22, 2024
Merged

Conversation

ilCosmico
Copy link

@ilCosmico ilCosmico commented Jan 19, 2024

Add strong-name signing.

Fixes

Proposed Changes/Todos

  • Add snk file
  • Add strong naming sign

Pull Request checklist:

  • I've named my PR in the form of "Fixes #issue. Terse description."
  • My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
  • My code follows the Terminal.Gui library design guidelines
  • I ran dotnet test before commit
  • I have made corresponding changes to the API documentation (using /// style comments)
  • My changes generate no new warnings
  • I have checked my code and corrected any poor grammar or misspellings
  • I conducted basic QA to assure all features are working

Add strong-name signing as requested for issue gui-cs#3178
Added strong name to UnitTests project too.
@ilCosmico
Copy link
Author

I suppose that NStack library should be strong-named too.

image

@ilCosmico
Copy link
Author

ilCosmico commented Jan 19, 2024

All tests passed!

image

BTW I'm not so expert on this library, so I don't know how "to assure all features are working".

@ilCosmico
Copy link
Author

Created PR #91 for NStack.

I tried to build again this version of Terminal.Gui with it and strong-name warning has now disappeared.

image

@tig
Copy link
Collaborator

tig commented Jan 19, 2024

BTW I'm not so expert on this library, so I don't know how "to assure all features are working".

Yeah, that's worded a bit unfairly.

Please, at least, do this:

  • Run the All Views Tester Scenario and select each view. Ensure nothing really odd happens.
  • Try a bunch of Scenarios. Some are more complex and tend to push the envelope and are good at sniffing out issues:
    • CharMap
    • Scrolling
    • Text Formatting
    • Editor
    • Dialogs
    • Wizards
  • Do the above under WSL
  • Do the above with -usc on both Windows and WSL

@tig tig changed the title Fixes #3178. Library has no strong-name. Fixes #3178. v1 - Library has no strong-name. Jan 19, 2024
@ilCosmico
Copy link
Author

@tig, @BDisp, I just added a strong-name option without changing the code. I don't think it compromises the library's running, do you?

@BDisp
Copy link
Collaborator

BDisp commented Jan 22, 2024

I don't think so and the tests pass. But you have to wait for @tig's decision.

@tig
Copy link
Collaborator

tig commented Jan 22, 2024

Do I need to do anything other than merge this for it to work for publishing to Nuget?

I was expecting to have to generate a private key that we'd need to store in a github secret or something.

@ilCosmico
Copy link
Author

It is not necessary to generate a password for the .snk file for strong name purposes, so you just need to merge it. Thank you, @tig.

@tig tig merged commit 18d38d3 into gui-cs:develop Jan 22, 2024
4 checks passed
@tig
Copy link
Collaborator

tig commented Jan 22, 2024

Ok, NStack v1.1.1 is published with an SN.

Now for Terminal.Gui...

@tig
Copy link
Collaborator

tig commented Jan 22, 2024

v1.15.1 is published.

Please verify that I didn't screw something up ;-)

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.

Enable strong name for Terminal.Gui v1
3 participants