mbox

[bug#38678,v2,0/4] Add '--load-path' to subcommands

Message ID CAJ3okZ0fnvSK_QLw8VqoPxgLH3zwai9mhP3F=a5SuKRGDiheaA@mail.gmail.com
Headers show

Message

Simon Tournier Jan. 8, 2020, 6:53 p.m. UTC 
Dear,

Please find the amended patches about 'graph' and 'size'.

On Thu, 19 Dec 2019 at 22:10, zimoun <zimon.toutoune@gmail.com> wrote:

>  - guix refresh: -L already means --list-updaters
>    Only --load-path? Nothing?

The short option '-L' is dropped of the %standard-build-options.

Note that if I read correctly, nothing of the sucommand 'resfresh' is tested.

>  - guix edit: I am not convinced it is useful because if you apply -L
>    path/to/your/modules then it is easy to find the correct file to open.
>    What do you think?

Even it is easy to find the correct file to open, it adds consistency
with the other subcommands and it can help when using local channels.

Nothing about the subcommand 'edit' is tested neither.



All the best,
simon

Comments

Simon Tournier Jan. 8, 2020, 7:12 p.m. UTC | #1 
On Wed, 8 Jan 2020 at 19:53, zimoun <zimon.toutoune@gmail.com> wrote:

> The short option '-L' is dropped of the %standard-build-options.

My bad!
The patch v2 keeps only "load-path" when only #\L must be dropped out.
It is more about style than concrete issue.


All the best,
simon