Skip to content

Conversation

@ffontaine
Copy link
Contributor

@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 fontaine.fabrice@gmail.com

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:
 - http://autobuild.buildroot.org/results/9ebe1d11e80755d59190ef2aae82bbba5cc45e44

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
@mgood7123
Copy link

applied in #69

@a17r
Copy link
Contributor

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

@tim-eves tim-eves merged commit 92731a2 into silnrsi:master Oct 20, 2025
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.

4 participants