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

Empty exported parameters break $jube_wp_envstr #3

Closed
schwitanski opened this issue Aug 23, 2023 · 1 comment
Closed

Empty exported parameters break $jube_wp_envstr #3

schwitanski opened this issue Aug 23, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@schwitanski
Copy link

Using export="true" with an empty parameter breaks $jube_wp_envstr. For example, having three exported parameters ENV1=foo, ENV2= (empty) and ENV3=bar leads to:

export ENV1="foo"
export ENV2="export ENV3=bar"

MWE, tested with 2.5.1:

<?xml version="1.0" encoding="UTF-8"?>
<jube>
  <benchmark name="mwe_export" outpath="bench_run">
    <parameterset name="test_pset">
      <parameter name="ENV1" export="true">foo</parameter>
      <parameter name="ENV2" export="true"></parameter>
      <parameter name="ENV3" export="true">bar</parameter>
    </parameterset>

    <step name="test">
      <use>test_pset</use>
      <do>echo '$jube_wp_envstr'</do>
    </step>    
  </benchmark>
</jube>

Inspect run.log or stdout of the step. When setting ENV2 to any non-empty value, it works again as expected.

thobreuer added a commit that referenced this issue Nov 16, 2023
Fixed export of an empty parameter (Issue #3 on GitHub)

See merge request cstao/jube!43
@thobreuer thobreuer added the bug Something isn't working label Nov 16, 2023
@thobreuer
Copy link
Collaborator

Thanks for reporting. Fixed in c05e510 as part of REL-2.6.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants