Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nightly builds are not usable with Marlin #492

Closed
tmadlener opened this issue Jun 7, 2023 · 5 comments
Closed

Nightly builds are not usable with Marlin #492

tmadlener opened this issue Jun 7, 2023 · 5 comments

Comments

@tmadlener
Copy link
Contributor

The new setup for the install prefixes for the nightly builds puts equal signs (specifically =develop) into the paths. This breaks workflows that use Marlin using paths as input variables, the most important one being $lcgeo_DIR, e.g.

Marlin --constant.lcgeo_DIR=$lcgeo_DIR [...]

Marlin uses the = as delimiter between the argument key and its value for parsing and cannot deal with additional = signs in the value.

@andresailer
Copy link
Contributor

Does it work if you quote the arguments?

@tmadlener
Copy link
Contributor Author

No, unfortunately that doesn't fix things. Neither "$lcgeo_DIR" nor quoting the expanded variable changes things. The quickest and lowest effort workaround for the moment is probably to create a local symlink and then use that.

ln -s $lcgeo_DIR lcgeo_DIR
Marlin --constant.lcgeo_DIR=./lcgeo_DIR [...]

@Zehvogel
Copy link
Contributor

Zehvogel commented Jul 24, 2023

Do we still have no better solution for this?

maybe it should also go into #502 @jmcarcell?

@jmcarcell
Copy link
Member

jmcarcell commented Jul 25, 2023

No better solution right now, I've just added it to #502

@tmadlener
Copy link
Contributor Author

This has changed in upstream spack to use a _ instead of a =. Hence, everything works again in nightly builds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants