Hi Jan, thanks for the Patch. Actually at OPS4J we don't work with Patches or Pull Requests usually. Instead, everyone who steps up contributing something gets commit grants so he/she can drive the contribution without any barrier. Read [1] for more.
I just added you to the Dev Team on OPS4Js GitHub account so you can push there directly if you think the patch is appropriate and "mature enough". So this means you can still fork and work off the lines but can push it to the main repo by yourself without any delay. Hope that works for you, let us know if you need anything. Toni [1] http://team.ops4j.org/wiki/display/ops4j/Principles+of+Open+Participation+Software On Thu, Sep 22, 2011 at 8:54 AM, Jan Thomä < reply+i-1707870-4215e1bf4787b2d13b40562d9d2ca2f1dedb0...@reply.github.com>wrote: > Hi Toni, > > this is a fix for PAXRUNNER-406, which I reported this morning. > > Kind regards, > Jan > > You can merge this Pull Request by running: > > git pull https://github.com/derkork/org.ops4j.pax.runner master > > Or you can view, comment on it, or merge it online at: > > https://github.com/ops4j/org.ops4j.pax.runner/pull/2 > > -- Commit Summary -- > > * PAXRUNNER-406 fixed NPE > > -- File Changes -- > > M > pax-runner-platform/src/main/java/org/ops4j/pax/runner/platform/InProcessJavaRunner.java > (11) > > -- Patch Links -- > > https://github.com/ops4j/org.ops4j.pax.runner/pull/2.patch > https://github.com/ops4j/org.ops4j.pax.runner/pull/2.diff > > -- > Reply to this email directly or view it on GitHub: > https://github.com/ops4j/org.ops4j.pax.runner/pull/2 > -- Toni Menzel Source <http://tonimenzel.com>
_______________________________________________ general mailing list [email protected] http://lists.ops4j.org/mailman/listinfo/general
