optimism icon indicating copy to clipboard operation
optimism copied to clipboard

Makefile: Fix executable command detection

Open karlb opened this issue 1 year ago • 3 comments

$(command -v geth) is valid shell syntax, but it gets processed by make first, so that this never gets correctly executed by the shell. As a result, make install-geth and make install-eth2-testnet-genesis are always executed and not, as intended, just when the commands are not present.

The fix is easy: escape the dollar by duplicating it, so that make won't do anything and the correct command reaches the shell.

karlb avatar Sep 24 '24 12:09 karlb

The CI jobs seem stuck. Should I do a new push to trigger them again or is there a better way to deal with it?

karlb avatar Oct 03 '24 09:10 karlb

The CI jobs seem stuck. Should I do a new push to trigger them again or is there a better way to deal with it?

yeah. try rebasing on develop.

Inphi avatar Oct 03 '24 15:10 Inphi

After rebasing on develop, I get a reproducible semgrep fail in CI:

Command failed with output:
fatal: Not a valid object name develop

Failed to run 'git cat-file -e develop'. Possible reasons: [...]

karlb avatar Oct 04 '24 08:10 karlb

/ci authorize c11e62c7756923096da5e7c2d6b275e93cdecac3

smartcontracts avatar Oct 26 '24 05:10 smartcontracts

Can you try rebasing this one more time?

smartcontracts avatar Oct 26 '24 05:10 smartcontracts

Can you try rebasing this one more time?

Done, but CI still fails with We cannot run this job using the selected resource class. Please check your configuration and try again..

karlb avatar Oct 26 '24 08:10 karlb

/ci authorize dd28c7e114376ca295e9a3b6cecf42c6e8cdb52e

smartcontracts avatar Oct 26 '24 08:10 smartcontracts

Ok yeah no idea what's up here but working on figuring it out

smartcontracts avatar Oct 26 '24 10:10 smartcontracts

/ci authorize dd28c7e114376ca295e9a3b6cecf42c6e8cdb52e

mslipper avatar Oct 27 '24 00:10 mslipper

Seems to be working now. This may have just needed authorization, though it's odd that the error message changed.

mslipper avatar Oct 27 '24 00:10 mslipper