all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Simon Tournier <zimon.toutoune@gmail.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: Graham James Addis <graham@addis.org.uk>,
	Josselin Poiret <dev@jpoiret.xyz>,
	Christopher Baines <mail@cbaines.net>,
	64173@debbugs.gnu.org, Mathieu Othacehe <othacehe@gnu.org>,
	Graham James Addis <grahamjamesaddis@gmail.com>,
	Tobias Geerinckx-Rice <me@tobias.gr>,
	64171@debbugs.gnu.org, Ricardo Wurmus <rekado@elephly.net>
Subject: [bug#64173] [PATCH 0/1] guix: pack: add --entry-point-argument option
Date: Fri, 18 Aug 2023 16:15:08 +0200	[thread overview]
Message-ID: <CAJ3okZ1LbkaSydP_m=712D4LTLDwRUBbowp7tQ24LKjp6jWBkg@mail.gmail.com> (raw)
In-Reply-To: <87zg2oqwkv.fsf@gnu.org>

Hi,

On Fri, 18 Aug 2023 at 16:00, Ludovic Courtès <ludo@gnu.org> wrote:

> > Well, since the order of the various command-line arguments might
> > matter, and since a ’reverse’ is suggested, and since all the Guix
> > command-lines do not behave the same way – for instance “guix package”
> > processes command-line argument from right to left; see #43585 or #50473
> > [1,2]; anyway :-) – I would suggest to add a sentence in the
> > documentation (manual) that the command-line arguments are parsed from
> > left to right.
>
> No: the ‘reverse’ puts them back in the right order (because
> ‘args-fold*’ traverses the option list from left to right and thus
> conses the result in reverse order.)

Is "no" for not adding a sentence in the documentation? :-)

BTW, what means the "right order"? :-)
I point that "guix package" and "guix show" process the command-line
option in different order -- from right to left vs from left to right
respectively.  Thus, this "right order" you are assuming can be
confusing, IMHO.

Therefore, applying POLA, it appears to me worth to add under the
option description of this new "--entry-point-argument" in the manual:
"The @code{--entry-point-arguement} option is applied from left to
right."  Or something along this line.

My 2 cents.

Cheers,
simon




      reply	other threads:[~2023-08-18 14:16 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-19 15:37 [bug#64171] [PATCH 0/1] guix: pack: docker add docker-entry-point options Graham James Addis
2023-06-19 15:38 ` [bug#64173] [PATCH 1/1] " Graham James Addis
     [not found] ` <handler.64172.B.168718911211626.ack@debbugs.gnu.org>
2023-06-19 15:49   ` bug#64172: Acknowledgement ([PATCH 0/1] ] guix: pack: docker add docker-entry-point options) Graham Addis
2023-07-03  9:25 ` [bug#64171] [PATCH 0/1] guix: pack: docker add docker-entry-point options Ludovic Courtès
2023-07-03 12:44   ` Graham Addis
2023-07-03 16:40     ` Giovanni Biscuolo
2023-07-08 16:08       ` Graham Addis
2023-07-09  8:31         ` Josselin Poiret via Guix-patches via
2023-07-12  8:17 ` [bug#64171] [PATCH 0/1 v2] guix: pack: docker add --entry-point-argument options Graham James Addis
2023-08-17  9:42   ` [bug#64173] [PATCH 0/1] guix: pack: add --entry-point-argument option Ludovic Courtès
2023-08-17 11:48     ` Simon Tournier
2023-08-18 14:00       ` [bug#64171] " Ludovic Courtès
2023-08-18 14:15         ` Simon Tournier [this message]

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

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

  git send-email \
    --in-reply-to='CAJ3okZ1LbkaSydP_m=712D4LTLDwRUBbowp7tQ24LKjp6jWBkg@mail.gmail.com' \
    --to=zimon.toutoune@gmail.com \
    --cc=64171@debbugs.gnu.org \
    --cc=64173@debbugs.gnu.org \
    --cc=dev@jpoiret.xyz \
    --cc=graham@addis.org.uk \
    --cc=grahamjamesaddis@gmail.com \
    --cc=ludo@gnu.org \
    --cc=mail@cbaines.net \
    --cc=me@tobias.gr \
    --cc=othacehe@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 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.