• Icon: Patch Patch
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • None
    • Platform: All, OS: All

      Some of our projects check out dozens of top-level CVS modules. So our console
      output often has such meaningful entries as

      $ cvs -q -z9 update -PdC
      $ cvs -q -z9 update -PdC
      P build.xml
      $ cvs -q -z9 update -PdC
      $ cvs -q -z9 update -PdC
      $ cvs -q -z9 update -PdC
      $ cvs -q -z9 update -PdC
      $ cvs -q -z9 update -PdC
      P build.xml
      $ cvs -q -z9 update -PdC

      Hmm, what was it updating now?

      It would be preferable to run CVS update just once, at the top level, which
      would probably be faster and also print more informative paths. But in any case
      it would also be helpful to see what dir a command is being run in. Attaching
      (as yet untested) patch which should give that information succinctly and be
      useful in our case at least.

          [JENKINS-65] Print cwd when running CVS commands

          Jesse Glick added a comment -

          Created an attachment (id=8)
          Suggested patch

          Jesse Glick added a comment - Created an attachment (id=8) Suggested patch

          Thank you for the patch. I particularly liked that you are only printing the
          last portion of the path, which keeps the screen tidy.

          Patch applied.

          Kohsuke Kawaguchi added a comment - Thank you for the patch. I particularly liked that you are only printing the last portion of the path, which keeps the screen tidy. Patch applied.

          Jesse Glick added a comment -

          Oops, this will teach me to submit untested patches...

          Jesse Glick added a comment - Oops, this will teach me to submit untested patches...

          Jesse Glick added a comment -

          Created an attachment (id=10)
          Patch correction

          Jesse Glick added a comment - Created an attachment (id=10) Patch correction

          And that should teach me about applying an untested patch he he

          I applied your new patch.

          Kohsuke Kawaguchi added a comment - And that should teach me about applying an untested patch he he I applied your new patch.

            Unassigned Unassigned
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: