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

Font manager 0.8.8 fails to build with clang15 #310

Closed
ehaupt opened this issue Feb 16, 2023 · 5 comments
Closed

Font manager 0.8.8 fails to build with clang15 #310

ehaupt opened this issue Feb 16, 2023 · 5 comments

Comments

@ehaupt
Copy link

ehaupt commented Feb 16, 2023

Describe the bug
FreeBSD recently imported llvm15 to the main branch. Font manager now fails to compile.

To Reproduce
Compile font manager 0.8.8 with llvm15.

Expected behavior
Expect it to compile without an error.

System Information

  • OS: FreeBSD 14.0-CURRENT, amd64 (not arch specific)
  • Desktop Environment not relevant for this compiler error
  • Font Manager version 0.8.8

Additional context
Compiler error with clang15 font-manager-0.8.8_2.log
freebsd-git pushed a commit to freebsd/freebsd-ports that referenced this issue Feb 16, 2023
@ehaupt
x11-fonts/font-manager: Mark port broken on 14.x
528b576 
Port does not compile with llvm15.

Upstream issue:	FontManager/font-manager#310

Notified by:	pkg-fallout
@JerryCasiano
Copy link
Contributor

JerryCasiano commented Feb 17, 2023 •
edited

Huh. Looks like it doesn't like the code generated by Vala.

Thanks for reporting, I'll see if I can workaround it over the weekend.

JerryCasiano added a commit that referenced this issue Feb 17, 2023
@JerryCasiano
- Address build failure with Clang 15 - Issue #310
b65f10a
@JerryCasiano
Copy link
Contributor

Can you please verify that the latest commit allows the build to succeed on FreeBSD?

If you need a release tagged please let me know. Otherwise feel free to close if the latest commit is good to go.

Thanks.

@ehaupt
Copy link
Author

ehaupt commented Feb 18, 2023

Can you please verify that the latest commit allows the build to succeed on FreeBSD?

Thank you, will do, I'll get back to you ASAP.

@ehaupt
Copy link
Author

ehaupt commented Feb 18, 2023

Can you please verify that the latest commit allows the build to succeed on FreeBSD?

I confirm that the commit b65f10a solves the issue.

If you need a release tagged please let me know. Otherwise feel free to close if the latest commit is good to go.

If you could tag the commit with a bugfix or minor release that would definitely help with the packaging but if you'd like to wait I could work around it.

Thank you for that very fast response 💯

@JerryCasiano
Copy link
Contributor

If you can workaround it please do, I would rather not bump the version until absolutely necessary at this point.

Thanks again for reporting this and verifying the fix.

freebsd-git pushed a commit to freebsd/freebsd-ports that referenced this issue Feb 18, 2023
@ehaupt
x11-fonts/font-manager: Fix build with llvm15
3b32597 
Import upstream patch in order to fix build with llvm15.

Upstream issue:	FontManager/font-manager#310
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ehaupt @JerryCasiano