Re: 01/01: scripts: refresh: Allow searching recursively.

[Ludovic Courtès]

Efraim Flashner <address@hidden> skribis:

> On Tue, Dec 25, 2018 at 04:05:11PM +0100, Ludovic Courtès wrote:
>> Hello!
>> 
>> address@hidden skribis:
>> 
>> > efraim pushed a commit to branch master
>> > in repository guix.
>> >
>> > commit c39491829a0c1d870f8133b8f7a699152fc71503
>> > Author: Efraim Flashner <address@hidden>
>> > Date:   Wed Dec 19 22:08:18 2018 +0200
>> >
>> >     scripts: refresh: Allow searching recursively.
>> >     
>> >     * guix/scripts/refresh.scm (refresh-recursive, list-transitive): New
>> >     procedures.
>> >     (show-help): Document it.
>> >     (guix-refresh): Add flags and checks for new options.
>> >     * doc/guix.texi (Invoking guix refresh): Document new options.
>> 
>> Nice work!
>> 
>> > address@hidden --list-transitive
>> > +List all the packages which one or more packages depend upon.
>> 
>> What about ‘--list-dependencies’, for clarity and symmetry with
>> ‘--list-dependents’?
>> 
>
> It was originally '--list-recursive' but Bavier suggested
> '--list-transitive' so it would match 'guix build --sources=transitive'
>
> If it's too close to '--list-dependents' then I'm worried people could
> look at it and ask 'packages FOO is dependent on or packages that are
> dependent on FOO?'

Good point.  I do feel like ‘--list-dependencies’ is clearer though.
With ‘list-transitive’, since “transitive” is an adjective, I first
wondered: transitive what?

What do people think?

Thanks,
Ludo’.