unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Ricardo Wurmus <rekado@elephly.net>
Cc: 33111@debbugs.gnu.org
Subject: [bug#33111] [PATCH 0/3] Have the binary tarball populate ~root/.config/guix/current
Date: Fri, 26 Oct 2018 11:57:02 +0200	[thread overview]
Message-ID: <871s8dni8h.fsf@gnu.org> (raw)
In-Reply-To: <87a7n15l1g.fsf@elephly.net> (Ricardo Wurmus's message of "Fri, 26 Oct 2018 07:33:47 +0200")

Hello!

Ricardo Wurmus <rekado@elephly.net> skribis:

>>> These patches address this by having the binary tarball populate
>>> ~root/.config/guix/current like ‘guix pull’ does.
>>>
>>> There’s one downside though: with the last patch, the ‘glibc-utf8-locales’
>>> is no longer included because ~root/.config/guix/current would be the
>>> wrong place for it.  Consequently, users have to explicitly install it
>>> in ~root/.guix-profile and set GUIX_LOCPATH accordingly.
>>
>> Any comments?  Ricardo?
>
> Thank you for fixing this very confusing situation!  It is very good to
> start out with a

With a what?  :-)

> I’m not sure I understand why ~root/.config/guix/current would be the
> wrong place for the locales package.  It is true that this directory is
> for Guix only, but users don’t need to know about the locales package.
>
> Is it a problem to install the locales package alongside Guix in the
> “guix pull” profile, or is it just inelegant?

It’s inelegant and just a partial solution because ‘guix pull’ won’t
upgrade it.  It would also show up in ‘guix pull -l’, which isn’t great.

> I think it would be unfortunate if older versions of Guix would stop
> working or report warnings when they are used in combination with a
> separately managed profile containing the locales.  The locales need to
> match the glibc version that the program is linked with, so I would
> prefer if we could keep Guix and the locales together.
>
> You know that I find the separation of glibc-locales to be an
> unfortunate tradeoff, which makes using Guix on foreign distros a little
> less convenient.  While I think that this patch set is a definite
> improvement over the current situation, I would be sad to see the
> locales separated and become a source of frustration or confusion.
>
> Is there something we can do about this?  Would it be acceptable to add
> glibc-locales as an explicit input to the result of “guix pull”?  I
> understand that we don’t usually add glibc-*locales as an input, but I
> think in the special case of “guix pull” it could be a reasonable
> compromise / an acceptable exception.  What do you think?

I share your concern but I can’t think of a good solution.

Since ‘glibc-locales’ is big (520M on disk!) and more than what people
need, we certainly don’t want to pull it.  So we’d be pulling
‘glibc-utf8-locales’ as we currently do, knowing that it’s an arbitrary
choice of locales that favors Western countries.

Let’s say we arrange so ‘guix’ is wrapped such that GUIX_LOCPATH points
to a bundled ‘glibc-utf8-locales’.  That solves the problem for the
‘guix’ command, but it doesn’t solve it for other packages installed
with ‘guix’: users would still need to install ‘glibc-utf8-locales’.

A “proper fix” might be to add ‘glibc-utf8-locales’ to
~root/.guix-profile in the binary tarball.  However, as I wrote, it
would require us to improve ‘guix pack’ so it can populate several
profiles, which is not trivial and probably not very useful in other
situations.

On the bright side, ‘guix’ gives users the command to copy/paste to
install locales (commit 26db747a863b08ebcfd630cce635be86c23d829d), so
it’s not that bad.  :-)

Also, we could have the install script run ‘guix package -i
glibc-utf8-locales’ automatically.

Thoughts?

Thanks for your feedback!

Ludo’.

  reply	other threads:[~2018-10-26  9:58 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-10-21 20:45 [bug#33111] [PATCH 0/3] Have the binary tarball populate ~root/.config/guix/current Ludovic Courtès
2018-10-21 20:49 ` [bug#33111] [PATCH 1/3] install: Parameterize the profile name for 'populate-single-profile-directory' Ludovic Courtès
2018-10-21 20:49   ` [bug#33111] [PATCH 2/3] pack: Add '--profile-name' Ludovic Courtès
2018-10-21 20:49   ` [bug#33111] [PATCH 3/3] build: Binary tarball now populates the "current-guix" profile Ludovic Courtès
2018-10-25 20:24 ` [bug#33111] [PATCH 0/3] Have the binary tarball populate ~root/.config/guix/current Ludovic Courtès
2018-10-26  5:33   ` Ricardo Wurmus
2018-10-26  9:57     ` Ludovic Courtès [this message]
2018-11-16 21:59       ` Ludovic Courtès
2018-11-23 14:43         ` bug#33111: " Ludovic Courtès

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://guix.gnu.org/

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

  git send-email \
    --in-reply-to=871s8dni8h.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=33111@debbugs.gnu.org \
    --cc=rekado@elephly.net \
    /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.
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).