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.

Sign up for GitHub

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

Jump to bottom

llvm-windres does not accept language id in hex #50637

Closed
jeremyd2019 opened this issue Jul 31, 2021 · 3 comments
Closed

llvm-windres does not accept language id in hex #50637

jeremyd2019 opened this issue Jul 31, 2021 · 3 comments
Labels
bugzilla Issues migrated from bugzilla

Comments

@jeremyd2019
Copy link
Contributor

Bugzilla Link 51295
Resolution FIXED
Resolved on Oct 11, 2021 20:29
Version 12.0
OS Windows NT
Blocks #51489
CC @mstorsjo,@tstellar
Fixed by commit(s) 46020f6 37e964d

Extended Description

llvm-windres --language 0x1 -i /dev/null -o /dev/null fails

llvm-windres --language 1 -i /dev/null -o /dev/null works

This was encountered trying to build graphviz for mingw-w64 using clang/llvm.
@mstorsjo
Copy link
Member

mstorsjo commented Aug 5, 2021

The fix was pushed in 46020f6.

This is trivial enough to warrant backporting to 13.x at this time, IMO.

@tstellar
Copy link
Collaborator

tstellar commented Aug 6, 2021

Merged: 37e964d

@tstellar
Copy link
Collaborator

mentioned in issue #51489

@tstellar tstellar mentioned this issue Oct 12, 2021
Closed
@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 11, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mstorsjo @tstellar @jeremyd2019