[ptxdist] pkg-config-wrapper breakage.

[ptxdist] pkg-config-wrapper breakage.

[Christian Melki]

I'm having trouble building.
Reverting this commit fixes my issue.
https://git.pengutronix.de/cgit/ptxdist/commit/scripts/pkg-config-wrapper?id=ee2867017402a411c279095a0bd7669693c98352

whitelist source is not empty, first option is not help or version.
Hence next call fails because it mixes --print-provides with --libs and
--cflags.

In this empty test build ethtool is the first package after libmnl for
target. PKG_CONFIG_DEBUG_SPEW enabled set in ethtool configure script.

-----------------------
target: ethtool.prepare
-----------------------

checking for a BSD-compatible install...
/home/xxx/work/tracker/lib/ptxdist-2021.04.0/bin/install -c
checking whether build environment is sane... yes
checking for x86_64-tracker-linux-gnu-strip...
x86_64-tracker-linux-gnu-strip
checking for a thread-safe mkdir -p...
/home/xxx/work/tracker/lib/ptxdist-2021.04.0/bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking whether make supports nested variables... yes
checking whether to enable maintainer-specific portions of Makefiles... no
checking for x86_64-tracker-linux-gnu-gcc... x86_64-tracker-linux-gnu-gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... yes
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether x86_64-tracker-linux-gnu-gcc accepts -g... yes
checking for x86_64-tracker-linux-gnu-gcc option to accept ISO C89...
none needed
checking whether x86_64-tracker-linux-gnu-gcc understands -c and -o
together... yes
checking for style of include used by make... GNU
checking dependency style of x86_64-tracker-linux-gnu-gcc... gcc3
checking how to run the C preprocessor... x86_64-tracker-linux-gnu-gcc -E
checking for grep that handles long lines and -e... /usr/bin/grep
checking for egrep... /usr/bin/grep -E
checking whether x86_64-tracker-linux-gnu-gcc needs -traditional... no
checking pkg-config is at least version 0.9.0... PKG_CONFIG_DEBUG_SPEW
variable enabling debug spew
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
from PKG_CONFIG_PATH
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
from PKG_CONFIG_PATH
Global variable definition 'pc_sysrootdir' = '/'
Global variable definition 'pc_top_builddir' = '$(top_builddir)'
no output option set, defaulting to --exists
Error printing disabled by default due to use of output options
--exists, --atleast/exact/max-version, --list-all or no output option at
all. Value of --print-errors: 1
Error printing enabled
yes
checking whether <linux/types.h> defines big-endian types... yes
checking for ANSI C header files... yes
checking for socket... yes
checking for strtol... yes
checking for bash-completion directory...
${datadir}/bash-completion/completions
checking pkg-config is at least version 0.9.0... PKG_CONFIG_DEBUG_SPEW
variable enabling debug spew
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
from PKG_CONFIG_PATH
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
from PKG_CONFIG_PATH
Global variable definition 'pc_sysrootdir' = '/'
Global variable definition 'pc_top_builddir' = '$(top_builddir)'
no output option set, defaulting to --exists
Error printing disabled by default due to use of output options
--exists, --atleast/exact/max-version, --list-all or no output option at
all. Value of --print-errors: 1
Error printing enabled
yes
checking for libmnl... no
PKG_CONFIG_DEBUG_SPEW variable enabling debug spew
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
from PKG_CONFIG_PATH
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
from PKG_CONFIG_PATH
Global variable definition 'pc_sysrootdir' = '/'
Global variable definition 'pc_top_builddir' = '$(top_builddir)'
no output option set, defaulting to --exists
Error printing disabled by default due to use of output options
--exists, --atleast/exact/max-version, --list-all or no output option at
all. Value of --print-errors: 1
Error printing enabled
configure: error: Package requirements (libmnl) were not met:

PKG_CONFIG_DEBUG_SPEW variable enabling debug spew
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
from PKG_CONFIG_PATH
Adding directory
'/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
from PKG_CONFIG_PATH
Global variable definition 'pc_sysrootdir' = '/'
Global variable definition 'pc_top_builddir' = '$(top_builddir)'
Ignoring incompatible output option "--cflags"
Ignoring incompatible output option "--libs"
Error printing enabled by default due to use of output options besides
--exists, --atleast/exact/max-version or --list-all. Value of
--silence-errors: 0
Error printing enabled
Adding virtual 'pkg-config' package to list of known packages
Looking for package 'libmnl'
Looking for package 'libmnl-uninstalled'
No package 'libmnl' found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables MNL_CFLAGS
and MNL_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.
make: ***
[/home/xxx/work/tracker/lib/ptxdist-2021.04.0/rules/post/ptxd_make_world_prepare.make:20:
/home/xxx/work/tracker/bin/platform-tracker/state/ethtool.prepare] Error 1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] pkg-config-wrapper breakage.

[Michael Olbrich]

On Thu, Apr 22, 2021 at 09:11:30PM +0200, Christian Melki wrote:
> I'm having trouble building.
> Reverting this commit fixes my issue.
> https://git.pengutronix.de/cgit/ptxdist/commit/scripts/pkg-config-wrapper?id=ee2867017402a411c279095a0bd7669693c98352
> 
> whitelist source is not empty, first option is not help or version.
> Hence next call fails because it mixes --print-provides with --libs and
> --cflags.

Did you change pkg-config in some way? The version in ptxdist does not fail
if --print-provides and --libs / --cflags are mixed. It just prints some
warnings.
And this commit does not change the way pkg-config is called. It just
aborts early on error, which means that the package is not found.

Michael

> In this empty test build ethtool is the first package after libmnl for
> target. PKG_CONFIG_DEBUG_SPEW enabled set in ethtool configure script.
> 
> -----------------------
> target: ethtool.prepare
> -----------------------
> 
> checking for a BSD-compatible install...
> /home/xxx/work/tracker/lib/ptxdist-2021.04.0/bin/install -c
> checking whether build environment is sane... yes
> checking for x86_64-tracker-linux-gnu-strip...
> x86_64-tracker-linux-gnu-strip
> checking for a thread-safe mkdir -p...
> /home/xxx/work/tracker/lib/ptxdist-2021.04.0/bin/mkdir -p
> checking for gawk... gawk
> checking whether make sets $(MAKE)... yes
> checking whether make supports nested variables... yes
> checking whether to enable maintainer-specific portions of Makefiles... no
> checking for x86_64-tracker-linux-gnu-gcc... x86_64-tracker-linux-gnu-gcc
> checking whether the C compiler works... yes
> checking for C compiler default output file name... a.out
> checking for suffix of executables...
> checking whether we are cross compiling... yes
> checking for suffix of object files... o
> checking whether we are using the GNU C compiler... yes
> checking whether x86_64-tracker-linux-gnu-gcc accepts -g... yes
> checking for x86_64-tracker-linux-gnu-gcc option to accept ISO C89...
> none needed
> checking whether x86_64-tracker-linux-gnu-gcc understands -c and -o
> together... yes
> checking for style of include used by make... GNU
> checking dependency style of x86_64-tracker-linux-gnu-gcc... gcc3
> checking how to run the C preprocessor... x86_64-tracker-linux-gnu-gcc -E
> checking for grep that handles long lines and -e... /usr/bin/grep
> checking for egrep... /usr/bin/grep -E
> checking whether x86_64-tracker-linux-gnu-gcc needs -traditional... no
> checking pkg-config is at least version 0.9.0... PKG_CONFIG_DEBUG_SPEW
> variable enabling debug spew
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
> from PKG_CONFIG_PATH
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
> from PKG_CONFIG_PATH
> Global variable definition 'pc_sysrootdir' = '/'
> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
> no output option set, defaulting to --exists
> Error printing disabled by default due to use of output options
> --exists, --atleast/exact/max-version, --list-all or no output option at
> all. Value of --print-errors: 1
> Error printing enabled
> yes
> checking whether <linux/types.h> defines big-endian types... yes
> checking for ANSI C header files... yes
> checking for socket... yes
> checking for strtol... yes
> checking for bash-completion directory...
> ${datadir}/bash-completion/completions
> checking pkg-config is at least version 0.9.0... PKG_CONFIG_DEBUG_SPEW
> variable enabling debug spew
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
> from PKG_CONFIG_PATH
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
> from PKG_CONFIG_PATH
> Global variable definition 'pc_sysrootdir' = '/'
> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
> no output option set, defaulting to --exists
> Error printing disabled by default due to use of output options
> --exists, --atleast/exact/max-version, --list-all or no output option at
> all. Value of --print-errors: 1
> Error printing enabled
> yes
> checking for libmnl... no
> PKG_CONFIG_DEBUG_SPEW variable enabling debug spew
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
> from PKG_CONFIG_PATH
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
> from PKG_CONFIG_PATH
> Global variable definition 'pc_sysrootdir' = '/'
> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
> no output option set, defaulting to --exists
> Error printing disabled by default due to use of output options
> --exists, --atleast/exact/max-version, --list-all or no output option at
> all. Value of --print-errors: 1
> Error printing enabled
> configure: error: Package requirements (libmnl) were not met:
> 
> PKG_CONFIG_DEBUG_SPEW variable enabling debug spew
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
> from PKG_CONFIG_PATH
> Adding directory
> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
> from PKG_CONFIG_PATH
> Global variable definition 'pc_sysrootdir' = '/'
> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
> Ignoring incompatible output option "--cflags"
> Ignoring incompatible output option "--libs"
> Error printing enabled by default due to use of output options besides
> --exists, --atleast/exact/max-version or --list-all. Value of
> --silence-errors: 0
> Error printing enabled
> Adding virtual 'pkg-config' package to list of known packages
> Looking for package 'libmnl'
> Looking for package 'libmnl-uninstalled'
> No package 'libmnl' found
> 
> Consider adjusting the PKG_CONFIG_PATH environment variable if you
> installed software in a non-standard prefix.
> 
> Alternatively, you may set the environment variables MNL_CFLAGS
> and MNL_LIBS to avoid the need to call pkg-config.
> See the pkg-config man page for more details.
> make: ***
> [/home/xxx/work/tracker/lib/ptxdist-2021.04.0/rules/post/ptxd_make_world_prepare.make:20:
> /home/xxx/work/tracker/bin/platform-tracker/state/ethtool.prepare] Error 1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

Re: [ptxdist] pkg-config-wrapper breakage.

[Christian Melki]

On 4/23/21 9:17 AM, Michael Olbrich wrote:
> On Thu, Apr 22, 2021 at 09:11:30PM +0200, Christian Melki wrote:
>> I'm having trouble building.
>> Reverting this commit fixes my issue.
>> https://git.pengutronix.de/cgit/ptxdist/commit/scripts/pkg-config-wrapper?id=ee2867017402a411c279095a0bd7669693c98352
>>
>> whitelist source is not empty, first option is not help or version.
>> Hence next call fails because it mixes --print-provides with --libs and
>> --cflags.
> 
> Did you change pkg-config in some way? The version in ptxdist does not fail
> if --print-provides and --libs / --cflags are mixed. It just prints some
> warnings.
> And this commit does not change the way pkg-config is called. It just
> aborts early on error, which means that the package is not found.
> 
> Michael
> 

Nope. But maybe I misunderstood the error...
I came to the conclusion that it couldn't find libmnl because of the
mixed parameters. I was probably mistaking the warning whine for actual
libmnl .pc lookup failure (which was due to placement in lib64).

Thanks for the clarification, made me look more closely.
So, probably not an issue. :)

>> In this empty test build ethtool is the first package after libmnl for
>> target. PKG_CONFIG_DEBUG_SPEW enabled set in ethtool configure script.
>>
>> -----------------------
>> target: ethtool.prepare
>> -----------------------
>>
>> checking for a BSD-compatible install...
>> /home/xxx/work/tracker/lib/ptxdist-2021.04.0/bin/install -c
>> checking whether build environment is sane... yes
>> checking for x86_64-tracker-linux-gnu-strip...
>> x86_64-tracker-linux-gnu-strip
>> checking for a thread-safe mkdir -p...
>> /home/xxx/work/tracker/lib/ptxdist-2021.04.0/bin/mkdir -p
>> checking for gawk... gawk
>> checking whether make sets $(MAKE)... yes
>> checking whether make supports nested variables... yes
>> checking whether to enable maintainer-specific portions of Makefiles... no
>> checking for x86_64-tracker-linux-gnu-gcc... x86_64-tracker-linux-gnu-gcc
>> checking whether the C compiler works... yes
>> checking for C compiler default output file name... a.out
>> checking for suffix of executables...
>> checking whether we are cross compiling... yes
>> checking for suffix of object files... o
>> checking whether we are using the GNU C compiler... yes
>> checking whether x86_64-tracker-linux-gnu-gcc accepts -g... yes
>> checking for x86_64-tracker-linux-gnu-gcc option to accept ISO C89...
>> none needed
>> checking whether x86_64-tracker-linux-gnu-gcc understands -c and -o
>> together... yes
>> checking for style of include used by make... GNU
>> checking dependency style of x86_64-tracker-linux-gnu-gcc... gcc3
>> checking how to run the C preprocessor... x86_64-tracker-linux-gnu-gcc -E
>> checking for grep that handles long lines and -e... /usr/bin/grep
>> checking for egrep... /usr/bin/grep -E
>> checking whether x86_64-tracker-linux-gnu-gcc needs -traditional... no
>> checking pkg-config is at least version 0.9.0... PKG_CONFIG_DEBUG_SPEW
>> variable enabling debug spew
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
>> from PKG_CONFIG_PATH
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
>> from PKG_CONFIG_PATH
>> Global variable definition 'pc_sysrootdir' = '/'
>> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
>> no output option set, defaulting to --exists
>> Error printing disabled by default due to use of output options
>> --exists, --atleast/exact/max-version, --list-all or no output option at
>> all. Value of --print-errors: 1
>> Error printing enabled
>> yes
>> checking whether <linux/types.h> defines big-endian types... yes
>> checking for ANSI C header files... yes
>> checking for socket... yes
>> checking for strtol... yes
>> checking for bash-completion directory...
>> ${datadir}/bash-completion/completions
>> checking pkg-config is at least version 0.9.0... PKG_CONFIG_DEBUG_SPEW
>> variable enabling debug spew
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
>> from PKG_CONFIG_PATH
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
>> from PKG_CONFIG_PATH
>> Global variable definition 'pc_sysrootdir' = '/'
>> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
>> no output option set, defaulting to --exists
>> Error printing disabled by default due to use of output options
>> --exists, --atleast/exact/max-version, --list-all or no output option at
>> all. Value of --print-errors: 1
>> Error printing enabled
>> yes
>> checking for libmnl... no
>> PKG_CONFIG_DEBUG_SPEW variable enabling debug spew
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
>> from PKG_CONFIG_PATH
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
>> from PKG_CONFIG_PATH
>> Global variable definition 'pc_sysrootdir' = '/'
>> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
>> no output option set, defaulting to --exists
>> Error printing disabled by default due to use of output options
>> --exists, --atleast/exact/max-version, --list-all or no output option at
>> all. Value of --print-errors: 1
>> Error printing enabled
>> configure: error: Package requirements (libmnl) were not met:
>>
>> PKG_CONFIG_DEBUG_SPEW variable enabling debug spew
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/lib/pkgconfig'
>> from PKG_CONFIG_PATH
>> Adding directory
>> '/home/xxx/work/tracker/bin/platform-tracker/sysroot-target/usr/share/pkgconfig'
>> from PKG_CONFIG_PATH
>> Global variable definition 'pc_sysrootdir' = '/'
>> Global variable definition 'pc_top_builddir' = '$(top_builddir)'
>> Ignoring incompatible output option "--cflags"
>> Ignoring incompatible output option "--libs"
>> Error printing enabled by default due to use of output options besides
>> --exists, --atleast/exact/max-version or --list-all. Value of
>> --silence-errors: 0
>> Error printing enabled
>> Adding virtual 'pkg-config' package to list of known packages
>> Looking for package 'libmnl'
>> Looking for package 'libmnl-uninstalled'
>> No package 'libmnl' found
>>
>> Consider adjusting the PKG_CONFIG_PATH environment variable if you
>> installed software in a non-standard prefix.
>>
>> Alternatively, you may set the environment variables MNL_CFLAGS
>> and MNL_LIBS to avoid the need to call pkg-config.
>> See the pkg-config man page for more details.
>> make: ***
>> [/home/xxx/work/tracker/lib/ptxdist-2021.04.0/rules/post/ptxd_make_world_prepare.make:20:
>> /home/xxx/work/tracker/bin/platform-tracker/state/ethtool.prepare] Error 1
>>
>>
>> _______________________________________________
>> ptxdist mailing list
>> ptxdist@pengutronix.de
>> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
>>


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de