-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20173/
-----------------------------------------------------------

(Updated April 10, 2014, 3:31 a.m.)


Review request for mesos and Vinod Kone.


Changes
-------

First patch was thrown together as a proof of concept, updated based on 
comments and refactored


Bugs: MESOS-1202
    https://issues.apache.org/jira/browse/MESOS-1202


Repository: mesos-git


Description
-------

Updates support/apply-review.sh to work with both github and reviewboard.


Diffs (updated)
-----

  support/apply-review.sh 8e2f916 

Diff: https://reviews.apache.org/r/20173/diff/


Testing (updated)
-------

Tested against review 11099 and pull request 11 and printed out author and 
message to be committed. 


trunk $ ./support/apply-review.sh -r 11099
--2014-04-09 23:21:21--  https://reviews.apache.org/r/11099/diff/raw/
Resolving reviews.apache.org... 140.211.11.74
Connecting to reviews.apache.org|140.211.11.74|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 1388 (1.4K) [text/x-patch]
Saving to: ‘11099.patch’

100%[============================================================================================================>]
 1,388       --.-K/s   in 0s      

2014-04-09 23:21:22 (27.0 MB/s) - ‘11099.patch’ saved [1388/1388]

Author: Vinod Kone <[email protected]>
Added rename() to os.hpp.

Discarding old review, because post-reviews is giving me issues with the recent 
git repo changes.

Old: https://reviews.apache.org/r/11056


Review: https://reviews.apache.org/r/11099


trunk $ ./support/apply-review.sh -g 11
--2014-04-09 23:21:36--  https://github.com/apache/mesos/pull/11.patch
Resolving github.com... 192.30.252.129
Connecting to github.com|192.30.252.129|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 1287 (1.3K) [text/plain]
Saving to: ‘11.patch’

100%[============================================================================================================>]
 1,287       --.-K/s   in 0s      

2014-04-09 23:21:36 (5.66 MB/s) - ‘11.patch’ saved [1287/1287]

11.patch:22: trailing whitespace.
Jenkins, Spark, Aurora, and other frameworks on a dynamically shared pool of 
warning: 1 line adds whitespace errors.

Author: Dave Lester <[email protected]>
Updates README project description.

Updates README project description to be in sync with language used on the 
Mesos website.

This closes: #11
Review: https://github.com/apache/mesos/pull/11


Thanks,

Jake Farrell

Reply via email to