Bug#912566: sparse FTBFS with llvm/clang 7

[Sylvestre Ledru]

Le 18/11/2018 à 22:08, Uwe Kleine-König a écrit :

Hello,

On Sat, Nov 17, 2018 at 09:40:39AM +0100, Sylvestre Ledru wrote:

reassign 912566 sparse
forwarded 912566 https://bugs.llvm.org/show_bug.cgi?id=8220
thanks

similar to Adrian I fail to follow why this is spare's problem. Unless
sparse used llvm-config wrongly I think the problem at hand is that the
output of llvm-config is bad and responsible for the build failure.

Agreed that this is llvm-config. I reassigned to sparse because this isn't
a recent issue that llvm-config is bad and sparse is already workarounding
its limitations.

I proposed this MR to address that:
https://salsa.debian.org/ukleinek/sparse/merge_requests/1

Right, sparse could work around the problem by filtering the output of
llvm-config, but this is not the right fix for this problem.

Please implement the filtering in llvm (and also drop -pedantic and
-DNDEBUG from it, too).


I will (you are welcome providing a patch if you are interested)

Cheers,

S

Bug#912566: sparse FTBFS with llvm/clang 7

[Uwe Kleine-König]

Hello,

On Sat, Nov 17, 2018 at 09:40:39AM +0100, Sylvestre Ledru wrote:
> reassign 912566 sparse
> forwarded 912566 https://bugs.llvm.org/show_bug.cgi?id=8220
> thanks

similar to Adrian I fail to follow why this is spare's problem. Unless
sparse used llvm-config wrongly I think the problem at hand is that the
output of llvm-config is bad and responsible for the build failure.

> I proposed this MR to address that:
> https://salsa.debian.org/ukleinek/sparse/merge_requests/1

Right, sparse could work around the problem by filtering the output of
llvm-config, but this is not the right fix for this problem.

Please implement the filtering in llvm (and also drop -pedantic and
-DNDEBUG from it, too).

Best regards
Uwe


signature.asc
Description: PGP signature

Bug#912566: sparse FTBFS with llvm/clang 7

[Adrian Bunk]

On Sat, Nov 17, 2018 at 10:20:36AM +0100, Sylvestre Ledru wrote:
> 
> Le 17/11/2018 à 10:16, Adrian Bunk a écrit :
> > On Sat, Nov 17, 2018 at 09:40:39AM +0100, Sylvestre Ledru wrote:
> > > reassign 912566 sparse
> > > forwarded 912566 https://bugs.llvm.org/show_bug.cgi?id=8220
> > > thanks
> > > 
> > > Hello
> > > This is because llvm-config isn't great (understatement).
> > > It inherits of the flags used to compile llvm.
> > > 
> > > See also:
> > > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697755
> > > 
> > > I proposed this MR to address that:
> > > https://salsa.debian.org/ukleinek/sparse/merge_requests/1
> > Why isn't this filtering done in the llvm package where the bug is,
> > so that llvm-config is actually usable?
> 
> Yeah, I am planning to do that at some point. Or implement
> #897275  - llvm-6.0 should provide a
> pkg-config file

The latter is orthogonal - while pkg-config would be nicer than 
llvm-config, changing this won't help with the problem at hand
that the contents returned is bogus.

> S

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

Bug#912566: sparse FTBFS with llvm/clang 7

[Sylvestre Ledru]

Le 17/11/2018 à 10:16, Adrian Bunk a écrit :

On Sat, Nov 17, 2018 at 09:40:39AM +0100, Sylvestre Ledru wrote:

reassign 912566 sparse
forwarded 912566 https://bugs.llvm.org/show_bug.cgi?id=8220
thanks

Hello
This is because llvm-config isn't great (understatement).
It inherits of the flags used to compile llvm.

See also:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697755

I proposed this MR to address that:
https://salsa.debian.org/ukleinek/sparse/merge_requests/1

Why isn't this filtering done in the llvm package where the bug is,
so that llvm-config is actually usable?


Yeah, I am planning to do that at some point. Or implement
#897275  - llvm-6.0 should provide a 
pkg-config file


S

Bug#912566: sparse FTBFS with llvm/clang 7

[Adrian Bunk]

On Sat, Nov 17, 2018 at 09:40:39AM +0100, Sylvestre Ledru wrote:
> reassign 912566 sparse
> forwarded 912566 https://bugs.llvm.org/show_bug.cgi?id=8220
> thanks
> 
> Hello
> This is because llvm-config isn't great (understatement).
> It inherits of the flags used to compile llvm.
> 
> See also:
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697755
> 
> I proposed this MR to address that:
> https://salsa.debian.org/ukleinek/sparse/merge_requests/1

Why isn't this filtering done in the llvm package where the bug is,
so that llvm-config is actually usable?

> Cheers,
> Sylvestre

cu
Adrian

-- 

   "Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
   "Only a promise," Lao Er said.
   Pearl S. Buck - Dragon Seed

Bug#912566: sparse FTBFS with llvm/clang 7

[Sylvestre Ledru]

reassign 912566 sparse
forwarded 912566 https://bugs.llvm.org/show_bug.cgi?id=8220
thanks

Hello
This is because llvm-config isn't great (understatement).
It inherits of the flags used to compile llvm.

See also:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697755

I proposed this MR to address that:
https://salsa.debian.org/ukleinek/sparse/merge_requests/1

Cheers,
Sylvestre

Processed: Re: Bug#912566: sparse FTBFS with llvm/clang 7

[Debian Bug Tracking System]

Processing control commands:

> tags -1 - moreinfo unreproducible
Bug #912566 [sparse] sparse FTBFS with llvm/clang 7
Removed tag(s) moreinfo and unreproducible.
> reassign -1 llvm-7 1:7-9~exp1
Bug #912566 [sparse] sparse FTBFS with llvm/clang 7
Bug reassigned from package 'sparse' to 'llvm-7'.
Ignoring request to alter found versions of bug #912566 to the same values 
previously set
Ignoring request to alter fixed versions of bug #912566 to the same values 
previously set
Bug #912566 [llvm-7] sparse FTBFS with llvm/clang 7
Marked as found in versions llvm-toolchain-7/1:7-9~exp1.
> severity -1 serious
Bug #912566 [llvm-7] sparse FTBFS with llvm/clang 7
Severity set to 'serious' from 'important'
> unblock 912164 by -1
Bug #912164 [release.debian.org] transition: llvm-defaults
912164 was blocked by: 912163 912563 912561 893189 912162 912566 912564
912164 was blocking: 912163
Removed blocking bug(s) of 912164: 912566
> affects -1 src:sparse
Bug #912566 [llvm-7] sparse FTBFS with llvm/clang 7
Added indication that 912566 affects src:sparse

-- 
912164: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=912164
912566: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=912566
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems