[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#62264] [PATCH] Add 'guix locate' command
From: |
pelzflorian (Florian Pelz) |
Subject: |
[bug#62264] [PATCH] Add 'guix locate' command |
Date: |
Sat, 17 Jun 2023 17:56:47 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Greetings Ludo,
Ludovic Courtès <ludo@gnu.org> writes:
>> After deleting ~/.cache/guix/locate and cd’ing out of my home directory,
>> then inside “guix shell -CW coreutils”, “guix locate ls” does not find
>> ls.
>
> Works for me:
>
> $ guix shell -CW -D guix
> […]
> Note that ‘-W’ has the effect of sharing ~/.cache with the container.
> That means that the database is already there.
Ah of course that is why it works for you. I had deleted the locate
directory of the cache, so after `guix shell -CW -D guix` the `guix
locate ls` did not work, and did not work either once I had left the
container, because an empty database had been created in the cache from
within the container. Without -CW it works fine. It is a minor bug.
>> When I use “guix locate icecat”, it legitimately also locates a file
>> lib/icecat/icecat in addition to the desired bin/icecat. I try to
>> filter by “guix locate bin/icecat” or “guix locate -g bin/icecat”, but
>> it seems locate does not support file names with slashes.
> Right: ‘guix locate’ only checks the “basename”; it does not let you
> search on the absolute file name. We could add an option to do that
> later (say ‘-f’) but I thought it’s less frequently useful.
I’m nitpicking, but if relative file names are not meant to be
supported, could you change the doc/guix.texi at
+@example
+guix locate [@var{options}@dots{}] @var{file}@dots{}
+@end example
+
+@noindent
+... where @var{file} is the name of a file to search for.
to reflect that FILE must be the basename? Because in other parts of
the doc, the term path is avoided and the term filename is used. Now a
filename is only a basename all of a sudden.
Regards,
Florian
- [bug#62264] [PATCH v2 2/3] Add 'guix locate'., (continued)
- [bug#62264] [PATCH v2 2/3] Add 'guix locate'., Ludovic Courtès, 2023/06/07
- [bug#62264] [PATCH v2 2/3] Add 'guix locate'., Jelle Licht, 2023/06/08
- [bug#62264] [PATCH v2 2/3] Add 'guix locate'., pelzflorian (Florian Pelz), 2023/06/08
- [bug#62264] [PATCH v3 1/3] store: Tolerate non-existent GC root directories., Ludovic Courtès, 2023/06/16
- [bug#62264] [PATCH v3 2/3] Add 'guix locate'., Ludovic Courtès, 2023/06/16
- [bug#62264] [PATCH] Add 'guix locate' command,
pelzflorian (Florian Pelz) <=
- [bug#62264] [PATCH] Add 'guix locate' command, Ludovic Courtès, 2023/06/18