The wrapper is not needed because the runpath is already set correctly,
and LD_LIBRARY_PATH was breaking child processes linked against
different libc versions.
Signed-off-by: Anders Kaseorg <andersk@mit.edu>
`seafile-client` does not build successfully because of a hash mismatch
in `seafile-shared`. The old hash appears to point to commit
`b7f2be78c1d979ff1242c3b65b41228e31427843` rather than
`ea8f5e2b45612d77fee9934f942d1d7d55560dad`, which is the commit that the
`v6.2.10` tag currently references.
Since the 0.21 upgrade, the host `$PATH` is not forwarded anymore by
default to the sandboxes in charge to realize Bazel actions. This
default change broke the `py_binary` rule among other things.
Every python binary is wrapped in a stub in charge to setup the
execution environment. Currently, this stub's shebang points to a
`/usr/bin/env python` which cannot be resolved with the current
`$PATH`.
This results in breaking any build pipeline requiring the use of
python at some point. On top of the incorrect shebang, the stub
template is unable to find the actual python binary using
`SearchPath`.
This PR fixes those two things by re-writing the stub template shebang
to the actual python binary and by substituting the faulty default
python binary lookup to the right one.
* openblas: simplify a bit, fix doCheck so tests are enabled non-cross.
* doCheck should be 'true' in (at least) the non-cross case,
this looks like an inverted check that's largely benign
* doCheck will be set to 'false' in the cross case anyway,
makeDerivation does this IIRC
* targetPrefix can be used without checking, probably by design
Derivation hash does change but no "real" functionality change intended.
* openblas: nix types for config attrs (hash-preserving)
* openblas: more nix-ification, merge in cross attrs, prefer to always set
(but set appropriately for cross and non-cross cases both)
* I'm not sure what NO_BINARY_MODE does,
this change now sets explicitly false in the non-cross scenario
(previously unset unless cross).
* Drop musl NO_AFFINITY case, will be removed in upgrade shortly
* openblas: 0.3.4 -> 0.3.5
Poppler was upgraded in [0] and therefore the build broke, as
poppler/goo/gtypes.h was moved into poppler/goo/gfile.h [1].
The patch is intended to be broght upstream and then can be reverted.
[0] 7757e43fcb
[1] ef3ef702bc
Anders Kaseorg
R. RyanTM
Jörg Thalheim
Elis Hirwing
Cray Elliott
Orivej Desh
Tim Steinbach
Robert Schütz
Robert Schütz
Jon Banafato
Vladyslav M
/u/leo60228
Matthew Bauer
Pierre-Etienne Meunier
Will Dietz
Michael Weiss
gnidorah
WilliButz
Francesco Gazzetta
Will Dietz
Félix Baylac-Jacqué
Patrick Mahoney
Yorick van Pelt
Jörg Thalheim
Frederik Rietdijk
Tor Hedin Brønner
Robert Hensing
Justin Humm
