mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de, christian.melki@t2data.com
Subject: Re: [ptxdist] pkg-config-wrapper breakage.
Date: Fri, 23 Apr 2021 09:17:12 +0200	[thread overview]
Message-ID: <20210423071712.GF4162561@pengutronix.de> (raw)
In-Reply-To: <3aeb907e-2049-4b47-ded1-a4a262761146@t2data.com>

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


  reply	other threads:[~2021-04-23  7:17 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 [this message]
2021-04-23  7:58   ` Christian Melki

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=20210423071712.GF4162561@pengutronix.de \
    --to=m.olbrich@pengutronix.de \
    --cc=christian.melki@t2data.com \
    --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