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

Validate#check should be case insensitive

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: p4-plugin
    • Labels:
    • Environment:
      p4-plugin 1.9.6
    • Similar Issues:

      Description

      Perforce is inconsistent with the casing in filespec status messages that the server returns.

      As an example, different teams that I work with have seen p4 merge commands return the following messages depending on the p4 server version and/or merge options provides.

      all revision(s) already integrated.
      All revision(s) already integrated.
      

      It would be simple to update the Validate#check method to lowercase both the filespec status messages and messages in the ignore list.

        Attachments

          Activity

          Show
          stuartrowe Stuart Rowe added a comment - https://github.com/jenkinsci/p4-plugin/pull/96
          Hide
          p4karl Karl Wirth added a comment -

          Hi Stuart Rowe, Thanks for highlighting this. The workaround makes sense so I've passed it to the developers.

          Show
          p4karl Karl Wirth added a comment - Hi Stuart Rowe , Thanks for highlighting this. The workaround makes sense so I've passed it to the developers.
          Hide
          p4paul Paul Allen added a comment -

          Ready for release.

          Show
          p4paul Paul Allen added a comment - Ready for release.
          Hide
          p4paul Paul Allen added a comment -

          Released in 1.10.0

          Show
          p4paul Paul Allen added a comment - Released in 1.10.0

            People

            Assignee:
            stuartrowe Stuart Rowe
            Reporter:
            stuartrowe Stuart Rowe
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: