Branch: refs/heads/master
Home: https://github.com/jenkinsci/phabricator-plugin
Commit: e1f9ee855da61e508d98c4feb7e6658a8df17ced
https://github.com/jenkinsci/phabricator-plugin/commit/e1f9ee855da61e508d98c4feb7e6658a8df17ced
Author: Aiden Scandella <[email protected]>
Date: 2015-06-06 (Sat, 06 Jun 2015)
Changed paths:
M src/main/java/com/uber/jenkins/phabricator/PhabricatorBuildWrapper.java
M src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistClient.java
M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java
Log Message:
-----------
Use stdin for arcanist instead of piping via shell
This makes things a lot less hacky and allows us to output the command
we're running but scrub the conduit token (if present).
The BEST part of about this is we're no longer doing hacky string
escaping to avoid the fact that we're constructing JSON strings that
have to be shell-escaped.
We also now check for the results of the return code of calling arcanist.
Commit: 19e45271b3103733936248df9ceab110548506a7
https://github.com/jenkinsci/phabricator-plugin/commit/19e45271b3103733936248df9ceab110548506a7
Author: Aiden Scandella <[email protected]>
Date: 2015-06-06 (Sat, 06 Jun 2015)
Changed paths:
M src/main/java/com/uber/jenkins/phabricator/PhabricatorBuildWrapper.java
M src/main/java/com/uber/jenkins/phabricator/conduit/ArcanistClient.java
M src/main/java/com/uber/jenkins/phabricator/conduit/Differential.java
Log Message:
-----------
Merge pull request #25 from uber/input-stream
Use stdin for arcanist instead of piping via shell
Compare:
https://github.com/jenkinsci/phabricator-plugin/compare/fa63ce08a7c9...19e45271b310
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
