unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
@ 2017-11-26 17:07 Clément Lassieur
  2017-11-26 20:51 ` Leo Famulari
  2017-11-26 22:15 ` Ludovic Courtès
  0 siblings, 2 replies; 8+ messages in thread
From: Clément Lassieur @ 2017-11-26 17:07 UTC (permalink / raw)
  To: 29457

* gnu/packages/emacs.scm (emacs-org-contrib)[source]: Fix sha256 checksum.
---
 gnu/packages/emacs.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index acd6ec30a..7e16c8b4b 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -4230,7 +4230,7 @@ reproducible research.")
                                   (package-version emacs-org) ".tar"))
               (sha256
                (base32
-                "0xy2xrndlhs4kyvh6mmv24dnh3fn5p63d2gaimnrypf1p8znwzh4"))))
+                "071vqv6hdyjp85ap39930782ks07ypjzch81r8kax3ybwfrf0chx"))))
     (arguments
      `(#:modules ((guix build emacs-build-system)
                   (guix build utils)
-- 
2.15.0

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

* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
  2017-11-26 17:07 [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update Clément Lassieur
@ 2017-11-26 20:51 ` Leo Famulari
  2017-11-26 22:15 ` Ludovic Courtès
  1 sibling, 0 replies; 8+ messages in thread
From: Leo Famulari @ 2017-11-26 20:51 UTC (permalink / raw)
  To: Clément Lassieur; +Cc: 29457

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

On Sun, Nov 26, 2017 at 06:07:55PM +0100, Clément Lassieur wrote:
> * gnu/packages/emacs.scm (emacs-org-contrib)[source]: Fix sha256 checksum.

Can you add a reminder comment to emacs-org so that this doesn't happen
again? :)

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

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

* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
  2017-11-26 17:07 [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update Clément Lassieur
  2017-11-26 20:51 ` Leo Famulari
@ 2017-11-26 22:15 ` Ludovic Courtès
  2017-11-26 22:35   ` Leo Famulari
  1 sibling, 1 reply; 8+ messages in thread
From: Ludovic Courtès @ 2017-11-26 22:15 UTC (permalink / raw)
  To: Clément Lassieur, Nicolas Goaziou; +Cc: 29457

Clément Lassieur <clement@lassieur.org> skribis:

> * gnu/packages/emacs.scm (emacs-org-contrib)[source]: Fix sha256 checksum.
> ---
>  gnu/packages/emacs.scm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index acd6ec30a..7e16c8b4b 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -4230,7 +4230,7 @@ reproducible research.")
>                                    (package-version emacs-org) ".tar"))
>                (sha256
>                 (base32
> -                "0xy2xrndlhs4kyvh6mmv24dnh3fn5p63d2gaimnrypf1p8znwzh4"))))
> +                "071vqv6hdyjp85ap39930782ks07ypjzch81r8kax3ybwfrf0chx"))))

Was the SHA256 simply erroneous, or was the file modified in-place
upstream?

It’s a good idea to investigate a bit in such cases IMO.

Ludo’.

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

* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
  2017-11-26 22:15 ` Ludovic Courtès
@ 2017-11-26 22:35   ` Leo Famulari
  2017-11-28  2:43     ` Clément Lassieur
  0 siblings, 1 reply; 8+ messages in thread
From: Leo Famulari @ 2017-11-26 22:35 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 29457, Clément Lassieur

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

On Sun, Nov 26, 2017 at 11:15:41PM +0100, Ludovic Courtès wrote:
> Clément Lassieur <clement@lassieur.org> skribis:
> 
> > * gnu/packages/emacs.scm (emacs-org-contrib)[source]: Fix sha256 checksum.
> > ---
> >  gnu/packages/emacs.scm | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> > index acd6ec30a..7e16c8b4b 100644
> > --- a/gnu/packages/emacs.scm
> > +++ b/gnu/packages/emacs.scm
> > @@ -4230,7 +4230,7 @@ reproducible research.")
> >                                    (package-version emacs-org) ".tar"))
> >                (sha256
> >                 (base32
> > -                "0xy2xrndlhs4kyvh6mmv24dnh3fn5p63d2gaimnrypf1p8znwzh4"))))
> > +                "071vqv6hdyjp85ap39930782ks07ypjzch81r8kax3ybwfrf0chx"))))
> 
> Was the SHA256 simply erroneous, or was the file modified in-place
> upstream?
> 
> It’s a good idea to investigate a bit in such cases IMO.

I assumed this was a case where a package FOO inherits another package
BAR's version, and BAR was updated, leaving FOO with a broken source.

Otherwise, yes, all hash mismatches should be investigated and reported
upstream.

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

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

* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
  2017-11-26 22:35   ` Leo Famulari
@ 2017-11-28  2:43     ` Clément Lassieur
  2017-11-28 17:00       ` Nicolas Goaziou
  0 siblings, 1 reply; 8+ messages in thread
From: Clément Lassieur @ 2017-11-28  2:43 UTC (permalink / raw)
  To: Leo Famulari; +Cc: 29457

Leo Famulari <leo@famulari.name> writes:
> On Sun, Nov 26, 2017 at 11:15:41PM +0100, Ludovic Courtès wrote:
>>
>> Was the SHA256 simply erroneous, or was the file modified in-place
>> upstream?
>> 
>> It’s a good idea to investigate a bit in such cases IMO.
>
> I assumed this was a case where a package FOO inherits another package
> BAR's version, and BAR was updated, leaving FOO with a broken source.
>
> Otherwise, yes, all hash mismatches should be investigated and reported
> upstream.

Exactly.  I'll put a comment, as suggested by Leo, so that we don't
forget to update it anymore.

Thanks,
Clément

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

* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
  2017-11-28  2:43     ` Clément Lassieur
@ 2017-11-28 17:00       ` Nicolas Goaziou
  2017-12-10 14:33         ` Clément Lassieur
  0 siblings, 1 reply; 8+ messages in thread
From: Nicolas Goaziou @ 2017-11-28 17:00 UTC (permalink / raw)
  To: Clément Lassieur; +Cc: 29457

Hello,

Clément Lassieur <clement@lassieur.org> writes:

> Leo Famulari <leo@famulari.name> writes:
>> On Sun, Nov 26, 2017 at 11:15:41PM +0100, Ludovic Courtès wrote:
>>>
>>> Was the SHA256 simply erroneous, or was the file modified in-place
>>> upstream?
>>> 
>>> It’s a good idea to investigate a bit in such cases IMO.
>>
>> I assumed this was a case where a package FOO inherits another package
>> BAR's version, and BAR was updated, leaving FOO with a broken source.
>>
>> Otherwise, yes, all hash mismatches should be investigated and reported
>> upstream.
>
> Exactly.  I'll put a comment, as suggested by Leo, so that we don't
> forget to update it anymore.

But wouldn't it make more sense, in this case, to merge both packages
and let "contrib" be an output for emacs-org?

Regards,

-- 
Nicolas Goaziou                                                0x80A93738

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

* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
  2017-11-28 17:00       ` Nicolas Goaziou
@ 2017-12-10 14:33         ` Clément Lassieur
  2017-12-19 16:36           ` Leo Famulari
  0 siblings, 1 reply; 8+ messages in thread
From: Clément Lassieur @ 2017-12-10 14:33 UTC (permalink / raw)
  To: Nicolas Goaziou; +Cc: 29457

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

> Hello,
>
> Clément Lassieur <clement@lassieur.org> writes:
>
>> Leo Famulari <leo@famulari.name> writes:
>>> On Sun, Nov 26, 2017 at 11:15:41PM +0100, Ludovic Courtès wrote:
>>>>
>>>> Was the SHA256 simply erroneous, or was the file modified in-place
>>>> upstream?
>>>> 
>>>> It’s a good idea to investigate a bit in such cases IMO.
>>>
>>> I assumed this was a case where a package FOO inherits another package
>>> BAR's version, and BAR was updated, leaving FOO with a broken source.
>>>
>>> Otherwise, yes, all hash mismatches should be investigated and reported
>>> upstream.
>>
>> Exactly.  I'll put a comment, as suggested by Leo, so that we don't
>> forget to update it anymore.
>
> But wouldn't it make more sense, in this case, to merge both packages
> and let "contrib" be an output for emacs-org?

Indeed, to me it makes sense, but Leo knows much more than me about
packages, so I cc'ed him.

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

* [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update.
  2017-12-10 14:33         ` Clément Lassieur
@ 2017-12-19 16:36           ` Leo Famulari
  0 siblings, 0 replies; 8+ messages in thread
From: Leo Famulari @ 2017-12-19 16:36 UTC (permalink / raw)
  To: Clément Lassieur; +Cc: 29457

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

On Sun, Dec 10, 2017 at 03:33:08PM +0100, Clément Lassieur wrote:
> Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:
> > But wouldn't it make more sense, in this case, to merge both packages
> > and let "contrib" be an output for emacs-org?
> 
> Indeed, to me it makes sense, but Leo knows much more than me about
> packages, so I cc'ed him.

I think it's a matter of taste, depending on the package. I'm not
familiar with the Emacs packages, so I leave it up to you Emacs users :)

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

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

end of thread, other threads:[~2017-12-19 16:37 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-26 17:07 [bug#29457] [PATCH] gnu: emacs-org-contrib: Fix sha256 checksum due to emacs-org update Clément Lassieur
2017-11-26 20:51 ` Leo Famulari
2017-11-26 22:15 ` Ludovic Courtès
2017-11-26 22:35   ` Leo Famulari
2017-11-28  2:43     ` Clément Lassieur
2017-11-28 17:00       ` Nicolas Goaziou
2017-12-10 14:33         ` Clément Lassieur
2017-12-19 16:36           ` Leo Famulari

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).