> On March 14, 2013, 1:37 p.m., Vicente J. Ruiz Jurado wrote: > > As junit github download is not working, following Thomas Broyer comment, > > we can use this code in your new script: > > > > [[ -f $out/junit/junit.jar ]] || ( > > dir junit > > get http://repo1.maven.org/maven2/junit/junit/4.10/ junit-4.10.jar > > get http://repo1.maven.org/maven2/junit/junit/4.10/ junit-4.10-sources.jar > > mkdir -p $out/junit > > cp junit-4.10.jar $out/junit/junit.jar > > cp junit-4.10-sources.jar $out/junit/src.jar > > cd .. > > rm -rf junit > > ) > >
LGTM, but can I help with this review? - Vicente J. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9045/#review17872 ----------------------------------------------------------- On Jan. 21, 2013, 7:05 p.m., Yuri Zelikov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/9045/ > ----------------------------------------------------------- > > (Updated Jan. 21, 2013, 7:05 p.m.) > > > Review request for wave, Vicente J. Ruiz Jurado and Ali Lown. > > > Description > ------- > > Enhancements for get-third-party.sh script. > 1.Renames the script to use "-" instead of "_" to comply with the naming > policy > 2. Adds validation to the build.xml to warn in case the test dependencies are > missing. > > > Diffs > ----- > > README da77f49 > build.properties c476a82 > build.xml 3d3c125 > get-third-party.sh PRE-CREATION > get_third_party.sh d8b1ce2 > > Diff: https://reviews.apache.org/r/9045/diff/ > > > Testing > ------- > > > Thanks, > > Yuri Zelikov > >
