Re: tests: dis/allow '.' in PATH?

2021-11-23 Thread Kamil Dudka
On Tuesday, November 23, 2021 11:19:22 PM CET Bernhard Voelker wrote: > GNU findutils got a bug report for a failing test in the testsuite [1]. > It turned out that the calling environment had the current directory '.' > in PATH. This triggered a warning in `find -execdir ...` and therefore > made

Re: tests: dis/allow '.' in PATH?

2021-11-23 Thread Paul Eggert
On 11/23/21 14:19, Bernhard Voelker wrote: Of course, the findutils tests could (and probably will) remove '.' from PATH, but I wonder if this should or should not be done for other packages via a change in 'tests/init.sh' as well? Sounds good to me. It might also help to sanitize other sensi

tests: dis/allow '.' in PATH?

2021-11-23 Thread Bernhard Voelker
GNU findutils got a bug report for a failing test in the testsuite [1]. It turned out that the calling environment had the current directory '.' in PATH. This triggered a warning in `find -execdir ...` and therefore made some tests fail. [1] https://lists.gnu.org/r/bug-findutils/2021-11/msg4.

Prefer AM_GNU_GETTEXT_REQUIRE_VERSION?

2021-11-23 Thread Simon Josefsson via Gnulib discussion list
Hi. This may mostly be for Bruno, but I believe it is more relevent to gnulib than gettext, even though it is gettext-related, and maybe others on this list can provide feedback too. I got a bug report that suggested using AM_GNU_GETTEXT_REQUIRE_VERSION: https://bugs.debian.org/cgi-bin/bugreport

Re: [PATCH 01/11] Fix base64 module to work with grub codebase

2021-11-23 Thread Daniel Kiper
CC-ing Vladimir... On Tue, Nov 09, 2021 at 02:34:25PM -0500, Robbie Harwood wrote: > Paul Eggert writes: > > > On 10/28/21 12:32, Robbie Harwood wrote: > > > >> I don't know why Patrick chose to > >> not use that instead, but a local test seems to work. > > > > Is grub2 intended to be portable to