Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-34165

Fix syntax error in lsstsw bin/envconfig

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: lsstsw
    • Labels:
      None
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      There's an extra < character in line 102, a branch of the code that is not normally executed.

        Attachments

          Activity

          Hide
          ktl Kian-Tat Lim added a comment -

          Actually, it turns out that this is not an extra character due to bash process substitution. The logic is still more complex than it needs to be.

          Show
          ktl Kian-Tat Lim added a comment - Actually, it turns out that this is not an extra character due to bash process substitution. The logic is still more complex than it needs to be.
          Hide
          ktl Kian-Tat Lim added a comment -

          A hopefully quick +6 -11. This was triggered by a parse problem likely due to an incorrect shell, but I think it's a valuable simplification anyway.

          Show
          ktl Kian-Tat Lim added a comment - A hopefully quick +6 -11. This was triggered by a parse problem likely due to an incorrect shell, but I think it's a valuable simplification anyway.
          Hide
          erykoff Eli Rykoff added a comment -

          Looks fine, if it works.

          Show
          erykoff Eli Rykoff added a comment - Looks fine, if it works.

            People

            Assignee:
            ktl Kian-Tat Lim
            Reporter:
            ktl Kian-Tat Lim
            Reviewers:
            Eli Rykoff
            Watchers:
            Eli Rykoff, Kian-Tat Lim
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.