GNU bug report logs - #28347
Cadaver

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 28347 in the body.
You can then email your comments to 28347 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> infotropique.org>
To: guix-patches <at> gnu.org
Subject: Cadaver
Date: Mon, 4 Sep 2017 18:11:13 +0000

[Message part 1 (text/plain, inline)]

A package from my "crypt". I tried various ways
to unbundle neon but did not succeed.
The package itself works.
-- 
ng0
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://n0is.noblogs.org/my-keys
https://www.infotropique.org https://krosos.org

[0001-gnu-Add-cadaver.patch (text/plain, attachment)]

[signature.asc (application/pgp-signature, inline)]

Message #8 received at 28347 <at> debbugs.gnu.org (full text, mbox):

From: Alex Kost <alezost <at> gmail.com>
To: ng0 <ng0 <at> infotropique.org>
Cc: 28347 <at> debbugs.gnu.org
Subject: Re: [bug#28347] Cadaver
Date: Tue, 19 Sep 2017 19:43:08 +0300

ng0 (2017-09-04 18:11 +0000) wrote:

> A package from my "crypt". I tried various ways
> to unbundle neon but did not succeed.

I think it would be good if you wrote what you tried :-)

Here are my attempts:

At first, there are the following messages during "configure" phase:

  configure: no external neon library found
  configure: using bundled neon (0.29.1)

So, when I added ("neon" ,neon) input, I got:

  configure: incompatible neon library version 0.30.2: wanted 0.27 28 29
  configure: using bundled neon (0.29.1)

Ouch, so 0.30.2 is not supported.  This is the line from "configure.ac":

  NE_REQUIRE_VERSIONS([0], [27 28 29])

OK, I tried to build it with 0.30.2 anyway using the following phase:

         (add-after 'unpack 'fix-required-neon-version
           (lambda _
             (substitute* "configure"
               (("27 28 29")
                ,(string-join
                  (cdr (string-split (package-version neon) #\.))
                  ".")))
             #t))

But I got:

  configure: incompatible neon library version 0.30.2: wanted 0.30.2
  configure: using bundled neon (0.29.1)

What?  Well, now I don't know what to do (perhaps I should have patched
"configure.ac" instead and added "autoreconf" phase, but I didn't try).

> The package itself works.

Great, I think if there will be no comments in a week or so, I will just
go ahead and commit your patch (in a hope this unbundling will happen in
the future somehow).  So if anyone has something to say about this
patch, please do it now :-)

-- 
Alex

Message #11 received at 28347 <at> debbugs.gnu.org (full text, mbox):

From: ng0 <ng0 <at> infotropique.org>
To: Alex Kost <alezost <at> gmail.com>
Cc: 28347 <at> debbugs.gnu.org, ng0 <ng0 <at> infotropique.org>
Subject: Re: [bug#28347] Cadaver
Date: Tue, 19 Sep 2017 18:17:49 +0000

[Message part 1 (text/plain, inline)]

Alex Kost transcribed 1.5K bytes:
> ng0 (2017-09-04 18:11 +0000) wrote:
> 
> > A package from my "crypt". I tried various ways
> > to unbundle neon but did not succeed.
> 
> I think it would be good if you wrote what you tried :-)
> 
> Here are my attempts:
> 
> At first, there are the following messages during "configure" phase:
> 
>   configure: no external neon library found
>   configure: using bundled neon (0.29.1)
> 
> So, when I added ("neon" ,neon) input, I got:
> 
>   configure: incompatible neon library version 0.30.2: wanted 0.27 28 29
>   configure: using bundled neon (0.29.1)
> 
> Ouch, so 0.30.2 is not supported.  This is the line from "configure.ac":
> 
>   NE_REQUIRE_VERSIONS([0], [27 28 29])
> 
> OK, I tried to build it with 0.30.2 anyway using the following phase:
> 
>          (add-after 'unpack 'fix-required-neon-version
>            (lambda _
>              (substitute* "configure"
>                (("27 28 29")
>                 ,(string-join
>                   (cdr (string-split (package-version neon) #\.))
>                   ".")))
>              #t))
> 
> But I got:
> 
>   configure: incompatible neon library version 0.30.2: wanted 0.30.2
>   configure: using bundled neon (0.29.1)
> 
> What?  Well, now I don't know what to do (perhaps I should have patched
> "configure.ac" instead and added "autoreconf" phase, but I didn't try).

Basically at some point I tried to replicate
https://gitweb.gentoo.org/repo/gentoo.git/tree/net-misc/cadaver/cadaver-0.23.3.ebuild
which worked but then I messed it up by unbundling more
than just neon. And then many months back I had no regular
use for the package anymore.

What Gentoo does seems to work as Gentoo is on neon 30.2

So, I tried exactly what you did, it just had been so
long ago that I worked on this.

> > The package itself works.
> 
> Great, I think if there will be no comments in a week or so, I will just
> go ahead and commit your patch (in a hope this unbundling will happen in
> the future somehow).  So if anyone has something to say about this
> patch, please do it now :-)
> 
> -- 
> Alex
> 

Thanks
-- 
ng0
GnuPG: A88C8ADD129828D7EAC02E52E22F9BBFEE348588
GnuPG: https://krosos.org/dist/keys/
https://www.infotropique.org https://www.krosos.org

[signature.asc (application/pgp-signature, inline)]

Message #14 received at 28347 <at> debbugs.gnu.org (full text, mbox):

From: ludo <at> gnu.org (Ludovic Courtès)
To: Alex Kost <alezost <at> gmail.com>
Cc: 28347 <at> debbugs.gnu.org, ng0 <ng0 <at> infotropique.org>
Subject: Re: [bug#28347] Cadaver
Date: Tue, 19 Sep 2017 23:25:12 +0200

Alex Kost <alezost <at> gmail.com> skribis:

> Ouch, so 0.30.2 is not supported.  This is the line from "configure.ac":
>
>   NE_REQUIRE_VERSIONS([0], [27 28 29])
>
> OK, I tried to build it with 0.30.2 anyway using the following phase:
>
>          (add-after 'unpack 'fix-required-neon-version
>            (lambda _
>              (substitute* "configure"
>                (("27 28 29")
>                 ,(string-join
>                   (cdr (string-split (package-version neon) #\.))
>                   ".")))
>              #t))
>
> But I got:
>
>   configure: incompatible neon library version 0.30.2: wanted 0.30.2
>   configure: using bundled neon (0.29.1)

In ‘configure’, I think you need to replace “27 28 29” with “30”, not
“30.2”, because of the way NEON_CHECK_VERSION in m4/neon.m4 matches
versions:

       for v in ne_require_minor; do
          case $ne_libver in
          ne_require_major.$v.*) ne_cv_lib_neonver=yes ;;
          esac
       done

I’m sure you’re almost there.  :-)

Thanks,
Ludo’.

Message #17 received at 28347 <at> debbugs.gnu.org (full text, mbox):

From: Alex Kost <alezost <at> gmail.com>
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 28347 <at> debbugs.gnu.org, ng0 <ng0 <at> infotropique.org>
Subject: Re: [bug#28347] Cadaver
Date: Wed, 20 Sep 2017 19:35:17 +0300

Ludovic Courtès (2017-09-19 23:25 +0200) wrote:

> Alex Kost <alezost <at> gmail.com> skribis:
>
>> Ouch, so 0.30.2 is not supported.  This is the line from "configure.ac":
>>
>>   NE_REQUIRE_VERSIONS([0], [27 28 29])
>>
>> OK, I tried to build it with 0.30.2 anyway using the following phase:
>>
>>          (add-after 'unpack 'fix-required-neon-version
>>            (lambda _
>>              (substitute* "configure"
>>                (("27 28 29")
>>                 ,(string-join
>>                   (cdr (string-split (package-version neon) #\.))
>>                   ".")))
>>              #t))
>>
>> But I got:
>>
>>   configure: incompatible neon library version 0.30.2: wanted 0.30.2
>>   configure: using bundled neon (0.29.1)
>
> In ‘configure’, I think you need to replace “27 28 29” with “30”, not
> “30.2”, because of the way NEON_CHECK_VERSION in m4/neon.m4 matches
> versions:
>
>        for v in ne_require_minor; do
>           case $ne_libver in
>           ne_require_major.$v.*) ne_cv_lib_neonver=yes ;;
>           esac
>        done

Yeah, I tried to use "30", but it failed as well.

> I’m sure you’re almost there.  :-)

Actually, I don't have a wish to continue, sorry :-)

-- 
Alex

Message #22 received at 28347-done <at> debbugs.gnu.org (full text, mbox):

From: Alex Kost <alezost <at> gmail.com>
To: ng0 <ng0 <at> infotropique.org>
Cc: 28347-done <at> debbugs.gnu.org
Subject: Re: [bug#28347] Cadaver
Date: Thu, 28 Sep 2017 18:53:56 +0300

Alex Kost (2017-09-19 19:43 +0300) wrote:

> ng0 (2017-09-04 18:11 +0000) wrote:

>> A package from my "crypt". I tried various ways
>> to unbundle neon but did not succeed.
>> The package itself works.
[...]
> Great, I think if there will be no comments in a week or so, I will just
> go ahead and commit your patch (in a hope this unbundling will happen in
> the future somehow).

So I went ahead and applied this patch, thank you ng0!

Otherwise, I'm afraid this patch would stuck in <guix-patches> for a
very long time.  Please let me know if I was wrong (I mean if I should
have not committed it).

-- 
Alex

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.