Summary: | libdir is not expanded and p11-kit tries to load modules from ${exec_prefix}/lib/pkcs11/ | ||
---|---|---|---|
Product: | p11-glue | Reporter: | Ionut Biru <biru.ionut> |
Component: | p11-kit | Assignee: | Stef Walter <stefw> |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | ||
Version: | unspecified | ||
Hardware: | Other | ||
OS: | All | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: |
patch that fixes this issue
quoting fix |
Description
Ionut Biru
2011-09-18 03:24:01 UTC
Created attachment 51308 [details] [review] patch that fixes this issue FWIW, there's a reason why you cannot expand variables depending on ${libdir} in the configure script. It's because autoconf leaves some variables unexpended to be able to override then when running make (eg. make exec_prefix=/blah and libdir is ${exec_prefix}/lib)and thus you can only reliably defines variables depending on ${libdir} in the Makefiles. AFAIK, the way to solve that is, instead of using config.h and expect to be able to expand variables in configure.ac, to declare #defines in the CPPFLAGS with a -D option (eg. -DP11_FOO=\""${libdir}/p11/foo"\") Expanding it in the flags sounds like a good plan. Pushed a commit which does this. Could you test it to make sure it fixes the problem? http://cgit.freedesktop.org/p11-glue/p11-kit/commit/?id=a081b6652acc9d9a9af22a266f9175f689b8c5d1 Created attachment 51366 [details] [review] quoting fix git head is slightly broken, the quoting is not yet correct: ------------------------------------ [...] + exec ./configure --build=i486-linux-gnu --prefix=/usr --includedir=${prefix}/include --mandir=${prefix}/share/man --infodir=${prefix}/share/info --sysconfdir=/etc --localstatedir=/var --libexecdir=${prefix}/lib/p11-kit --disable-maintainer-mode --disable-dependency-tracking --libdir=${prefix}/lib/i386-linux-gnu [...] configure: build options: System Global Config: /etc/pkcs11/pkcs11.conf System Module Config Directory: /etc/pkcs11/modules User Global Config: ~/.pkcs11/pkcs11.conf User Module Config Directory: ~/.pkcs11/modules Load relative module paths from: ${prefix}/lib/i386-linux-gnu/pkcs11 [...] make[2]: Entering directory `/tmp/P11-KIT/p11-kit.git/p11-kit' CC util.lo /bin/bash ../libtool --silent --tag=CC --mode=compile gcc -DHAVE_CONFIG_H -I. -I.. -I.. -DP11_KIT_FUTURE_UNSTABLE_API -DP11_SYSTEM_CONFIG_FILE=\""/etc/pkcs11/pkcs11.conf"\" -DP11_SYSTEM_CONFIG_MODULES=\""/etc/pkcs11/modules"\" -DP11_USER_CONFIG_FILE=\""~/.pkcs11/pkcs11.conf"\" -DP11_USER_CONFIG_MODULES=\""11_user_config_modules"\" -DP11_MODULE_PATH=\""11_module_path"\" -g -O2 -Wall -Wstrict-prototypes -Wmissing-declarations -Wmissing-prototypes -Wnested-externs -Wpointer-arith -Wdeclaration-after-statement -Wformat=2 -Winit-self -Waggregate-return -Wno-missing-format-attribute -Wmissing-include-dirs -Wundef -c -o util.lo util.c ------------------------------------ Simple two-line fix attached. reopen - see last comment Comment on attachment 51366 [details] [review] quoting fix Thanks, merged this patch. Will prep another release soon. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.