You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by deragon <gi...@git.apache.org> on 2016/10/31 16:58:55 UTC

[GitHub] kafka pull request #2078: kafka-run-class.sh can now handle paths containing...

GitHub user deragon opened a pull request:

    https://github.com/apache/kafka/pull/2078

    kafka-run-class.sh can now handle paths containing spaces used in ${CLASSPATH} and in ${JAVA}.

    kafka-run-class.sh can now handle paths containing spaces used in ${CLASSPATH} and in ${JAVA}.
    
    This was required on Mac OSX since many library paths have spaces within them.  This was resolved by surrounding ${CLASSPATH} and ${JAVA} with double quotes (").  The change is very minor.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/deragon/kafka trunk

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/2078.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2078
    
----
commit 7d1ad2f57a855126e7ce0e520a8bb327f030abe6
Author: deragon <32nx9812masakjds>
Date:   2016-06-09T15:04:55Z

    kafka-run-class.sh now runs under Cygwin; paths and classpath are setup properly.

commit 6f3e43cf694bb9b15154c338e0b3261d3f23f821
Author: deragon <32nx9812masakjds>
Date:   2016-06-13T15:40:56Z

    MINOR: kafka-run-class.sh - Now 'set -e' safe and replaced backticks ` with $() syntax.

commit bf494a79d06efeb0cbf54e1557abe28ce64e87a8
Author: deragon <32nx9812masakjds>
Date:   2016-06-20T19:07:21Z

    Merge branch 'trunk' of https://github.com/apache/kafka into trunk

commit eb5ec97e3b11051da4ab8933de3d00474c1ad835
Author: deragon <32nx9812masakjds>
Date:   2016-07-12T12:59:17Z

    Merge branch 'trunk' of https://github.com/apache/kafka into trunk

commit 4e38eb2f3dc8c6c3a2900d70e3adc48d6a23a63a
Author: Hans Deragon <ha...@pj.ca>
Date:   2016-10-31T16:52:31Z

    Merge branch 'trunk' of https://github.com/apache/kafka into trunk

commit 471cdee82cc5899d907490f352ee8715b3edb489
Author: Hans Deragon <ha...@pj.ca>
Date:   2016-10-31T16:53:39Z

    kafka-run-class.sh can now handle paths containing spaces used in ${CLASSPATH} and in ${JAVA}.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---