Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-22 Thread Andreas Metzler
Control: reassign -1 pkgconf
Control: found -1 1.8.0-6
Control: close -1 1.8.0-9

On 2022-10-22 Andreas Metzler  wrote:
> On 2022-10-21 Andreas Metzler  wrote:
> [...]
> > Indeed applying the upstream patch fixes the testcase Nilesh provided.

> > I have not tried rebuilding jellyfish with patched pkgconf.

> Strangely the fix does not work in the installed version:

Nevermind. User-error, still had old libpkgconf3.

jellyfish builds successfully with the new pkgconf upload in
experimental.

cu Andreas

-- 
`What a good friend you are to him, Dr. Maturin. His other friends are
so grateful to you.'
`I sew his ears on from time to time, sure'



Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-22 Thread Andreas Metzler
X-Debbugs-Cc: pkgc...@packages.debian.org

On 2022-10-21 Andreas Metzler  wrote:
[...]
> Indeed applying the upstream patch fixes the testcase Nilesh provided.

> I have not tried rebuilding jellyfish with patched pkgconf.

Strangely the fix does not work in the installed version:
(sid)ametzler@argenau:/dev/shm/JEL/pkgconf-1.8.0$ ln -s pkgconf pkg-config
(sid)ametzler@argenau:/dev/shm/JEL/pkgconf-1.8.0$ PKG_CONFIG_PATH=/tmp/ 
PKG_CONFIG_SYSROOT_DIR=debian/tmp/ ./pkg-config --cflags jellyfish-2.0
-Idebian/tmp//usr/include/jellyfish-2.3.0
(sid)ametzler@argenau:/dev/shm/JEL/pkgconf-1.8.0$ PKG_CONFIG_PATH=/tmp/ 
PKG_CONFIG_SYSROOT_DIR=debian/tmp/ pkg-config --cflags jellyfish-2.0
-I/usr/include/jellyfish-2.3.0
(sid)ametzler@argenau:/dev/shm/JEL/pkgconf-1.8.0$ pkg-config --version
1.8.0
(sid)ametzler@argenau:/dev/shm/JEL/pkgconf-1.8.0$ ./pkg-config --version
1.8.0
(sid)ametzler@argenau:/dev/shm/JEL/pkgconf-1.8.0$ dpkg -s pkgconf | grep ^Versio
n
Version: 1.8.0-9

cu Andreas
-- 
`What a good friend you are to him, Dr. Maturin. His other friends are
so grateful to you.'
`I sew his ears on from time to time, sure'



Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-21 Thread Andrej Shadura
Hi,

On Fri, 21 Oct 2022, at 19:34, Nilesh Patra wrote:
> D'you plan to rebuild this, and other few currently failing packages w/ 
> the new pkgconf release?

Yes, I also will rebuild all packages that failed because of missing 
dependencies (Perl transition) or aborted during the build (possibly OOM or 
disk space).

-- 
Cheers,
  Andrej



Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-21 Thread Nilesh Patra
On Fri, Oct 21, 2022 at 07:24:59PM +0200, Andrej Shadura wrote:
> On Fri, 21 Oct 2022, at 19:01, Andreas Metzler wrote:
> > On 2022-10-21 Andreas Metzler  wrote:
> >> Looks like this could be https://github.com/pkgconf/pkgconf/issues/267
> >> which was fixed in June
> >> https://github.com/pkgconf/pkgconf/commit/a61193c7236f5b240585c4f8eef6f452f1d9a7ee
> >> but has not hit a release yet.
> >
> > Indeed applying the upstream patch fixes the testcase Nilesh provided.
> >
> > I have not tried rebuilding jellyfish with patched pkgconf.
> 
> Thanks for confirming, I was looking at the same upstream commit, and 
> actually I have just uploaded it to experimental.

D'you plan to rebuild this, and other few currently failing packages w/ the new 
pkgconf release?

-- 
Best,
Nilesh


signature.asc
Description: PGP signature


Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-21 Thread Andrej Shadura
Hi,

On Fri, 21 Oct 2022, at 19:01, Andreas Metzler wrote:
> On 2022-10-21 Andreas Metzler  wrote:
>> Looks like this could be https://github.com/pkgconf/pkgconf/issues/267
>> which was fixed in June
>> https://github.com/pkgconf/pkgconf/commit/a61193c7236f5b240585c4f8eef6f452f1d9a7ee
>> but has not hit a release yet.
>
> Indeed applying the upstream patch fixes the testcase Nilesh provided.
>
> I have not tried rebuilding jellyfish with patched pkgconf.

Thanks for confirming, I was looking at the same upstream commit, and actually 
I have just uploaded it to experimental.

-- 
Cheers,
  Andrej



Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-21 Thread Andreas Metzler
On 2022-10-21 Andreas Metzler  wrote:
> On 2022-10-20 Nilesh Patra  wrote:
>> On Wed, 19 Oct 2022 21:11:12 +0100 Andrej Shadura  
>> wrote:
>> > Source: jellyfish
> []

>> I took a look at this one, and this seems to FTBFS with pkgconf
>> because pkgconf does not seem to honor atleast PKG_CONFIG_SYSROOT_DIR
>> which is set in d/rules for this package[1]
> [...]
>> with old pkg-config:

>> $ PKG_CONFIG_PATH=. PKG_CONFIG_SYSROOT_DIR=debian/tmp/ pkg-config --cflags 
>> jellyfish-2.0
>> -Idebian/tmp//usr/include/jellyfish-2.3.0

>> with pkgconf:

>> $ PKG_CONFIG_PATH=. PKG_CONFIG_SYSROOT_DIR=debian/tmp/ pkg-config --cflags 
>> jellyfish-2.0
>> -I/usr/include/jellyfish-2.3.0
> [...]

> Hello,

> Looks like this could be https://github.com/pkgconf/pkgconf/issues/267
> which was fixed in June
> https://github.com/pkgconf/pkgconf/commit/a61193c7236f5b240585c4f8eef6f452f1d9a7ee
> but has not hit a release yet.

Indeed applying the upstream patch fixes the testcase Nilesh provided.

I have not tried rebuilding jellyfish with patched pkgconf.

cu Andreas

-- 
`What a good friend you are to him, Dr. Maturin. His other friends are
so grateful to you.'
`I sew his ears on from time to time, sure'



Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-21 Thread Andreas Metzler
On 2022-10-20 Nilesh Patra  wrote:
> On Wed, 19 Oct 2022 21:11:12 +0100 Andrej Shadura  wrote:
> > Source: jellyfish
[]

> I took a look at this one, and this seems to FTBFS with pkgconf
> because pkgconf does not seem to honor atleast PKG_CONFIG_SYSROOT_DIR
> which is set in d/rules for this package[1]
[...]
> with old pkg-config:

> $ PKG_CONFIG_PATH=. PKG_CONFIG_SYSROOT_DIR=debian/tmp/ pkg-config --cflags 
> jellyfish-2.0
> -Idebian/tmp//usr/include/jellyfish-2.3.0

> with pkgconf:

> $ PKG_CONFIG_PATH=. PKG_CONFIG_SYSROOT_DIR=debian/tmp/ pkg-config --cflags 
> jellyfish-2.0
> -I/usr/include/jellyfish-2.3.0
[...]

Hello,

Looks like this could be https://github.com/pkgconf/pkgconf/issues/267
which was fixed in June
https://github.com/pkgconf/pkgconf/commit/a61193c7236f5b240585c4f8eef6f452f1d9a7ee
but has not hit a release yet.

cu Andreas
-- 
`What a good friend you are to him, Dr. Maturin. His other friends are
so grateful to you.'
`I sew his ears on from time to time, sure'



Bug#1022087: jellyfish: FTBFS with pkgconf

2022-10-20 Thread Nilesh Patra
Hi Andrej,

On Wed, 19 Oct 2022 21:11:12 +0100 Andrej Shadura  wrote:
> Source: jellyfish
> Version: 2.3.0-15
> Severity: normal
> Tags: ftbfs
> User: andre...@debian.org
> Usertags: pkgconf-rebuild-ftbfs

I took a look at this one, and this seems to FTBFS with pkgconf because pkgconf 
does not seem
to honor atleast PKG_CONFIG_SYSROOT_DIR which is set in d/rules for this 
package[1]

In particular:

| $ cat jellyfish-2.0.pc
| prefix=/usr
| exec_prefix=${prefix}
| libdir=${prefix}/lib/x86_64-linux-gnu
| includedir=${prefix}/include
|
| Name: Jellyfish
| Description: A multi-threaded hash based k-mer counter.
| Version: 2.3.0
| Libs: -L${libdir} -ljellyfish-2.0 -lpthr

with old pkg-config:

$ PKG_CONFIG_PATH=. PKG_CONFIG_SYSROOT_DIR=debian/tmp/ pkg-config --cflags 
jellyfish-2.0
-Idebian/tmp//usr/include/jellyfish-2.3.0

with pkgconf:

$ PKG_CONFIG_PATH=. PKG_CONFIG_SYSROOT_DIR=debian/tmp/ pkg-config --cflags 
jellyfish-2.0
-I/usr/include/jellyfish-2.3.0

This looks like a problem with pkgconf instead. Let me know if I am mistaken.
If not, could you fix this, please?

[1]: https://sources.debian.org/src/jellyfish/2.3.0-15/debian/rules/#L12

Thanks!

-- 
Best,
Nilesh


signature.asc
Description: PGP signature