mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Christian Melki <christian.melki@t2data.com>
To: ptxdist@pengutronix.de, Michael Olbrich <m.olbrich@pengutronix.de>
Subject: Re: [ptxdist] pkg-config-wrapper breakage.
Date: Fri, 23 Apr 2021 09:58:01 +0200	[thread overview]
Message-ID: <47e2bcd8-df32-7ee7-a917-9c7fa0427f73@t2data.com> (raw)
In-Reply-To: <20210423071712.GF4162561@pengutronix.de>

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


      reply	other threads:[~2021-04-23  7:58 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-04-22 19:11 Christian Melki
2021-04-23  7:17 ` Michael Olbrich
2021-04-23  7:58   ` Christian Melki [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=47e2bcd8-df32-7ee7-a917-9c7fa0427f73@t2data.com \
    --to=christian.melki@t2data.com \
    --cc=m.olbrich@pengutronix.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox