> On Dec. 16, 2012, 10:39 p.m., Ali Lown wrote: > > get_third_party.sh, line 41 > > <https://reviews.apache.org/r/8630/diff/2/?file=239263#file239263line41> > > > > It feels like these should be merged into a function 'fetch'(?), given > > they all: > > [[-f]], dir, get, mkdir -p > > > > You can always pass a function as an argument to do the specific cp, rm > > etc. for each 'fetch'
Maybe, but I think that passing functions as argument won't make the script more clear. - Yuri ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8630/#review14572 ----------------------------------------------------------- On Dec. 18, 2012, 8:25 p.m., Yuri Zelikov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8630/ > ----------------------------------------------------------- > > (Updated Dec. 18, 2012, 8:25 p.m.) > > > Review request for wave, Michael MacFadden, Vicente J. Ruiz Jurado, and Ali > Lown. > > > Description > ------- > > Adds get_third_party script to download junit and emma dependencies. > It also updates the junit to version 4.10 since I couldn't find download for > the old 3.8.2 version. > > > Diffs > ----- > > .gitignore 151cca9 > README eba3b57 > get_third_party.sh PRE-CREATION > third_party/test/emma/emma.jar 52f80e5 > third_party/test/emma/emma_ant.jar ee5134b > third_party/test/junit/README.google f92d92b > third_party/test/junit/junit.jar c8f711d > third_party/test/junit/src.jar 28ea5fc > > Diff: https://reviews.apache.org/r/8630/diff/ > > > Testing > ------- > > Executed "ant test" > > > Thanks, > > Yuri Zelikov > >
