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

Unsorted output makes 'diff' dirty and difficult to read : Role Strategy Pluigin

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Minor Minor
    • role-strategy-plugin
    • None
    • Jenkins 1.410, role-strategy 1.1

      this plugin outputs data to config.xml without sorting elements, for example:

      • permissions/pemisson
      • assignedSIDs/sid

      Of course, this is no harm, but as a result,
      simple diff of config.xml (ex. jobConfigHistory plugin) becomes very dirty ...

          [JENKINS-10005] Unsorted output makes 'diff' dirty and difficult to read : Role Strategy Pluigin

          Kuh Kro created issue -

          Kuh Kro added a comment -

          I found this tiny problem when using role-strategy and jobConfigHistory plugin.
          When someone added 1 job, I checked system config.xml with jobConfigHistory's diff display.
          Of course, diff output in this case should be 1 line, but nearly 50 lines appeared

          However, this tiny problem may occur in other plugins and all kind of software code

          Kuh Kro added a comment - I found this tiny problem when using role-strategy and jobConfigHistory plugin. When someone added 1 job, I checked system config.xml with jobConfigHistory's diff display. Of course, diff output in this case should be 1 line, but nearly 50 lines appeared However, this tiny problem may occur in other plugins and all kind of software code

          Kuh Kro added a comment -

          BTW, 'pemisson' is your typo ?
          It was found by spell checker of Firefox !

          I know my coworker who had typoed his function name as 'sygnal' and since that time, this name had been used for 10 years and more ..

          Kuh Kro added a comment - BTW, 'pemisson' is your typo ? It was found by spell checker of Firefox ! I know my coworker who had typoed his function name as 'sygnal' and since that time, this name had been used for 10 years and more ..
          Romain Seguy made changes -
          Assignee Original: Romain Seguy [ rseguy ] New: Daniel Petisme [ danielpetisme ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 140193 ] New: JNJira + In-Review [ 175227 ]
          Conrad T. Pino made changes -
          Assignee Original: Daniel Petisme [ danielpetisme ]

          Conrad T. Pino added a comment - - edited

          Per oleg_nenashev in JENKINS-8075

          Conrad T. Pino added a comment - - edited Per oleg_nenashev  in  JENKINS-8075

            Unassigned Unassigned
            k96 Kuh Kro
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: