unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
From: Maxime Devos <maximedevos@telenet.be>
To: "Aleix Conchillo Flaqué" <aconchillo@gmail.com>
Cc: Guix-devel <guix-devel@gnu.org>, guile-devel <guile-devel@gnu.org>
Subject: Re: (ice-9 base64)?
Date: Thu, 18 Aug 2022 09:56:15 +0200	[thread overview]
Message-ID: <2dea77ed-4f1b-b799-62d7-149072fd2103@telenet.be> (raw)
In-Reply-To: <CA+XASoXPLroB1LLb6yi4w0nAd20xHGC0HBEJtjUYzuFP7XCAWw@mail.gmail.com>


[-- Attachment #1.1.1.1: Type: text/plain, Size: 2741 bytes --]

> Then, if I understood correctly, IMO I would say Guile should not 
> really care about Guix's bundling/unbundling. That is, adding (ice-9 
> base64) (or however we want to call it... maybe (encoding base64) 
> following Golang and Guile's (web ....) module) should be totally 
> independent of Guix. So, if we add (ice-9 base64) to Guile then Guix 
> should figure out what to do with it, but it's Guix's concern not Guile's.

It's not some Guix-specific quirk. It's the same for at least Debian. It 
benefits not only Guix itself but all users of the software:

> [...] allows [...] to make transverse changes
> such as applying security updates for a given software package in a
> single place and have them affect the whole system—something that
> bundled copies prevent.
... that was written with Guix in mind, but it applies to every 
distribution and everyone.

Besides, your goal appears to be to unbundle the base64 into a single 
location (as a module of Guile), if we do that I think we should go all 
the way -- just adding it to Guile increases bundling instead of 
decreasing bundling, only if the various upstreams are modified to 
unbundle and use the new location then the unbundling is completed.

On 18-08-2022 02:09, Aleix Conchillo Flaqué wrote:
> About Guix's unbundling (maybe that's something that should go on 
> Guix's mailing list),
I don't see why, there's nothing to write about except "oops some 
packages are bundling base64, let's unbundle those", and for unbundling 
those, it seems more practical to write about that here on guile-devel. 
Also I noticed I sent some messages to guix-devel instead of 
guile-devel, correcting now.
> I don't think currently there's any unbundling for base64 modules or 
> at least not in a package I maintain guile-jwt (guile-jwt bundles 
> base64). And probably there's no unbundling because there's no 
> canonical implementation? Even if there was a canonical 
> implementation, how would that look like in Guix's guile-jwt package? 
> What would the snippet actually do?
Currently, it's not done yet, presumably for that reason and maybe also 
due to nobody having noticed it yet?

How it would look like, for upstreams that refuse to unbundle or are 
unresponsive:

#~(begin
         (delete-file "local/copy/of/base64.scm")
         [also remove it from the Makefile.am]
         (substitute* (find-files "." "\\.scm$")
           (("(\\local base64 module\\)") "(gcrypt base64)")))

For responsive upstreams that do not mind these kind of improvements, 
there is a preference for submitting a patch upstream -- that way, 
everyone benefits, not only Guix.

Greetings,
Maxime.


[-- Attachment #1.1.1.2: Type: text/html, Size: 4009 bytes --]

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 929 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 236 bytes --]

  parent reply	other threads:[~2022-08-18  7:56 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-16 16:10 (ice-9 base64)? Aleix Conchillo Flaqué
2022-08-16 16:16 ` [EXT] " Thompson, David
2022-08-16 16:58 ` Maxime Devos
2022-08-16 17:21   ` Aleix Conchillo Flaqué
     [not found]     ` <df5ce89f-a7a2-7337-8dc7-67372cb1c48c@telenet.be>
     [not found]       ` <CA+XASoWyG7xDNcRyX46RERoORB=a=hckh5uU3MQvJ-z8ou-9Fg@mail.gmail.com>
     [not found]         ` <5331d2f3-13a5-e40c-f3bb-398438a0b103@telenet.be>
     [not found]           ` <CA+XASoXPLroB1LLb6yi4w0nAd20xHGC0HBEJtjUYzuFP7XCAWw@mail.gmail.com>
2022-08-18  7:56             ` Maxime Devos [this message]
2022-08-19  0:20               ` Aleix Conchillo Flaqué
2022-08-19 10:06                 ` Maxime Devos

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/guile/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2dea77ed-4f1b-b799-62d7-149072fd2103@telenet.be \
    --to=maximedevos@telenet.be \
    --cc=aconchillo@gmail.com \
    --cc=guile-devel@gnu.org \
    --cc=guix-devel@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).