-
Notifications
You must be signed in to change notification settings - Fork 439
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
Add extra freetype lib only when required #3578
Conversation
The BOINC manager needs to deal with fonts, but nothing else, really. To always incude it may have unexpected side-effects, as reported for the CLI-only build for OpenWrt: openwrt/packages#11768 (comment)
For platforms that have a BOINC screensaver I believe that the default BOINC screensaver boincscr also uses freetype. |
In reaction to a comment by @hnyman in openwrt/packages#11768 (comment) on BOINC embracing too many libraries it finds on its build host, the inclusion of the freetype library is now excluded by an embracing "if" statement. The patch is presented to BOINC upstream at BOINC/boinc#3578 Signed-off-by: Steffen Möller <moeller@debian.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my comment above
If not building the manager, the compilation works. I cannot tell what configure options trigger the building of these "slideshow" or "graphics" apps. |
In reaction to a comment by @hnyman in openwrt#11768 (comment) on BOINC embracing too many libraries it finds on its build host, the inclusion of the freetype library is now excluded by an embracing "if" statement. The patch is presented to BOINC upstream at BOINC/boinc#3578 Signed-off-by: Steffen Möller <moeller@debian.org>
In reaction to a comment by @hnyman in openwrt#11768 (comment) on BOINC embracing too many libraries it finds on its build host, the inclusion of the freetype library is now excluded by an embracing "if" statement. The patch is presented to BOINC upstream at BOINC/boinc#3578 Signed-off-by: Steffen Möller <moeller@debian.org>
In reaction to a comment by @hnyman in openwrt#11768 (comment) on BOINC embracing too many libraries it finds on its build host, the inclusion of the freetype library is now excluded by an embracing "if" statement. The patch is presented to BOINC upstream at BOINC/boinc#3578 Signed-off-by: Steffen Möller <moeller@debian.org>
I just saw the "Waiting for Response"-flag. That is not a response from me, is it? |
It's exactly for you. You fix is not correct in this PR |
Shall we close this PR, then? |
The BOINC manager needs to deal with fonts, but nothing else, really. To always include it may have unexpected side-effects, as reported for the CLI-only build for OpenWrt: openwrt/packages#11768 (comment)
Description of the Change
BOINC_EXTRA_LIBS are grasping all libraries of a list manually aggregated in configure.ac. Since the freetype library is only dealing with fonts, and fonts are used only the GUI, these should not be linked against if only building the command-line interface.
Admittedly, I am not really liking this patch but to do more may require a bit of an extra discussion. The BOINC_EXTRA_LIBS shall possibly be divided up - one set of libs for the command line, one for the GUI, and one that the two share.
Release Notes
N/A