On Tue, Jul 19, 2011 at 11:37 AM, Paul Benedict <[email protected]>wrote:
> Gary, > > I think it's about intention. If you have intentions of further > expanding the user of Commons IO in test cases, I think it makes sense > to use it as a dependency. But if it is simply a one-off without plans > of further use, then I would copy the method. > Hello Paul, Ah, I see, yes, I do plan on reusing and not copying whenever possible, especially when that functionality is already well tested. That's what I do at work. Here, I certainly do not want to cut and paste one method, then another and another as time goes on. At what point is it no longer acceptable to cut and paste and depend on the jar? I do not see a clear metric. I also do not want to spend time undoing cutting and pasting methods to implement depending on the jar. That feels like spinning my wheels. Out of curiosity, I did look at how much code could would be needed to cut and paste from [io], and because of the way the code is factored, it would have been more than one method (5 or 6 I think.) The functionality can be easily written in one method, but I do not plan on spending my time doing that either, I prefer to reuse a well tested method. Gary > > Paul > > On Tue, Jul 19, 2011 at 10:26 AM, Gary Gregory <[email protected]> > wrote: > > HI All: > > > > I do not understand the general reluctance to reuse code in a clean way > > through jars, especially in this case for tests. > > > > I shake my head every time I open the search type dialog in Eclipse for > my > > (larger) work projects and see a dozen StringUtils classes (half of them > in > > org.apache). > > > > I understand that each project is independent and so on, but my hope is > that > > [commons] as one project could at least eat its own dog food. > > > > Gary > > > > On Tue, Jul 19, 2011 at 10:47 AM, Stephen Colebourne > > <[email protected]>wrote: > > > >> Personally, I'm OK with using JUnit and mocking utilities as they are > >> both specifically intended for testing. I think using IOUtils in > >> testing [lang] is distinctly dubious, especially for a single method, > >> and I'd much rather see the code copied. This isn't a -1 veto, but its > >> a strong disapproval. > >> > >> Stephen > >> > >> > >> On 19 July 2011 15:13, Gary Gregory <[email protected]> wrote: > >> > On Tue, Jul 19, 2011 at 9:28 AM, Paul Benedict <[email protected]> > >> wrote: > >> > > >> >> As long as Commons IO is marked as a test dependency, I am okay with > >> >> it. I just don't want it to be a compile-time dependency for the main > >> >> source. > >> >> > >> > > >> > It is specified in the test scope in the POM. > >> > > >> > Gary > >> > > >> > > >> >> > >> >> On Tue, Jul 19, 2011 at 8:24 AM, Gary Gregory < > [email protected]> > >> >> wrote: > >> >> > On Mon, Jul 18, 2011 at 9:25 PM, Henri Yandell <[email protected] > > > >> >> wrote: > >> >> > > >> >> >> Interesting issue; though thankfully it's post RC4 so not an issue > >> wrt > >> >> >> releasing 3.0. > >> >> >> > >> >> >> Assuming (for argument's sake) that IO Test depends on Lang & Lang > >> >> >> Test depends on IO; is this bad? I'm not convinced it is. Dealing > >> with > >> >> >> something like that is something the build system needs to know > how > >> to > >> >> >> do. > >> >> >> > >> >> > > >> >> > We depend on JUnit and EasyMock for testing, so I really think it > is > >> OK > >> >> to > >> >> > also depend on [io] for testing as well. C&P'ing code is lame in > this > >> >> case > >> >> > IMO. > >> >> > > >> >> > Gary > >> >> > > >> >> > > >> >> >> Hen > >> >> >> > >> >> >> On Mon, Jul 18, 2011 at 3:50 PM, Stephen Colebourne > >> >> >> <[email protected]> wrote: > >> >> >> > StringEscapeUtils test includes IOUtils, which it shouldn't. (If > >> its > >> >> >> > been added as a dependency, then it needs to be removed, even > for > >> >> >> > testing) > >> >> >> > > >> >> >> > Stephen > >> >> >> > > >> >> >> > On 18 July 2011 23:41, Gary Gregory <[email protected]> > >> wrote: > >> >> >> >> On Jul 18, 2011, at 18:36, Stephen Colebourne < > >> [email protected]> > >> >> >> wrote: > >> >> >> >> > >> >> >> >>> I'm willing to vote +1 > >> >> >> >>> Although I haven't checked every recent change, but AFAIK > recent > >> >> >> >>> changes have been minor and my previous issues are resolved. > >> >> >> >>> > >> >> >> >>> I would note that the svn as of right now does not compile, > due > >> to > >> >> an > >> >> >> >>> IOUtils reference that shouldn't be > >> >> >> >> > >> >> >> >> Hi Stephen, > >> >> >> >> > >> >> >> >> Can you specify what your error is? I check both the maven and > ant > >> >> >> >> builds before my commit. > >> >> >> >> > >> >> >> >> Gary > >> >> >> >> > >> >> >> >>> > >> >> >> >>> Stephen > >> >> >> >>> > >> >> >> >>> > >> >> >> >>> On 16 July 2011 01:18, Henri Yandell <[email protected]> > wrote: > >> >> >> >>>> Thanks Gary. > >> >> >> >>>> > >> >> >> >>>> So 4 +1s. > >> >> >> >>>> > >> >> >> >>>> Stephen, Niall, Paul, Phil, Sebb, James - nudge to consider > >> voting > >> >> >> >>>> (apologies if I missed anyone else who has committed to Lang > >> 3.0)? > >> >> >> >>>> > >> >> >> >>>> Hen > >> >> >> >>>> > >> >> >> >>>> On Fri, Jul 15, 2011 at 12:32 PM, Gary Gregory < > >> >> >> [email protected]> wrote: > >> >> >> >>>>> That's true too. In the spirit of release early, release > often, > >> I > >> >> >> remove my > >> >> >> >>>>> -1 :) > >> >> >> >>>>> > >> >> >> >>>>> Gary > >> >> >> >>>>> > >> >> >> >>>>> > >> >> >> >>>>>> On Fri, Jul 15, 2011 at 10:54 AM, Henri Yandell < > >> >> [email protected] > >> >> >> >wrote: > >> >> >> >>>>>> > >> >> >> >>>>>>> Less that it is painful (though I agree that it is), more > >> that > >> >> if > >> >> >> you > >> >> >> >>>>>>> hold up a release for every bug that comes in then you > >> >> continually > >> >> >> sit > >> >> >> >>>>>>> in a non-releasing state. We have a really bad habit of > that > >> in > >> >> >> >>>>>>> Commons, constantly polishing and polishing before a > release. > >> >> >> >>>>>>> > >> >> >> >>>>>>> Hen > >> >> >> >>>>>>> > >> >> >> >>>>>>> On Fri, Jul 15, 2011 at 6:58 AM, Gary Gregory < > >> >> >> [email protected]> > >> >> >> >>>>>>> wrote: > >> >> >> >>>>>>>> Here is my main issue: we are releasing a major new > version > >> and > >> >> >> there is > >> >> >> >>>>>>> a > >> >> >> >>>>>>>> known bug reported by a user which has been fixed in SVN. > It > >> >> feels > >> >> >> like > >> >> >> >>>>>>> we > >> >> >> >>>>>>>> are unwilling to cut a new RC because our build process > and > >> >> >> validation > >> >> >> >>>>>>> is > >> >> >> >>>>>>>> painful (it is so in my experience at least, your mileage > >> may > >> >> vary > >> >> >> using > >> >> >> >>>>>>>> custom scripts, Nexus, or other incantations.) This is > not a > >> >> good > >> >> >> reason > >> >> >> >>>>>>> IMO > >> >> >> >>>>>>>> to avoid rebuilding. In the case of a major release like > >> 3.0, I > >> >> do > >> >> >> not > >> >> >> >>>>>>> want > >> >> >> >>>>>>>> to leave a bad taste in a user's mouth with a class that > is > >> not > >> >> >> fully > >> >> >> >>>>>>> baked, > >> >> >> >>>>>>>> especially in code new to 3.0. I like that we are > planning a > >> >> >> 3.0.1, but > >> >> >> >>>>>>> I do > >> >> >> >>>>>>>> not see why we should not include something that is > already > >> >> fixed > >> >> >> for > >> >> >> >>>>>>> 3.0. > >> >> >> >>>>>>>> It's not like this issue needs more time on > investigating, > >> >> coding, > >> >> >> and > >> >> >> >>>>>>>> testing. > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> Now, if you all really think I am being unreasonable, > I'll > >> be > >> >> >> happy to > >> >> >> >>>>>>> go > >> >> >> >>>>>>>> with the flow and reverse -1, but for now, I wanted to > >> express > >> >> my > >> >> >> full > >> >> >> >>>>>>> POV. > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> Thank you for reading and talking :) > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> Gary > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> On Fri, Jul 15, 2011 at 3:44 AM, Henri Yandell < > >> >> >> [email protected]> > >> >> >> >>>>>>> wrote: > >> >> >> >>>>>>>>> > >> >> >> >>>>>>>>> Waiting on you to determine whether your -1 is still > there > >> on > >> >> >> LANG-720. > >> >> >> >>>>>>>>> > >> >> >> >>>>>>>>> Then need to poke Niall, Stephen et al to do a review :) > >> >> >> >>>>>>>>> > >> >> >> >>>>>>>>> On Thu, Jul 14, 2011 at 11:54 AM, Gary Gregory < > >> >> >> [email protected] > >> >> >> >>>>>>>> > >> >> >> >>>>>>>>> wrote: > >> >> >> >>>>>>>>>> -1, let's pick up the committed fix for > >> >> >> >>>>>>>>>> https://issues.apache.org/jira/browse/LANG-720 > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>>>> I recall seeing traffic in the escape/unescape area so > it > >> >> makes > >> >> >> sense > >> >> >> >>>>>>> to > >> >> >> >>>>>>>>>> polish this new code as much as possible IMO. > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>>>> Gary > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>>>> On Thu, Jul 14, 2011 at 12:47 AM, Henri Yandell < > >> >> >> [email protected]> > >> >> >> >>>>>>>>>> wrote: > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>>>>> Lang is ready to consider 3.0 release again. > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> RC4 is available here: > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > http://people.apache.org/~bayard/commons-lang3-3.0-RC4/ > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> SVN: > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>> > >> >> >> > >> http://svn.apache.org/repos/asf/commons/proper/lang/tags/LANG_3_0_RC4/ > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> Maven artifacts: > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> http://people.apache.org/~bayard/commons-lang3-3.0-RC4/maven/ > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> Website: > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> http://people.apache.org/~bayard/commons-lang3-3.0-RC4/site/ > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> Note that there is a 2.6->3.0 Clirr report in the site > >> that > >> >> may > >> >> >> >>>>>>> prove > >> >> >> >>>>>>>>>>> useful: > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>> > >> >> >> > >> >> > >> > http://people.apache.org/~bayard/commons-lang3-3.0-RC4/site/lang2-lang3-clirr--report.html > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> This vote will close no sooner than in 72 hours time, > >> 0500 > >> >> GMT > >> >> >> 16 > >> >> >> >>>>>>> July > >> >> >> >>>>>>>>>>> 2011. > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> ================ > >> >> >> >>>>>>>>>>> [ ] +1 > >> >> >> >>>>>>>>>>> [ ] -1, with reason > >> >> >> >>>>>>>>>>> ================ > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> Hen > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> *fingers crossed - two of my children are younger than > >> the > >> >> Lang > >> >> >> 3.0 > >> >> >> >>>>>>>>>>> effort* > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>> > >> >> >> > --------------------------------------------------------------------- > >> >> >> >>>>>>>>>>> To unsubscribe, e-mail: > >> [email protected] > >> >> >> >>>>>>>>>>> For additional commands, e-mail: > >> >> [email protected] > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>>> > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>>>> -- > >> >> >> >>>>>>>>>> Thank you, > >> >> >> >>>>>>>>>> Gary > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>>>> http://garygregory.wordpress.com/ > >> >> >> >>>>>>>>>> http://garygregory.com/ > >> >> >> >>>>>>>>>> http://people.apache.org/~ggregory/ > >> >> >> >>>>>>>>>> http://twitter.com/GaryGregory > >> >> >> >>>>>>>>>> > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> -- > >> >> >> >>>>>>>> Thank you, > >> >> >> >>>>>>>> Gary > >> >> >> >>>>>>>> > >> >> >> >>>>>>>> http://garygregory.wordpress.com/ > >> >> >> >>>>>>>> http://garygregory.com/ > >> >> >> >>>>>>>> http://people.apache.org/~ggregory/ > >> >> >> >>>>>>>> http://twitter.com/GaryGregory > >> >> >> >>>>>>>> > >> >> >> >>>>>>> > >> >> >> >>>>>> > >> >> >> >>>>>> > >> >> >> >>>>>> > >> >> >> >>>>>> -- > >> >> >> >>>>>> Thank you, > >> >> >> >>>>>> Gary > >> >> >> >>>>>> > >> >> >> >>>>>> http://garygregory.wordpress.com/ > >> >> >> >>>>>> http://garygregory.com/ > >> >> >> >>>>>> http://people.apache.org/~ggregory/ > >> >> >> >>>>>> http://twitter.com/GaryGregory > >> >> >> >>>>>> > >> >> >> >>>>> > >> >> >> >>>>> > >> >> >> >>>>> > >> >> >> >>>>> -- > >> >> >> >>>>> Thank you, > >> >> >> >>>>> Gary > >> >> >> >>>>> > >> >> >> >>>>> http://garygregory.wordpress.com/ > >> >> >> >>>>> http://garygregory.com/ > >> >> >> >>>>> http://people.apache.org/~ggregory/ > >> >> >> >>>>> http://twitter.com/GaryGregory > >> >> >> >>>>> > >> >> >> >>>> > >> >> >> >>>> > >> >> --------------------------------------------------------------------- > >> >> >> >>>> To unsubscribe, e-mail: [email protected] > >> >> >> >>>> For additional commands, e-mail: [email protected] > >> >> >> >>>> > >> >> >> >>>> > >> >> >> >>> > >> >> >> >>> > >> >> --------------------------------------------------------------------- > >> >> >> >>> To unsubscribe, e-mail: [email protected] > >> >> >> >>> For additional commands, e-mail: [email protected] > >> >> >> >>> > >> >> >> >> > >> >> >> >> > >> --------------------------------------------------------------------- > >> >> >> >> To unsubscribe, e-mail: [email protected] > >> >> >> >> For additional commands, e-mail: [email protected] > >> >> >> >> > >> >> >> >> > >> >> >> > > >> >> >> > > >> --------------------------------------------------------------------- > >> >> >> > To unsubscribe, e-mail: [email protected] > >> >> >> > For additional commands, e-mail: [email protected] > >> >> >> > > >> >> >> > > >> >> >> > >> >> >> > --------------------------------------------------------------------- > >> >> >> To unsubscribe, e-mail: [email protected] > >> >> >> For additional commands, e-mail: [email protected] > >> >> >> > >> >> >> > >> >> > > >> >> > > >> >> > -- > >> >> > Thank you, > >> >> > Gary > >> >> > > >> >> > http://garygregory.wordpress.com/ > >> >> > http://garygregory.com/ > >> >> > http://people.apache.org/~ggregory/ > >> >> > http://twitter.com/GaryGregory > >> >> > > >> >> > >> >> --------------------------------------------------------------------- > >> >> To unsubscribe, e-mail: [email protected] > >> >> For additional commands, e-mail: [email protected] > >> >> > >> >> > >> > > >> > > >> > -- > >> > Thank you, > >> > Gary > >> > > >> > http://garygregory.wordpress.com/ > >> > http://garygregory.com/ > >> > http://people.apache.org/~ggregory/ > >> > http://twitter.com/GaryGregory > >> > > >> > >> --------------------------------------------------------------------- > >> To unsubscribe, e-mail: [email protected] > >> For additional commands, e-mail: [email protected] > >> > >> > > > > > > -- > > Thank you, > > Gary > > > > http://garygregory.wordpress.com/ > > http://garygregory.com/ > > http://people.apache.org/~ggregory/ > > http://twitter.com/GaryGregory > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > > -- Thank you, Gary http://garygregory.wordpress.com/ http://garygregory.com/ http://people.apache.org/~ggregory/ http://twitter.com/GaryGregory
