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

Update snapshot view config spec even when using the 'Use update' option

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • clearcase-plugin
    • None
    • Platform: All, OS: All

    Description

      The current clearcase plugin does not update the config spec. of a snapshot view
      when the 'Use update' option is enabled. It would be nice to also be able to
      change the views config spec in this mode (e.g. by issuing a 'cleartool setcs
      <cs_file_name>').

      Attachments

        Activity

          redsolo redsolo added a comment -

          Fixed. Now it will do a catcs to get the current config spec for the view. If it
          is different from the one configured, the config spec is updated with setcs.

          redsolo redsolo added a comment - Fixed. Now it will do a catcs to get the current config spec for the view. If it is different from the one configured, the config spec is updated with setcs.
          esmalling Eric Smalling added a comment -

          Not sure if I should open a new issue of not, but looking in FishEye at what I
          believe are the code changes for this fix, you've added code to replace DOS
          linefeeds with Unix ones ("/r/n" -> "/n"). This is like something I had hacked
          into my local copy of the hudson-core jar in the FilePath.createTextTempFile method.

          The problem I was trying to solve was to remove the "/r"'s from the temporary
          configspec file as the Linux cleartool setcs command fails when they exist.

          Playing with your latest build (for testing issue 1053), I removed my custom
          code and the DOS linefeeds came right back.

          Please let me know if you want me to open a new issue of if this is actually a
          problem with the fix to this one. (and thanks for all the work you do on this
          plugin!)

          esmalling Eric Smalling added a comment - Not sure if I should open a new issue of not, but looking in FishEye at what I believe are the code changes for this fix, you've added code to replace DOS linefeeds with Unix ones ("/r/n" -> "/n"). This is like something I had hacked into my local copy of the hudson-core jar in the FilePath.createTextTempFile method. The problem I was trying to solve was to remove the "/r"'s from the temporary configspec file as the Linux cleartool setcs command fails when they exist. Playing with your latest build (for testing issue 1053), I removed my custom code and the DOS linefeeds came right back. Please let me know if you want me to open a new issue of if this is actually a problem with the fix to this one. (and thanks for all the work you do on this plugin!)
          redsolo redsolo added a comment -

          Please open a separate issue for the "\r\n" problems.

          redsolo redsolo added a comment - Please open a separate issue for the "\r\n" problems.
          abayer Andrew Bayer added a comment -

          Bulk-updating all bugs fixed through 0.8.1 to VERIFIED. Part of cleanup for
          0.8.2 release.

          abayer Andrew Bayer added a comment - Bulk-updating all bugs fixed through 0.8.1 to VERIFIED. Part of cleanup for 0.8.2 release.

          People

            redsolo redsolo
            ktetzlaff ktetzlaff
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: