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

Use NuGet to reference ResourceLib and use the latest version #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

samhocevar
Copy link

Hi! I have been successfully using InsertIcons for WinCompose for years but I have always had random crashes happening, and with recent changes in the Microsoft SDKs it just stopped working. While checking ResourceLib I noticed they had made a lot of bugfixes since 2012, including an off-by-one issue with multiple icon insertion, and one for compatibility with WPF applications.

This PR switches the ResourceLib dependency to NuGet so that it is trivial to upgrade to the latest version. Hope you find this useful.

@einaregilsson
Copy link
Collaborator

Thanks. I'll try to get this merged and do a release when I have access to a windows machine again, I don't have one right now :)

@robdplatt
Copy link

@einaregilsson I also find this library useful. If you're interested in having it maintained while you don't have a Windows machine, I'm happy to help.

@einaregilsson
Copy link
Collaborator

@robdplatt Thanks, but I'll get it sorted next week, and do a release then :) No problem with the new version, no crashes or anything like that?

@robdplatt
Copy link

@einaregilsson Works just the same with the updated ResourceLib. I did update a fork to .NET 4.7.2 for my projects.

@AraHaan
Copy link

AraHaan commented Oct 13, 2022

Will this work for programs built with .NET 6 without removing it's assembly metadata?

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.

4 participants