Skip to content

build: fixed pkg-config operations#10

Closed
ensc wants to merge 1 commit intoNLnetLabs:masterfrom
ensc:pkg-config
Closed

build: fixed pkg-config operations#10
ensc wants to merge 1 commit intoNLnetLabs:masterfrom
ensc:pkg-config

Conversation

@ensc
Copy link

@ensc ensc commented Nov 11, 2018

PKG_PROG_PKG_CONFIG must not be in a conditional section because
autoconf expands this macro only once. E.g. actually, builds without
python support will fail to detect systemd.

See in https://autotools.io/pkgconfig/pkg_check_modules.html the notes
at "3.4 Optional modules" for details.

Commit moves PKG_PROG_PKG_CONFIG out of the "if ..." condition.

Signed-off-by: Enrico Scholz enrico.scholz@ensc.de

@ensc ensc changed the title build: fiued pjg-config operations build: fixed pkg-config operations Nov 11, 2018
PKG_PROG_PKG_CONFIG must not be in a conditional section because
autoconf expands this macro only once.  E.g. actually, builds without
python support will fail to detect systemd.

See in https://autotools.io/pkgconfig/pkg_check_modules.html the notes
at "3.4 Optional modules" for details.

Commit moves PKG_PROG_PKG_CONFIG out of the "if ..." condition.

Signed-off-by: Enrico Scholz <enrico.scholz@ensc.de>
@wcawijngaards
Copy link
Member

Thanks, applied.

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.

2 participants