all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#44863: Warning about importing a MELPA package
@ 2020-11-25  9:57 Zhu Zihao
  2020-12-07  9:17 ` Carlo Zancanaro
  0 siblings, 1 reply; 3+ messages in thread
From: Zhu Zihao @ 2020-11-25  9:57 UTC (permalink / raw)
  To: 44863

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

ELPA importer supports MELPA[1] currently. But MELPA is a rolling
archive, which does not persist any old version tarball of
package, and harmful for reproducible build.

It's still useful for Guix packager to import package from MELPA to
draft a sketch of Emacs package, but it's not a reliable download
service. We may better warn user don't submit package which download url
belongs to MELPA. Maybe emit warning while executing `guix import elpa -a
melpa XXX`, or writing this rule to manual.

[1]: the stable archive is "MELPA stable", I use term "MELPA" to refer
to the unstable one.

-- 
Retrieve my PGP public key: https://meta.sr.ht/~citreu.pgp

Zihao

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

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

* bug#44863: Warning about importing a MELPA package
  2020-11-25  9:57 bug#44863: Warning about importing a MELPA package Zhu Zihao
@ 2020-12-07  9:17 ` Carlo Zancanaro
  2020-12-07  9:54   ` Carlo Zancanaro
  0 siblings, 1 reply; 3+ messages in thread
From: Carlo Zancanaro @ 2020-12-07  9:17 UTC (permalink / raw)
  To: Zhu Zihao; +Cc: 44863

On Wed, Nov 25 2020, Zhu Zihao wrote:
> ... We may better warn user don't submit package which download 
> url belongs to MELPA. Maybe emit warning while executing `guix 
> import elpa -a melpa XXX`, or writing this rule to manual.

I submitted a patch a while ago to make the MELPA importer pull 
the latest git commit (at the time of import) and use that as the 
source. The importer has to do a bit more work (eg. appropriately 
setting #:files in the Guix package).

You can find my patch at https://issues.guix.gnu.org/issue/38769 
but unfortunately it no longer applies cleanly to master. I'll 
take a look at fixing it up.




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

* bug#44863: Warning about importing a MELPA package
  2020-12-07  9:17 ` Carlo Zancanaro
@ 2020-12-07  9:54   ` Carlo Zancanaro
  0 siblings, 0 replies; 3+ messages in thread
From: Carlo Zancanaro @ 2020-12-07  9:54 UTC (permalink / raw)
  Cc: Zhu Zihao, 44863

On Mon, Dec 07 2020, Carlo Zancanaro wrote:
> ... unfortunately it no longer applies cleanly to master.

Actually, I think I was wrong about this. I must have done 
something wrong when I tried to apply the patch before sending my 
email. When I tried again it worked, and it seems to be fetching 
the git references properly.




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

end of thread, other threads:[~2020-12-07  9:55 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-25  9:57 bug#44863: Warning about importing a MELPA package Zhu Zihao
2020-12-07  9:17 ` Carlo Zancanaro
2020-12-07  9:54   ` Carlo Zancanaro

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.