On Mon, 19 Nov 2018, Olly Betts wrote:

On Mon, Nov 19, 2018 at 08:32:28AM +0000, Simon McVittie wrote:
Recent tests on tests.reproducible-builds.org use unmerged /usr for the
first build and merged /usr for the second, as a way to detect some
issues in this class.

Thanks for the report.

$EGREP is used by the script, and changing it to point to a path that
doesn't exists leads to wx-config failing with an error such as:

         Warning: No config found to match: /usr/bin/wx-config --cppflags
                  in /usr/lib/x86_64-linux-gnu/wx/config

I'd actually spotted this (also via reproducible-builds) yesterday
but only after I made the recent upload.  Unless any of the current
buildds has a merged /usr I suggest we let that upload migrate to
testing and then fix this.

This bug can probably be fixed by passing EGREP="/bin/grep -E" as an
additional command-line argument when invoking configure.

That was my thought too.

It would be nicer to address upstream, but I don't really see a way to
do that cleanly short of rewriting it to just use shell built-ins, which
seems like a lot of work as it's a hairy beast.

Or we could just patch wx-config to set EGREP to "grep -E" (without the path)?

https://salsa.debian.org/freewx-team/wx/blob/wx3.0-debian/wx-config.in#L92

Scott

Reply via email to