This repository has been archived by the owner. It is now read-only.

--without-gettext doesn't prevent libintl.so from being linked with #1501

Closed
yurivict opened this Issue Nov 8, 2015 · 6 comments

Comments

@yurivict

yurivict commented Nov 8, 2015

Seeing this in FreeBSD port.

mkvtoolnix-8.5.2

@mbunkus

This comment has been minimized.

Show comment
Hide comment
@mbunkus

mbunkus Nov 8, 2015

Owner

If you configure --without-gettext then configure will not use pkg-config and query for libintl. However, other libraries used by MKVToolNix may very well use libintl, and that's nothing that I can prevent.

So please grep -- -lintl build-config after configuring in order to see where that dependencies creeps in.

Owner

mbunkus commented Nov 8, 2015

If you configure --without-gettext then configure will not use pkg-config and query for libintl. However, other libraries used by MKVToolNix may very well use libintl, and that's nothing that I can prevent.

So please grep -- -lintl build-config after configuring in order to see where that dependencies creeps in.

@yurivict

This comment has been minimized.

Show comment
Hide comment
@yurivict

yurivict Nov 8, 2015

The file build-config still has LIBINTL_LIBS = -lintl.
And config.log still has this:

configure:10863: checking for gettext in -lintl
configure:10888: clang -o conftest -O2 -pipe -fno-omit-frame-pointer  -DLIBICONV_PLUG -fstack-protector -fno-strict-aliasing -I/usr/local/include -fno-omit-frame-pointer -I/usr/local/include -DLIBICONV_PLUG -I/usr/local/include  -L/usr/local/lib -fstack-protector -L/usr/local/lib conftest.c -lintl  -liconv   >&5
configure:10888: $? = 0
configure:10897: result: yes
configure:10911: checking libintl.h usability
configure:10911: clang -c -O2 -pipe -fno-omit-frame-pointer  -DLIBICONV_PLUG -fstack-protector -fno-strict-aliasing -I/usr/local/include -fno-omit-frame-pointer -I/usr/local/include -DLIBICONV_PLUG -I/usr/local/include conftest.c >&5
configure:10911: $? = 0
configure:10911: result: yes
configure:10911: checking libintl.h presence
configure:10911: clang-cpp -fno-omit-frame-pointer -I/usr/local/include -DLIBICONV_PLUG -I/usr/local/include conftest.c
configure:10911: $? = 0
configure:10911: result: yes
configure:10911: checking for libintl.h
configure:10911: result: yes

yurivict commented Nov 8, 2015

The file build-config still has LIBINTL_LIBS = -lintl.
And config.log still has this:

configure:10863: checking for gettext in -lintl
configure:10888: clang -o conftest -O2 -pipe -fno-omit-frame-pointer  -DLIBICONV_PLUG -fstack-protector -fno-strict-aliasing -I/usr/local/include -fno-omit-frame-pointer -I/usr/local/include -DLIBICONV_PLUG -I/usr/local/include  -L/usr/local/lib -fstack-protector -L/usr/local/lib conftest.c -lintl  -liconv   >&5
configure:10888: $? = 0
configure:10897: result: yes
configure:10911: checking libintl.h usability
configure:10911: clang -c -O2 -pipe -fno-omit-frame-pointer  -DLIBICONV_PLUG -fstack-protector -fno-strict-aliasing -I/usr/local/include -fno-omit-frame-pointer -I/usr/local/include -DLIBICONV_PLUG -I/usr/local/include conftest.c >&5
configure:10911: $? = 0
configure:10911: result: yes
configure:10911: checking libintl.h presence
configure:10911: clang-cpp -fno-omit-frame-pointer -I/usr/local/include -DLIBICONV_PLUG -I/usr/local/include conftest.c
configure:10911: $? = 0
configure:10911: result: yes
configure:10911: checking for libintl.h
configure:10911: result: yes
@mbunkus

This comment has been minimized.

Show comment
Hide comment
@mbunkus

mbunkus Nov 8, 2015

Owner

Hmm yeah, you're right. Please give this patch a try. You'll need autoconf/automake and run autogen.sh after applying and before running configure.

Owner

mbunkus commented Nov 8, 2015

Hmm yeah, you're right. Please give this patch a try. You'll need autoconf/automake and run autogen.sh after applying and before running configure.

@mbunkus mbunkus added this to the 8.6.0 milestone Nov 8, 2015

@mbunkus mbunkus self-assigned this Nov 8, 2015

@yurivict

This comment has been minimized.

Show comment
Hide comment
@yurivict

yurivict Nov 8, 2015

Yes, this patch causes libintl to disappear in all places.

yurivict commented Nov 8, 2015

Yes, this patch causes libintl to disappear in all places.

@mbunkus mbunkus closed this in bc596fa Nov 8, 2015

@mbunkus

This comment has been minimized.

Show comment
Hide comment
@mbunkus

mbunkus Nov 8, 2015

Owner

Thanks for the confirmation.

Owner

mbunkus commented Nov 8, 2015

Thanks for the confirmation.

@remuxer32

This comment has been minimized.

Show comment
Hide comment
@remuxer32

remuxer32 Jun 29, 2016

Collaborator

@mosu

A little reminder: you forgot to add a label [res:fixed/implemented] for this issue which was successfully fixed.

Collaborator

remuxer32 commented Jun 29, 2016

@mosu

A little reminder: you forgot to add a label [res:fixed/implemented] for this issue which was successfully fixed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.