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

don't install a libtool file with static library #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ffontaine
Copy link

@ffontaine ffontaine commented Aug 31, 2020

Static library is supported since version 1.3.11 and 2f143c0

However, graphite2 is still installing libgraphite2.la which contains incorrect information (i.e. dlname set to libgraphite2.so and
old_library set to ''):

dlname='libgraphite2.so'

library_names='libgraphite2.so.3.2.1 libgraphite2.so.3 libgraphite2.so'

old_library=''

dependency_libs=''

This will result in the following build failure with any applications using this file such as harfbuzz:

arm-linux-g++.br_real: error: /home/buildroot/autobuild/run/instance-3/output-1/host/arm-buildroot-linux-uclibcgnueabi/sysroot/usr/lib/libgraphite2.so: No such file or directory
make[5]: *** [main] Error 1

Instead of trying to fix this libtool file, just disable it when building a static library as it is not needed

Fixes:

Signed-off-by: Fabrice Fontaine [email protected]

Static library is supported since version 1.3.11 and
silnrsi@2f143c0

However, graphite2 is still installing libgraphite2.la which contains
incorrect information (i.e. dlname set to libgraphite2.so and
old_library set to ''):

dlname='libgraphite2.so'

library_names='libgraphite2.so.3.2.1 libgraphite2.so.3 libgraphite2.so'

old_library=''

dependency_libs=''

This will result in the following build failure with any applications
using this file such as harfbuzz:

arm-linux-g++.br_real: error: /home/buildroot/autobuild/run/instance-3/output-1/host/arm-buildroot-linux-uclibcgnueabi/sysroot/usr/lib/libgraphite2.so: No such file or directory
make[5]: *** [main] Error 1

Instead of trying to fix this libtool file, just disable it when
building a static library as it is not needed

Fixes:
 - http://autobuild.buildroot.org/results/9ebe1d11e80755d59190ef2aae82bbba5cc45e44

Signed-off-by: Fabrice Fontaine <[email protected]>
@mgood7123
Copy link

applied in #69

@a17r
Copy link

a17r commented Dec 28, 2021

It is even less justified to have with BUILD_SHARED_LIBS.

@mgood7123
Copy link

It is even less justified to have with BUILD_SHARED_LIBS.

im not sure what you mean

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants