Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-41974

variable replacement does not work as expected with some variable names

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      For example, if the build contains variables XXXX=a and XXXX_YYYY=b, XXXX_YYYY might not get replaced correctly if XXXX is parsed first:

      "$XXXX $XXXX_YYYY" will result in "a a_YYYY"

        Attachments

          Activity

          Show
          mhuin Matthieu Huin added a comment - Proposed fix in PR https://github.com/jenkinsci/mqtt-notification-plugin/pull/4
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: mhuin
          Path:
          src/main/java/jenkins/plugins/mqttnotification/MqttNotifier.java
          src/main/resources/jenkins/plugins/mqttnotification/MqttNotifier/help-message.html
          http://jenkins-ci.org/commit/mqtt-notification-plugin/1450c987397b1b65e7de245dff6148a58803f1e0
          Log:
          JENKINS-41974 Allow defining variables between brackets to avoid ambiguity

          This change maintains also the old way of defining variables for compatibility.

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: mhuin Path: src/main/java/jenkins/plugins/mqttnotification/MqttNotifier.java src/main/resources/jenkins/plugins/mqttnotification/MqttNotifier/help-message.html http://jenkins-ci.org/commit/mqtt-notification-plugin/1450c987397b1b65e7de245dff6148a58803f1e0 Log: JENKINS-41974 Allow defining variables between brackets to avoid ambiguity This change maintains also the old way of defining variables for compatibility.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Gareth Western
          Path:
          pom.xml
          src/main/java/jenkins/plugins/mqttnotification/MqttNotifier.java
          src/main/resources/jenkins/plugins/mqttnotification/MqttNotifier/help-message.html
          http://jenkins-ci.org/commit/mqtt-notification-plugin/def8f4c5cab5b757c631c94f59debe1fd47393e0
          Log:
          Merge pull request #4 from mhuin/JENKINS_41974

          JENKINS-41974 Allow defining variables between brackets to avoid am…

          Compare: https://github.com/jenkinsci/mqtt-notification-plugin/compare/eb2d3cf618d7...def8f4c5cab5

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gareth Western Path: pom.xml src/main/java/jenkins/plugins/mqttnotification/MqttNotifier.java src/main/resources/jenkins/plugins/mqttnotification/MqttNotifier/help-message.html http://jenkins-ci.org/commit/mqtt-notification-plugin/def8f4c5cab5b757c631c94f59debe1fd47393e0 Log: Merge pull request #4 from mhuin/JENKINS_41974 JENKINS-41974 Allow defining variables between brackets to avoid am… Compare: https://github.com/jenkinsci/mqtt-notification-plugin/compare/eb2d3cf618d7...def8f4c5cab5
          Hide
          gareth_western Gareth Western added a comment -

          (Hopefully) fixed as of 1.5

          Show
          gareth_western Gareth Western added a comment - (Hopefully) fixed as of 1.5

            People

            • Assignee:
              gareth_western Gareth Western
              Reporter:
              mhuin Matthieu Huin
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: