-
Notifications
You must be signed in to change notification settings - Fork 713
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
update easyconfig for Longshot 0.4.5 to use Cargo easyblock #17666
update easyconfig for Longshot 0.4.5 to use Cargo easyblock #17666
Conversation
@boegelbot please test @ generoso |
Test report by @Micket |
@Micket: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1500911989 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
Test report by @Micket |
Test report by @Micket |
@boegelbot please test @ generoso |
@Micket: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1542512130 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ jsc-zen2 |
@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1559771523 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
ok so, this is that march thing on jsc-zen2 again
just like with easybuilders/easybuild-easyblocks#2759 I'm not sure what i should do with that a dictionary here.. |
1 similar comment
ok so, this is that march thing on jsc-zen2 again
just like with easybuilders/easybuild-easyblocks#2759 I'm not sure what i should do with that a dictionary here.. |
@Micket Let's open an issue on that in the easyblocks repo, we may need to come up with a general approach there that works for OpenCV too... |
@boegelbot please test @ jsc-zen2 |
@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1559797539 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
i guess it's assigning a empty string. I can't tell what RUSTFLAS is assigned from that partial log I would have thought this could be enough to protect against None or empty strings? optarch = build_option('optarch')
if not optarch:
optarch = 'native'
env.setvar('RUSTFLAGS', '-C target-cpu=%s' % optarch |
@Micket |
Test report by @boegel |
yes but that's not what we have here? Not indicated by the error message either. Running a local
i have no issues. |
@boegelbot please test @ jsc-zen2 |
@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1560513283 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@Micket It works fine if I use |
Ah, here's the problem, the single quotes are escaped:
So the correct way is to use |
@boegelbot please test @ jsc-zen2 |
@boegel: Request for testing this PR well received on jsczen2l1.int.jsc-zen2.easybuild-test.cluster PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1560526731 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
@boegelbot please test @ generoso |
@boegel: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 1561321192 processed Message to humans: this is just bookkeeping information for me, |
Test report by @boegelbot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Going in, thanks @Micket! |
(created using
eb --new-pr
)Cargo
andCargoPythonPackage
easybuild-easyblocks#2902