From: Maxime Devos <maximedevos@telenet.be>
To: "Cees de Groot" <cg@evrl.com>, "Ludovic Courtès" <ludo@gnu.org>
Cc: 54304@debbugs.gnu.org, Tobias Geerinckx-Rice <me@tobias.gr>
Subject: [bug#54304] [PATCH] Don't fix git executable location during Elixir build
Date: Sat, 12 Mar 2022 00:16:18 +0100 [thread overview]
Message-ID: <13042012c34522fc9ed26746d11cc88b22f1ec6a.camel@telenet.be> (raw)
In-Reply-To: <WWBeielX0K08g-EXLiUTGaS1WVaXB-OBPqDO0Tks2kbPhz8lOpEdCV9qX_WBMnIIskckbTG2RDvVmlycgX_fWKoD316Xg2QcgqRbjGuWEYU=@evrl.com>
[-- Attachment #1: Type: text/plain, Size: 2448 bytes --]
Cees de Groot schreef op vr 11-03-2022 om 22:36 [+0000]:
> Well, feel free to do the patching work.
I'm just here reviewing things, looking for possible better
alternatives, etc.
> Realize that you are going to do work for a) a very small group of
> developers (Elixir is a small language and Guix isn’t hardly a widely
> spread system to begin with), b) a subset of those that need git
> dependencies, and c) a subset of _those_ that need these dependencies
> on a project source tree that itself was not checked out with git
> (otherwise the git executable will already be in the environment and
> everything will work).
>
> Please let me know when you found that unicorn developer :)
a) You're an Elixir developer, using Guix, and doing packaging work for
Elixir in Guix. Also, how is the popularity of Guix relevant on
guix-patches@, and how is the popularity of Elixir relevant when
this whole patch is about improving the Elixir package?
b) does not seem relevant to me, see other e-mails (about removing a
feature not worth the decrease in closure size IMO, and about
non-git references)
c) It's a bit niche; it would be a very nice thing of Guix to support
that niche.
Also, being checked out with git does not imply that git is
in the environment, it's possible to first set up a git repository
and later enter a pure environment for development (guix shell --
pure elixir elixir-foo elixir-bar). I often do this (not with
elixir, but with some other packages).
TBC, what do you mean with ‘unicorn developer’ here?
a) someone implementing this
b) someone in the subset
If (a), I was kind of hinting that you seem to be in a perfect position
to do so (you're an Elixir developer (*) in Guix and you seem to have
some knowledge of the serialisation formats).
(*) I'm assuming "Elixir developer" = "a developer of the Elixir
implementation" here.
> If Guix ever gets elixir packages, that can be tackled.
>
> I’m bowing out of this discussion. If my patch gets accepted then all
> good, if not, I’ll move it to my private channel and keep being happy
> there.
Ok, but I'm a bit confused on why you don't seem interested in fixing
the reference issue and why you write ‘ever’ instead of ‘when’ because
you're both an Elixir and Guix developer. (You don't have to answer,
not answering is fine.)
Greetings,
Maxime.
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]
next prev parent reply other threads:[~2022-03-11 23:17 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-08 20:52 [bug#54304] [PATCH] Don't fix git executable location during Elixir build Cees de Groot
2022-03-08 22:16 ` Maxime Devos
2022-03-09 17:35 ` Cees de Groot
2022-03-09 17:59 ` Tobias Geerinckx-Rice via Guix-patches via
2022-03-09 21:01 ` Cees de Groot
2022-03-09 21:49 ` Tobias Geerinckx-Rice via Guix-patches via
2022-03-11 21:33 ` Cees de Groot
2022-03-11 21:51 ` Ludovic Courtès
2022-03-11 21:57 ` Maxime Devos
2022-03-11 22:01 ` Cees de Groot
2022-03-11 22:11 ` Maxime Devos
2022-03-11 22:36 ` Cees de Groot
2022-03-11 22:45 ` Maxime Devos
2022-03-11 22:55 ` Cees de Groot
2022-03-11 23:16 ` Maxime Devos [this message]
2022-03-11 23:23 ` Cees de Groot
2022-03-13 22:14 ` Ludovic Courtès
2022-03-13 22:16 ` Cees de Groot
2022-03-08 22:17 ` Maxime Devos
[not found] ` <OtG7oZgFz6vgVWUbmqcU-n34hv-sQCpJYpJKR7rdQvIDnB1sUh_1LoUl9egvv6vKWH_THdYl9gwPrzwzXx_FnxlPXJHaB3D70jY32T3oMuk=@evrl.com>
2022-03-09 7:31 ` Maxime Devos
2022-03-09 17:29 ` Cees de Groot
2022-03-08 22:18 ` Maxime Devos
2022-03-08 22:22 ` Maxime Devos
2022-03-09 17:31 ` Cees de Groot
2022-03-09 19:45 ` 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://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=13042012c34522fc9ed26746d11cc88b22f1ec6a.camel@telenet.be \
--to=maximedevos@telenet.be \
--cc=54304@debbugs.gnu.org \
--cc=cg@evrl.com \
--cc=ludo@gnu.org \
--cc=me@tobias.gr \
/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).