Emm. Actually it's not a blocker or even critical for 4.2. Though I thought it's not harm, if needs I can revert it from 4.2 but still keep it in master.
> -----Original Message----- > From: Animesh Chaturvedi > Sent: Monday, August 12, 2013 5:37 PM > To: dev@cloudstack.apache.org; Frank Zhang; John Burwell > Cc: Laszlo Hornyak > Subject: RE: Review Request 11942: ProcessUtil cleanup > > Is this really needed for 4.2? 4.2 is in limited updates and only > blocker/critical > bug fixes and doc and test updates are allowed. All commits to 4.2 should have > associated BUG id. > > > -----Original Message----- > > From: Frank Zhang [mailto:nore...@reviews.apache.org] On Behalf Of > > Frank Zhang > > Sent: Monday, August 12, 2013 2:43 PM > > To: John Burwell; Frank Zhang > > Cc: Laszlo Hornyak; cloudstack > > Subject: Re: Review Request 11942: ProcessUtil cleanup > > > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/11942/#review25036 > > ----------------------------------------------------------- > > > > Ship it! > > > > > > This is very straightforward patch. Java filesystem api is known as > > unnecessary complex, we should take advantage of Apache common-io as > > much as possible. > > shipped in 4.2 and master > > > > - Frank Zhang > > > > > > On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote: > > > > > > ----------------------------------------------------------- > > > This is an automatically generated e-mail. To reply, visit: > > > https://reviews.apache.org/r/11942/ > > > ----------------------------------------------------------- > > > > > > (Updated July 24, 2013, 4:42 a.m.) > > > > > > > > > Review request for cloudstack, Frank Zhang and John Burwell. > > > > > > > > > Repository: cloudstack-git > > > > > > > > > Description > > > ------- > > > > > > - possible resource leak closed > > > - file content read uses now commons-lang FileUtils > > > - Added unit tests > > > > > > > > > Diffs > > > ----- > > > > > > utils/src/com/cloud/utils/ProcessUtil.java c9fdf35 > > > utils/test/com/cloud/utils/ProcessUtilTest.java PRE-CREATION > > > > > > Diff: https://reviews.apache.org/r/11942/diff/ > > > > > > > > > Testing > > > ------- > > > > > > test included > > > > > > > > > Thanks, > > > > > > Laszlo Hornyak > > > > > >