Skip to content

qdae: fix build with glibc 2.35 and deprecate#111515

Closed
cho-m wants to merge 1 commit intoHomebrew:masterfrom
cho-m:qdae-deprecate-and-linux
Closed

qdae: fix build with glibc 2.35 and deprecate#111515
cho-m wants to merge 1 commit intoHomebrew:masterfrom
cho-m:qdae-deprecate-and-linux

Conversation

@cho-m
Copy link
Copy Markdown
Member

@cho-m cho-m commented Sep 23, 2022

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

@cho-m cho-m added the CI-no-bottles Merge without publishing bottles label Sep 23, 2022
@BrewTestBot BrewTestBot added deprecated license Formula uses a deprecated SPDX license which should be updated formula deprecated Formula deprecated no Linux bottle Formula has no Linux bottle labels Sep 23, 2022
@cho-m cho-m force-pushed the qdae-deprecate-and-linux branch from 7a4250e to 075d481 Compare September 23, 2022 07:45
@cho-m cho-m changed the title qdae: fix build with GCC 11 and deprecate qdae: fix build with glibc 2.35 and deprecate Sep 23, 2022
Comment thread Formula/qdae.rb
Comment on lines +29 to +34
# Fix build failure with newer glibc:
# /usr/bin/ld: ../lib/.libs/libdsk.a(drvlinux.o): in function `linux_open':
# drvlinux.c:(.text+0x168): undefined reference to `major'
# /usr/bin/ld: ../lib/.libs/libdsk.a(compress.o): in function `comp_open':
# compress.c:(.text+0x268): undefined reference to `major'
ENV.append_to_cflags "-include sys/sysmacros.h" if OS.linux?
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could alternatively use patch/inreplace to add #include <sys/sysmacros.h>. Just trying simplest option since doesn't seem to be maintained.

@cho-m cho-m mentioned this pull request Sep 23, 2022
6 tasks
@cho-m cho-m marked this pull request as ready for review September 23, 2022 17:24
@BrewTestBot
Copy link
Copy Markdown
Contributor

🤖 A scheduled task has triggered a merge.

@cho-m cho-m deleted the qdae-deprecate-and-linux branch September 23, 2022 19:22
@github-actions github-actions bot added the outdated PR was locked due to age label Oct 24, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CI-no-bottles Merge without publishing bottles deprecated license Formula uses a deprecated SPDX license which should be updated formula deprecated Formula deprecated no Linux bottle Formula has no Linux bottle outdated PR was locked due to age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants