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

[rfc][depends] bump giflib but mainly to include static libs for texturepacker #9077

Closed
wants to merge 1 commit into from

Conversation

Paxxi
Copy link
Member

@Paxxi Paxxi commented Feb 7, 2016

@fetzerch asked yesterday about having a static library version of giflib for texturepacker so I rebuilt the package including static versions as well as the dll versions.

I haven't uploaded the package yet as I'm thinking we might get rid of the dll versions. They're currently only used by texturepacker and if that is built statically we can get rid of the dll's and avoid having them be included in the distribution.

Thoughts? @ace20022 @Montellese as you're familiar with giflib and texturepacker.

@Paxxi Paxxi added RFC PR submitted for gathering feedback Platform: Windows Component: Depends labels Feb 7, 2016
@ace20022
Copy link
Member

ace20022 commented Feb 7, 2016

5.1.1 had both static and dynamic. You could drop the dll though

@Paxxi
Copy link
Member Author

Paxxi commented Feb 7, 2016

ahh hadn't noticed that, no debug version though but not sure if it's needed.

@wsnipex
Copy link
Member

wsnipex commented Feb 9, 2016

nitpick: win32 depends only

@Paxxi
Copy link
Member Author

Paxxi commented May 22, 2016

obsoleted by #9856

@Paxxi Paxxi closed this May 22, 2016
@Paxxi Paxxi deleted the giflib branch May 22, 2016 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Depends Platform: Windows RFC PR submitted for gathering feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants