all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: mpop: Update to 1.2.6.
@ 2016-11-17 11:43 Arun Isaac
  2016-11-17 16:46 ` Marius Bakke
  0 siblings, 1 reply; 6+ messages in thread
From: Arun Isaac @ 2016-11-17 11:43 UTC (permalink / raw)
  To: guix-devel


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


Please find attached patch to update mpop from 1.2.5 to 1.2.6.

Thanks,
Arun


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-mpop-Update-to-1.2.6.patch --]
[-- Type: text/x-patch, Size: 1114 bytes --]

From 5cb8a651b9eb97b8f6db36bb479140d6ed5692c1 Mon Sep 17 00:00:00 2001
From: Arun Isaac <arunisaac@systemreboot.net>
Date: Thu, 17 Nov 2016 13:34:42 +0530
Subject: [PATCH] gnu: mpop: Update to 1.2.6.

* gnu/packages/mail.scm (mpop): Update to 1.2.6.
---
 gnu/packages/mail.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index 5d304d5..c294e20 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -1471,7 +1471,7 @@ converts them to maildir format directories.")
 (define-public mpop
   (package
     (name "mpop")
-    (version "1.2.5")
+    (version "1.2.6")
     (source
      (origin
        (method url-fetch)
@@ -1479,7 +1479,7 @@ converts them to maildir format directories.")
                            version ".tar.xz"))
        (sha256
         (base32
-         "0n0ij258kn8lfa6nyr6l6plc4hf1wvyf1hkwicvdbjqdqrgjnq81"))))
+         "0p1ix63jh64dibrlccch8q7gxl9nn18wd2qpyr5z1h4gs2fpmv4z"))))
     (build-system gnu-build-system)
     (inputs
      `(("gnutls" ,gnutls)
-- 
2.10.2


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

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

* Re: [PATCH] gnu: mpop: Update to 1.2.6.
  2016-11-17 11:43 [PATCH] gnu: mpop: Update to 1.2.6 Arun Isaac
@ 2016-11-17 16:46 ` Marius Bakke
  2016-11-17 19:46   ` Arun Isaac
  0 siblings, 1 reply; 6+ messages in thread
From: Marius Bakke @ 2016-11-17 16:46 UTC (permalink / raw)
  To: Arun Isaac, guix-devel

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

Arun Isaac <arunisaac@systemreboot.net> writes:

> Please find attached patch to update mpop from 1.2.5 to 1.2.6.

Thanks for this. I'm unable to download the new version with
`./pre-inst-env guix build mpop`; however downloading manually with
`guix download` works fine and returns the hash, so I pushed the patch.

Here is the output from `./pre-inst-env guix build`, it appears to give
up after the redirect to HTTPS from sourceforge.

@ build-started /gnu/store/10wpp69fc0v7dk0mqk3075cy9d7iwwmh-mpop-1.2.6.tar.xz.drv - x86_64-linux /var/log/guix/drvs/10//wpp69fc0v7dk0mqk3075cy9d7iwwmh-mpop-1.2.6.tar.xz.drv.bz2

Starting download of /gnu/store/nxf0mgh6dynniy99gsxhff9xr217s1rl-mpop-1.2.6.tar.xz
From http://downloads.sourceforge.net/mpop/mpop-1.2.6.tar.xz...
following redirection to `http://downloads.sourceforge.net/project/mpop/mpop/1.2.6/mpop-1.2.6.tar.xz'...
following redirection to `https://netcologne.dl.sourceforge.net/project/mpop/mpop/1.2.6/mpop-1.2.6.tar.xz'...
;;; Failed to autoload make-session in (gnutls):
;;; ERROR: missing interface for module (gnutls)
ERROR: In procedure module-lookup: Unbound variable: make-session

Starting download of /gnu/store/nxf0mgh6dynniy99gsxhff9xr217s1rl-mpop-1.2.6.tar.xz
From http://mirror.hydra.gnu.org/file/mpop-1.2.6.tar.xz/sha256/0p1ix63jh64dibrlccch8q7gxl9nn18wd2qpyr5z1h4gs2fpmv4z...
[...]

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

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

* Re: [PATCH] gnu: mpop: Update to 1.2.6.
  2016-11-17 16:46 ` Marius Bakke
@ 2016-11-17 19:46   ` Arun Isaac
  2016-11-17 20:46     ` Marius Bakke
  0 siblings, 1 reply; 6+ messages in thread
From: Arun Isaac @ 2016-11-17 19:46 UTC (permalink / raw)
  To: guix-devel

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


Marius Bakke writes:

> Thanks for this. I'm unable to download the new version with
> `./pre-inst-env guix build mpop`;

Not sure what the problem is. I was able to build mpop successfully on
my system. But, I did download manually with `guix download'
first. Perhaps the build used that downloaded source tarball.

> however downloading manually with
> `guix download` works fine and returns the hash, so I pushed the patch.

Thank you.

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

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

* Re: [PATCH] gnu: mpop: Update to 1.2.6.
  2016-11-17 19:46   ` Arun Isaac
@ 2016-11-17 20:46     ` Marius Bakke
  2016-11-17 21:09       ` Leo Famulari
  0 siblings, 1 reply; 6+ messages in thread
From: Marius Bakke @ 2016-11-17 20:46 UTC (permalink / raw)
  To: Arun Isaac, guix-devel

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

Arun Isaac <arunisaac@systemreboot.net> writes:

> Marius Bakke writes:
>
>> Thanks for this. I'm unable to download the new version with
>> `./pre-inst-env guix build mpop`;
>
> Not sure what the problem is. I was able to build mpop successfully on
> my system. But, I did download manually with `guix download'
> first. Perhaps the build used that downloaded source tarball.

Yes, if the hash already exists in the store, it will be used instead of
downloading.  This has caused some confusion before :-)

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

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

* Re: [PATCH] gnu: mpop: Update to 1.2.6.
  2016-11-17 20:46     ` Marius Bakke
@ 2016-11-17 21:09       ` Leo Famulari
  2016-11-19  2:24         ` Arun Isaac
  0 siblings, 1 reply; 6+ messages in thread
From: Leo Famulari @ 2016-11-17 21:09 UTC (permalink / raw)
  To: Marius Bakke; +Cc: guix-devel

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

On Thu, Nov 17, 2016 at 09:46:38PM +0100, Marius Bakke wrote:
> Arun Isaac <arunisaac@systemreboot.net> writes:
> > Marius Bakke writes:
> >
> >> Thanks for this. I'm unable to download the new version with
> >> `./pre-inst-env guix build mpop`;
> >
> > Not sure what the problem is. I was able to build mpop successfully on
> > my system. But, I did download manually with `guix download'
> > first. Perhaps the build used that downloaded source tarball.

I fixed the URL and made it use our SourceForge mirror in commit
5837e6993.

> Yes, if the hash already exists in the store, it will be used instead of
> downloading.  This has caused some confusion before :-)

As Marius hints at, using `guix download` to get source code while
writing a package masks errors in the package's URL. You should always
download the source code "out of band" with wget, curl, a web browser,
etc, and then use `guix hash` to calculate the hash.

That way, you can be sure the package works.

Also, you don't leak information (via `guix download`'s network
signature) to potentially hostile servers that you are packaging
software for Guix.

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

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

* Re: [PATCH] gnu: mpop: Update to 1.2.6.
  2016-11-17 21:09       ` Leo Famulari
@ 2016-11-19  2:24         ` Arun Isaac
  0 siblings, 0 replies; 6+ messages in thread
From: Arun Isaac @ 2016-11-19  2:24 UTC (permalink / raw)
  To: guix-devel

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


> I fixed the URL and made it use our SourceForge mirror in commit
> 5837e6993.

Thanks.

>> Yes, if the hash already exists in the store, it will be used instead of
>> downloading.  This has caused some confusion before :-)
>
> As Marius hints at, using `guix download` to get source code while
> writing a package masks errors in the package's URL. You should always
> download the source code "out of band" with wget, curl, a web browser,
> etc, and then use `guix hash` to calculate the hash.

Sure, will do from next time...!

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

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

end of thread, other threads:[~2016-11-19  2:25 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-17 11:43 [PATCH] gnu: mpop: Update to 1.2.6 Arun Isaac
2016-11-17 16:46 ` Marius Bakke
2016-11-17 19:46   ` Arun Isaac
2016-11-17 20:46     ` Marius Bakke
2016-11-17 21:09       ` Leo Famulari
2016-11-19  2:24         ` Arun Isaac

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.