ZooKeeper
  1. ZooKeeper
  2. ZOOKEEPER-606

bin scripts don't work in cygwin (spaces in paths)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.2.1
    • Fix Version/s: 3.3.0
    • Component/s: scripts
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Release Note:
      Committed revision 891944.

      Description

      the scripts in bin fail under cygwin due to spaces not handled properly

      1. ZOOKEEPER-606.patch
        6 kB
        Patrick Hunt
      2. ZOOKEEPER-606.patch
        6 kB
        Patrick Hunt

        Issue Links

          Activity

          Patrick Hunt created issue -
          Hide
          Patrick Hunt added a comment -

          this patch:

          1) fixes the spaces
          2) fixes the need for wndows java to have ; in classpath
          3) fixes kill command on cygwin

          Show
          Patrick Hunt added a comment - this patch: 1) fixes the spaces 2) fixes the need for wndows java to have ; in classpath 3) fixes kill command on cygwin
          Patrick Hunt made changes -
          Field Original Value New Value
          Attachment ZOOKEEPER-606.patch [ 12426610 ]
          Hide
          Patrick Hunt added a comment -

          Seems to work on unix as well (ubuntu), submitting.

          Show
          Patrick Hunt added a comment - Seems to work on unix as well (ubuntu), submitting.
          Patrick Hunt made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Assignee Patrick Hunt [ phunt ]
          Patrick Hunt made changes -
          Status Patch Available [ 10002 ] Open [ 1 ]
          Hide
          Patrick Hunt added a comment -

          same as before:
          1) fixed running from release build
          2) uncommented-out the rm of the pid (had removed for testing, forgot to add back)

          Show
          Patrick Hunt added a comment - same as before: 1) fixed running from release build 2) uncommented-out the rm of the pid (had removed for testing, forgot to add back)
          Patrick Hunt made changes -
          Attachment ZOOKEEPER-606.patch [ 12426639 ]
          Patrick Hunt made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Patrick Hunt made changes -
          Link This issue relates to ZOOKEEPER-426 [ ZOOKEEPER-426 ]
          Hide
          Patrick Hunt added a comment -

          note to reviewer(s): be sure to verify these on cygwin/unix & macosx

          Show
          Patrick Hunt added a comment - note to reviewer(s): be sure to verify these on cygwin/unix & macosx
          Hide
          Yonik Seeley added a comment -

          Verified (and using) on Win7+cygwin1.7
          Thanks!

          Show
          Yonik Seeley added a comment - Verified (and using) on Win7+cygwin1.7 Thanks!
          Hide
          Benjamin Reed added a comment -

          +1

          Show
          Benjamin Reed added a comment - +1
          Benjamin Reed made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags [Reviewed]
          Release Note Committed revision 891944.
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in ZooKeeper-trunk #634 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/634/)
          . bin scripts don't work with cygwin (phunt via breed)

          Show
          Hudson added a comment - Integrated in ZooKeeper-trunk #634 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/634/ ) . bin scripts don't work with cygwin (phunt via breed)
          Patrick Hunt made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Patrick Hunt
              Reporter:
              Patrick Hunt
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development