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 > > > >