On Sun, Oct 06, 2019 at 11:22:57AM +0200, Yves-Alexis Perez wrote:
> Or a “real” build log from the buildd network:
> https://buildd.debian.org/status/fetch.php?pkg=strongswan&arch=amd64&ver=5.8.0-1&stamp=1566867301&raw=0

Thanks, that's the link I was looking for.

> Here's an example:
>
> libtool: warning: relinking 'libstrongswan-eap-radius.la'
> libtool: install: (cd /<<PKGBUILDDIR>>/src/libcharon/plugins/eap_radius;
> /bin/bash "/<<PKGBUILDDIR>>/libtool"  --tag CC --mode=relink gcc -rdynamic -g
> -O2 -fdebug-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -Wformat
> -Werror=format-security -include /<<PKGBUILDDIR>>/config.h -module -avoid-
> version -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -Wl,-O1 -o libstrongswan-eap-
> radius.la -rpath /usr/lib/ipsec/plugins eap_radius_plugin.lo eap_radius.lo
> eap_radius_xauth.lo eap_radius_accounting.lo eap_radius_provider.lo
> eap_radius_dae.lo eap_radius_forward.lo ../../../../src/libradius/libradius.la
> -inst-prefix-dir /<<PKGBUILDDIR>>/debian/tmp)
> libtool: relink: gcc -shared  -fPIC -DPIC  .libs/eap_radius_plugin.o
> .libs/eap_radius.o .libs/eap_radius_xauth.o .libs/eap_radius_accounting.o
> .libs/eap_radius_provider.o .libs/eap_radius_dae.o
> .libs/eap_radius_forward.o   -Wl,-rpath -Wl,/usr/lib/ipsec
> -L/<<PKGBUILDDIR>>/debian/tmp/usr/lib/ipsec -L/usr/lib/ipsec -lradius  -g -O2
> -fstack-protector-strong -Wl,-z -Wl,relro -Wl,-z -Wl,now -Wl,--as-needed -Wl,-
> O1   -Wl,-soname -Wl,libstrongswan-eap-radius.so -o .libs/libstrongswan-eap-
> radius.so
>
> Libtool steps are displayed on two lines (libtool --mode=relink followed by
> libtool: relink), but this is already handled fine by blhc.

Now I'm somewhat confused. I think the issue in this case is not
"libtool: relink:" because I get no errors for those lines when
running blhc on the buildd log.

Instead, I think the problem is the quoting of the libtool
command. Could you try the attached patch?

Regards
Simon
-- 
+ privacy is necessary
+ using gnupg http://gnupg.org
+ public key id: 0x92FEFDB7E44C32F9
diff --git i/bin/blhc w/bin/blhc
index fba278f..d6e2690 100755
--- i/bin/blhc
+++ w/bin/blhc
@@ -53,7 +53,7 @@ my $cc_regex_normal = qr/
 my $warning_regex = qr/^(.+?):(\d+):\d+: warning: (.+?) \[(.+?)\]$/;
 # Regex to catch libtool commands and not lines which show commands executed
 # by libtool (e.g. libtool: link: ...).
-my $libtool_regex = qr/\blibtool\s.*--mode=/;
+my $libtool_regex = qr/\blibtool["']?\s.*--mode=/;
 my $libtool_link_regex = qr/\blibtool: link: /;
 
 # List of source file extensions which require preprocessing.

Attachment: signature.asc
Description: PGP signature

Reply via email to