unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#28347] Cadaver
@ 2017-09-04 18:11 ng0
  2017-09-19 16:43 ` Alex Kost
  0 siblings, 1 reply; 6+ messages in thread
From: ng0 @ 2017-09-04 18:11 UTC (permalink / raw)
  To: 28347


[-- Attachment #1.1: Type: text/plain, Size: 264 bytes --]

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

[-- Attachment #1.2: 0001-gnu-Add-cadaver.patch --]
[-- Type: text/plain, Size: 2407 bytes --]

From 29e1cce35ad65fb1f916425994910b49090b0d94 Mon Sep 17 00:00:00 2001
From: ng0 <contact.ng0@cryptolab.net>
Date: Wed, 1 Feb 2017 23:14:56 +0000
Subject: [PATCH] gnu: Add cadaver.

* gnu/packages/web.scm (cadaver): New variable.
---
 gnu/packages/web.scm | 34 +++++++++++++++++++++++++++++++++-
 1 file changed, 33 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/web.scm b/gnu/packages/web.scm
index ac65a8536..ffce97ca5 100644
--- a/gnu/packages/web.scm
+++ b/gnu/packages/web.scm
@@ -13,7 +13,7 @@
 ;;; Copyright © 2016 Rene Saavedra <rennes@openmailbox.org>
 ;;; Copyright © 2016 Ben Woodcroft <donttrustben@gmail.com>
 ;;; Copyright © 2016 Clément Lassieur <clement@lassieur.org>
-;;; Copyright © 2016, 2017 ng0 <ng0@no-reply.pragmatique.xyz>
+;;; Copyright © 2016, 2017 ng0 <ng0@infotropique.org>
 ;;; Copyright © 2016, 2017 Arun Isaac <arunisaac@systemreboot.net>
 ;;; Copyright © 2016 Tobias Geerinckx-Rice <me@tobias.gr>
 ;;; Copyright © 2016 Bake Timmons <b3timmons@speedymail.org>
@@ -5105,3 +5105,35 @@ file links.")
                    ;; https://github.com/wummel/linkchecker/issues/729
                    l:isc   ; third_party/dnspython
                    l:asl2.0)))) ; third_party/miniboa
+
+(define-public cadaver
+  (package
+    (name "cadaver")
+    (version "0.23.3")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "http://www.webdav.org/cadaver/"
+                           name "-" version ".tar.gz"))
+       (sha256
+        (base32
+         "1jizq69ifrjbjvz5y79wh1ny94gsdby4gdxwjad4bfih6a5fck7x"))))
+    (build-system gnu-build-system)
+    ;; TODO: Unbundle libneon and make build succeed with new neon.
+    (arguments
+     `(#:configure-flags (list "--with-ssl=openssl")
+       #:tests? #f)) ;No tests included
+    (native-inputs
+     `(("gettext" ,gnu-gettext)
+       ("pkg-config" ,pkg-config)
+       ("intltool" ,intltool)))
+    (inputs
+     `(("expat" ,expat)
+       ("openssl" ,openssl)))
+    (home-page "http://www.webdav.org/cadaver")
+    (synopsis "Command-line WebDAV client")
+    (description
+     "Cadaver is a command-line WebDAV client for Unix. It supports
+file upload, download, on-screen display, namespace operations (move/copy),
+collection creation and deletion, and locking operations.")
+    (license l:gpl2)))
-- 
2.14.1


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#28347] Cadaver
  2017-09-04 18:11 [bug#28347] Cadaver ng0
@ 2017-09-19 16:43 ` Alex Kost
  2017-09-19 18:17   ` ng0
                     ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Alex Kost @ 2017-09-19 16:43 UTC (permalink / raw)
  To: ng0; +Cc: 28347

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#28347] Cadaver
  2017-09-19 16:43 ` Alex Kost
@ 2017-09-19 18:17   ` ng0
  2017-09-19 21:25   ` Ludovic Courtès
  2017-09-28 15:53   ` bug#28347: Cadaver Alex Kost
  2 siblings, 0 replies; 6+ messages in thread
From: ng0 @ 2017-09-19 18:17 UTC (permalink / raw)
  To: Alex Kost; +Cc: 28347

[-- Attachment #1: Type: text/plain, Size: 2325 bytes --]

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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#28347] Cadaver
  2017-09-19 16:43 ` Alex Kost
  2017-09-19 18:17   ` ng0
@ 2017-09-19 21:25   ` Ludovic Courtès
  2017-09-20 16:35     ` Alex Kost
  2017-09-28 15:53   ` bug#28347: Cadaver Alex Kost
  2 siblings, 1 reply; 6+ messages in thread
From: Ludovic Courtès @ 2017-09-19 21:25 UTC (permalink / raw)
  To: Alex Kost; +Cc: 28347

Alex Kost <alezost@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’.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#28347] Cadaver
  2017-09-19 21:25   ` Ludovic Courtès
@ 2017-09-20 16:35     ` Alex Kost
  0 siblings, 0 replies; 6+ messages in thread
From: Alex Kost @ 2017-09-20 16:35 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 28347

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

> Alex Kost <alezost@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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#28347: Cadaver
  2017-09-19 16:43 ` Alex Kost
  2017-09-19 18:17   ` ng0
  2017-09-19 21:25   ` Ludovic Courtès
@ 2017-09-28 15:53   ` Alex Kost
  2 siblings, 0 replies; 6+ messages in thread
From: Alex Kost @ 2017-09-28 15:53 UTC (permalink / raw)
  To: ng0; +Cc: 28347-done

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2017-09-28 15:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-09-04 18:11 [bug#28347] Cadaver ng0
2017-09-19 16:43 ` Alex Kost
2017-09-19 18:17   ` ng0
2017-09-19 21:25   ` Ludovic Courtès
2017-09-20 16:35     ` Alex Kost
2017-09-28 15:53   ` bug#28347: Cadaver Alex Kost

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).