Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-29 Thread Rebecca N. Palmer
The patches in SVN have a bug: 4.0 is using LLVM_3.9 for libllvm (simple_version_script.map), though the correct LLVM_4.0 elsewhere (AddLLVM.cmake). The override_dh_makeshlibs/1:x symbols minimum version are needed in all versions, not just 3.8. Also, #857623 (the OpenCL bug) also affects

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-20 Thread Rebecca N. Palmer
Thanks, but I think we missed something: as packages rebuilt against new (versioned-symbols) LLVM don't work with old LLVM, we need a shlibs bump to make dependencies reflect that (i.e. require upgrading libllvm/libclang before their dependencies). I think that's --- /dev/null +++

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Rebecca N. Palmer
Rebuilding beignet and mesa against this llvm succeeds, and this beignet works. (beignet is statically linked to LLVM/Clang, so we can't use it as a test of whether non-rebuilt rdeps work.) blender+pocl-opencl-icd still crashes, with the bad jump being from 3.8 to a versioned 3.9 symbol, but

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Rebecca N. Palmer
On 19/03/17 16:09, Sylvestre Ledru wrote: Indeed. Are you going to update the symbol file? I just deleted the symbols file to get the build to finish (which it did, and objdump -T confirmed that libLLVM and libclang had versioned symbols), but if you want to keep it, here's the full patch

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Sylvestre Ledru
Le 19/03/2017 à 17:02, Rebecca N. Palmer a écrit : > 12 clang tests "unexpectedly" fail, but they're the same 12 that do so > in the existing package: > Failing Tests (12): > Clang :: CodeGen/linux-arm-atomic.c > Clang :: Driver/arm-cortex-cpus.c > Clang :: Driver/arm-features.c >

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Rebecca N. Palmer
12 clang tests "unexpectedly" fail, but they're the same 12 that do so in the existing package: Failing Tests (12): Clang :: CodeGen/linux-arm-atomic.c Clang :: Driver/arm-cortex-cpus.c Clang :: Driver/arm-features.c Clang :: Driver/arm-ias-Wa.s Clang :: Driver/arm-mfpu.c

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Rebecca N. Palmer
Looks like this is a known cowdancer issue (#640684 - fixed only for cowbuilder --build, not --login); continuing with LD_PRELOAD= debian/rules build, the LLVM tests pass: Expected Passes: 16742 Expected Failures : 124 Unsupported Tests : 339 Unexpected Passes : 17 Still

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Sylvestre Ledru
Le 19/03/2017 à 15:46, Rebecca N. Palmer a écrit : >> What do you mean by "not tested" ? You did built it right? > Not as of that message. > > I since tried to build the "version script" option: it failed most of > its tests with > -- > Exit Code: 126 > > Command Output (stderr): > -- > E: env

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Rebecca N. Palmer
What do you mean by "not tested" ? You did built it right? Not as of that message. I since tried to build the "version script" option: it failed most of its tests with -- Exit Code: 126 Command Output (stderr): -- E: env var COWDANCER_ILISTFILE not defined E: cowdancer: Fatal,

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Sylvestre Ledru
Le 19/03/2017 à 11:13, Rebecca N. Palmer a écrit : > Here's the 'version script' solution, now covering libLLVM, libclang, > liblldb, libLTO, BugpointPasses, LLVMHello and LLVMgold (I suspect > only the first 2-3 actually have use for it, but all except the first > are set from the same place). >

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Rebecca N. Palmer
Here's the 'version script' solution, now covering libLLVM, libclang, liblldb, libLTO, BugpointPasses, LLVMHello and LLVMgold (I suspect only the first 2-3 actually have use for it, but all except the first are set from the same place). Warning: this hasn't been tested either, and the

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-19 Thread Sylvestre Ledru
Le 18/03/2017 à 23:40, Rebecca N. Palmer a écrit : > Can this be fixed in time for Ubuntu 17.04? Their mesa is built against LLVM > 4.0, so if this bug isn't fixed, I (as beignet maintainer) have to choose > between "apply a very new 4.0-support patch that might or might not work" and > "leave

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-03-18 Thread Rebecca N. Palmer
Can this be fixed in time for Ubuntu 17.04? Their mesa is built against LLVM 4.0, so if this bug isn't fixed, I (as beignet maintainer) have to choose between "apply a very new 4.0-support patch that might or might not work" and "leave beignet on 3.9, making OpenCL+OpenGL applications (e.g.

Processed: Re: Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-23 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org: > tag 848368 - patch Bug #848368 [src:llvm-toolchain-3.9] llvm-toolchain-3.9: Please add ELF symbols versions to the libraries Removed tag(s) patch. > thanks Stopping processing here. Please contact me if you need assistance. -- 848368:

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-23 Thread Lisandro Damián Nicanor Pérez Meyer
tag 848368 - patch thanks I have tested the patch and indeed it only works on libllvm, so removing the patch tag for now. On domingo, 22 de enero de 2017 21:31:08 ART Rebecca N. Palmer wrote: > On 22/01/17 21:07, Sylvestre Ledru wrote: > > , you haven't pass the arg to LDFLAGS to make sure that

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-23 Thread Lisandro Damián Nicanor Pérez Meyer
On domingo, 22 de enero de 2017 21:31:08 ART Rebecca N. Palmer wrote: [snip] > >> - Install and test LLVM 3.9 with stuff built with previous versions, like > >> for example kdevelop ;-) Everything should just work. > > kdevelop (#846410) has already been fixed by moving it to all-3.9; if > you

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-22 Thread Lisandro Damián Nicanor Pérez Meyer
On domingo, 22 de enero de 2017 22:07:45 ART Sylvestre Ledru wrote: > Le 22/01/2017 à 21:02, Lisandro Damián Nicanor Pérez Meyer a écrit : > > Control: tag -1 patch > > > > I'm attaching a patch based on Rebecca's. I simply used a CMake variable > > to > > avoid harcoding a path. Note it's the

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-22 Thread Rebecca N. Palmer
On 22/01/17 21:07, Sylvestre Ledru wrote: > , you haven't pass the arg to LDFLAGS to make sure that libclang or > liblldb get it, > is that on purpose? My original intent was to avoid passing it to those parts of LLVM that already use a "version" (actually which-symbols-are-public) script, as I

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-22 Thread Sylvestre Ledru
Le 22/01/2017 à 21:02, Lisandro Damián Nicanor Pérez Meyer a écrit : > Control: tag -1 patch > > I'm attaching a patch based on Rebecca's. I simply used a CMake variable to > avoid harcoding a path. Note it's the patch itself and not a debdiff: you > need > to add it to quilt's series file. OK,

Processed: Re: Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-22 Thread Debian Bug Tracking System
Processing control commands: > tag -1 patch Bug #848368 [src:llvm-toolchain-3.9] llvm-toolchain-3.9: Please add ELF symbols versions to the libraries Ignoring request to alter tags of bug #848368 to the same tags previously set -- 848368: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=848368

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-22 Thread Lisandro Damián Nicanor Pérez Meyer
Control: tag -1 patch I'm attaching a patch based on Rebecca's. I simply used a CMake variable to avoid harcoding a path. Note it's the patch itself and not a debdiff: you need to add it to quilt's series file. I'm trying to get LLVM built into barriere.d.o. So far I've got to test failing

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-21 Thread Julien Cristau
On Sat, Jan 21, 2017 at 09:52:58 +0100, Sylvestre Ledru wrote: > I feel a bit uncomfortable to implement the ELF symbol versions that > late in the cycle. > Especially for two reasons: I don't know enough that stuff to evaluate > and fix side effects > it has always been like that without causing

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-21 Thread Sylvestre Ledru
Hello, First, Rebecca, many thanks for the patch :) Le 18/01/2017 à 19:54, Lisandro Damián Nicanor Pérez Meyer a écrit : > On martes, 17 de enero de 2017 22:21:51 ART Rebecca N. Palmer wrote: > [snip] >> This suggests the fix (warning: untested and not my area of expertise - >> and if it is

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-18 Thread Lisandro Damián Nicanor Pérez Meyer
On martes, 17 de enero de 2017 22:21:51 ART Rebecca N. Palmer wrote: [snip] > This suggests the fix (warning: untested and not my area of expertise - > and if it is using that line, why is there no -Wl,--no-whole-archive in > the build log?): This part I don't know. > ---

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-17 Thread Rebecca N. Palmer
Control: tags -1 patch One way to reproduce this bug: # apt-get install pocl-opencl-icd blender $ gdb blender open User Preferences -> System This promptly crashes; beignet-opencl-icd 1.2.1-1 (LLVM 3.8, like pocl) also triggers it, but beignet-opencl-icd 1.2.1-2 and mesa-opencl-icd (LLVM 3.9,

Processed: Re: Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2017-01-17 Thread Debian Bug Tracking System
Processing control commands: > tags -1 patch Bug #848368 [src:llvm-toolchain-3.9] llvm-toolchain-3.9: Please add ELF symbols versions to the libraries Added tag(s) patch. -- 848368: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=848368 Debian Bug Tracking System Contact ow...@bugs.debian.org

Processed: Re: Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2016-12-22 Thread Debian Bug Tracking System
Processing commands for cont...@bugs.debian.org: > clone 848368 -1 Bug #848368 [src:llvm-toolchain-3.9] llvm-toolchain-3.9: Please add ELF symbols versions to the libraries Bug 848368 cloned as bug 849098 > reassign -1 llvm-toolchain-3.8 Bug #849098 [src:llvm-toolchain-3.9] llvm-toolcha

Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2016-12-22 Thread Lisandro Damián Nicanor Pérez Meyer
clone 848368 -1 reassign -1 llvm-toolchain-3.8 thanks This should happen on all versions actually. -- Trabaja como si no necesitaras el dinero. Ama como si nunca hubieses sido herido. Baila como si nadie estuviera mirando. Canta como si nadie escuchara. Vive como si fuera el Cielo en la Tierra.

Processed: Re: Bug#848368: llvm-toolchain-3.9: Please add ELF symbols versions to the libraries

2016-12-22 Thread Debian Bug Tracking System
Processing control commands: > severity -1 serious Bug #848368 [src:llvm-toolchain-3.9] llvm-toolchain-3.9: Please add ELF symbols versions to the libraries Severity set to 'serious' from 'wishlist' -- 848368: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=848368 Debian Bug Tracking System