# Re: [JGIT] Request for help 10 messages from 2009-09-04 to 2009-09-05. Participants: Mark Struberg, Jonas Fonseca, Gabe, Douglas Campos, Gabe McArthur, Robin Rosenberg. Thread: https://gitlist.dev/t/20852 ## Mark Struberg, 2009-09-04 14:49 Subject: Re: [JGIT] Request for help Message-ID: <585278.66341.qm@web27802.mail.ukl.yahoo.com> URL: https://gitlist.dev/e/585278.66341.qm%40web27802.mail.ukl.yahoo.com In-Reply-To: ``` Hi Douglas! http://github.com/sonatype/JGit The branch will be called mavenizing or so. Will post this after I got the tests running. LieGrue, strub --- On Fri, 9/4/09, Douglas Campos wrote: > From: Douglas Campos > Subject: Re: [JGIT] Request for help > To: "Mark Struberg" > Cc: "Jonas Fonseca" , git@vger.kernel.org, "Gabe McArthur" > Date: Friday, September 4, 2009, 4:44 PM > On Fri, Sep 4, 2009 > at 9:47 AM, Mark Struberg > wrote: > > > as an old saying tells us: how to climb a mountain? step > after step! ;) > > > > I suggest we create a fresh branch based on the Shawns > current version and add all the features incrementally. > > > > please point out where this branch will happen, I want to > give some help too. > > ``` ## Mark Struberg, 2009-09-04 17:28 Subject: Re: [JGIT] Request for help Message-ID: <658028.86274.qm@web27804.mail.ukl.yahoo.com> URL: https://gitlist.dev/e/658028.86274.qm%40web27804.mail.ukl.yahoo.com In-Reply-To: <585278.66341.qm@web27802.mail.ukl.yahoo.com> ``` Hi! Work has been done at http://github.com/sonatype/JGit/tree/mavenize Please feel free to pull/fork and share your changes! I'd be happy to pull it in. @Gabe: your patch seems to got filtered by the list, I think sharing such big things is easier by using github. Would be cool if you could help us! LieGrue, strub --- On Fri, 9/4/09, Mark Struberg wrote: > From: Mark Struberg > Subject: Re: [JGIT] Request for help > To: "Douglas Campos" > Cc: "Jonas Fonseca" , git@vger.kernel.org, "Gabe McArthur" > Date: Friday, September 4, 2009, 4:49 PM > Hi Douglas! > > http://github.com/sonatype/JGit > > The branch will be called mavenizing or so. > > Will post this after I got the tests running. > > LieGrue, > strub > > --- On Fri, 9/4/09, Douglas Campos > wrote: > > > From: Douglas Campos > > Subject: Re: [JGIT] Request for help > > To: "Mark Struberg" > > Cc: "Jonas Fonseca" , > git@vger.kernel.org, > "Gabe McArthur" > > Date: Friday, September 4, 2009, 4:44 PM > > On Fri, Sep 4, 2009 > > at 9:47 AM, Mark Struberg > > wrote: > > > > > > as an old saying tells us: how to climb a mountain? > step > > after step! ;) > > > > > > > > I suggest we create a fresh branch based on the > Shawns > > current version and add all the features > incrementally. > > > > > > > > please point out where this branch will happen, I want > to > > give some help too. > > > > > > >       > -- > To unsubscribe from this list: send the line "unsubscribe > git" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at  http://vger.kernel.org/majordomo-info.html > ``` ## Jonas Fonseca, 2009-09-04 18:50 Subject: Re: [JGIT] Request for help Message-ID: <2c6b72b30909041150g6374be2ci4d36bd8ab0824a8d@mail.gmail.com> URL: https://gitlist.dev/e/2c6b72b30909041150g6374be2ci4d36bd8ab0824a8d%40mail.gmail.com In-Reply-To: <658028.86274.qm@web27804.mail.ukl.yahoo.com> ``` On Fri, Sep 4, 2009 at 13:28, Mark Struberg wrote: > Hi! > > Work has been done at > > http://github.com/sonatype/JGit/tree/mavenize > > Please feel free to pull/fork and share your changes! I'd be happy to pull it in. IMO, there are a lot of things that can be squashed together and cleaned up. I know that you advocated for incremental introduction, but it seems wrong to for example add a file and then completely reformat it a few commits later. The same thing with the .gitignore fixes in step 5. Some comments ... Some of them I initially entered in github's codereview, but I ended up writing it all here. Commit: "mavenizing step 1: moved over the initial poms from Jasons branch" * Please always add an empty line between the subject and the body of the commit message. Like this: mavenizing step 1: moved over the initial poms from Jasons branch Signed-off-by: Mark Struberg >struberg@yahoo.de> * The .gitignore pattern could be further limited to "target/" ... but you seem to change this to /target later. In org.spearce.jgit/pom.xml: * The use of maven-surefire-plugin should be removed. This module does not have any tests. * Shouldn't we retain the original ${groupId}:${artifactId} naming convention, being org.spearce:jgit? In org.spearce.jgit.test/pom.xml: * Dependency on jsch is unecessary since it is derived from org.spearce.jgit. * Maybe name as org.spearce:jgit-test? In org.spearce.jgit.pgm/pom.xml: * Maybe name as org.spearce:jgit-pgm? Commit: "mavenizing step 2: move the core libs from src to src/main/java" * Please also add an empty line to this commit message. * You might as well squash the whitespace fixes into the first commit. Commit: "mavenizing step 3: moving all core tests into the core module" * The commit message wrongly states: org.spearce.jgit.test/tst/ -> org.spearce.jgit/src/test/java/tst/ Should be: org.spearce.jgit.test/tst/ -> org.spearce.jgit/src/test/java/ Commit: "mavenizing step 4: moving some license files and META-INF" * Shouldn't the commit message rather say "remove JSch"? Then the moving of META-INF can be put in its own commit. * The new NOTICE file has a few typos and the info could fit into the README Then I got a bit lost in a huge reformatting. -- Jonas Fonseca ``` ## Mark Struberg, 2009-09-04 18:54 Subject: Re: [JGIT] Request for help Message-ID: <78662.25308.qm@web27808.mail.ukl.yahoo.com> URL: https://gitlist.dev/e/78662.25308.qm%40web27808.mail.ukl.yahoo.com In-Reply-To: <2c6b72b30909041150g6374be2ci4d36bd8ab0824a8d@mail.gmail.com> ``` doing a rebase -i with new stuff atm... ;) LieGrue, strub --- On Fri, 9/4/09, Jonas Fonseca wrote: > From: Jonas Fonseca > Subject: Re: [JGIT] Request for help > To: "Mark Struberg" > Cc: "Douglas Campos" , git@vger.kernel.org, "Gabe McArthur" > Date: Friday, September 4, 2009, 8:50 PM > On Fri, Sep 4, 2009 at 13:28, Mark > Struberg > wrote: > > Hi! > > > > Work has been done at > > > > http://github.com/sonatype/JGit/tree/mavenize > > > > Please feel free to pull/fork and share your changes! > I'd be happy to pull it in. > > IMO, there are a lot of things that can be squashed > together and > cleaned up. I know that you advocated for incremental > introduction, > but it seems wrong to for example add a file and then > completely > reformat it a few commits later. The same thing with the > .gitignore > fixes in step 5. > > Some comments ... Some of them I initially entered in > github's > codereview, but I ended up writing it all here. > > Commit: "mavenizing step 1: moved over the initial poms > from Jasons branch" > > * Please always add an empty line between the subject and > the body >    of the commit message. Like this: > >   mavenizing step 1: moved over the initial poms from > Jasons branch > >   Signed-off-by: Mark Struberg >struberg@yahoo.de> > > * The .gitignore pattern could be further limited to > "target/" ... > but you seem to change this to /target later. > > In org.spearce.jgit/pom.xml: > >     * The use of maven-surefire-plugin should be > removed. This module > does not have any tests. > >     * Shouldn't we retain the original > ${groupId}:${artifactId} naming > convention, being org.spearce:jgit? > > In org.spearce.jgit.test/pom.xml: > >     * Dependency on jsch is unecessary since it > is derived from > org.spearce.jgit. > >     * Maybe name as org.spearce:jgit-test? > > In org.spearce.jgit.pgm/pom.xml: > >     * Maybe name as org.spearce:jgit-pgm? > > Commit: "mavenizing step 2: move the core libs from src to > src/main/java" > > * Please also add an empty line to this commit message. > > * You might as well squash the whitespace fixes into the > first commit. > > Commit: "mavenizing step 3: moving all core tests into the > core module" > > * The commit message wrongly states: >     org.spearce.jgit.test/tst/ -> > org.spearce.jgit/src/test/java/tst/ >    Should be: >     org.spearce.jgit.test/tst/ -> > org.spearce.jgit/src/test/java/ > > Commit: "mavenizing step 4: moving some license files and > META-INF" > > * Shouldn't the commit message rather say "remove JSch"? >    Then the moving of META-INF can be put in > its own commit. > > * The new NOTICE file has a few typos and the info could > fit into the README > > Then I got a bit lost in a huge reformatting. > > -- > Jonas Fonseca > ``` ## Mark Struberg, 2009-09-04 19:51 Subject: Re: [JGIT] Request for help Message-ID: <904535.28308.qm@web27807.mail.ukl.yahoo.com> URL: https://gitlist.dev/e/904535.28308.qm%40web27807.mail.ukl.yahoo.com In-Reply-To: <2c6b72b30909041150g6374be2ci4d36bd8ab0824a8d@mail.gmail.com> ``` Thanks Jonas! I now squashed a lot of commits together where possible and republished the rebased branch. Next steps: * mavenizing org.spearce.jgit.pgm LieGrue, strub --- On Fri, 9/4/09, Jonas Fonseca wrote: > From: Jonas Fonseca > Subject: Re: [JGIT] Request for help > To: "Mark Struberg" > Cc: "Douglas Campos" , git@vger.kernel.org, "Gabe McArthur" > Date: Friday, September 4, 2009, 8:50 PM > On Fri, Sep 4, 2009 at 13:28, Mark > Struberg > wrote: > > Hi! > > > > Work has been done at > > > > http://github.com/sonatype/JGit/tree/mavenize > > > > Please feel free to pull/fork and share your changes! > I'd be happy to pull it in. > > IMO, there are a lot of things that can be squashed > together and > cleaned up. I know that you advocated for incremental > introduction, > but it seems wrong to for example add a file and then > completely > reformat it a few commits later. The same thing with the > .gitignore > fixes in step 5. > > Some comments ... Some of them I initially entered in > github's > codereview, but I ended up writing it all here. > > Commit: "mavenizing step 1: moved over the initial poms > from Jasons branch" > > * Please always add an empty line between the subject and > the body >    of the commit message. Like this: > >   mavenizing step 1: moved over the initial poms from > Jasons branch > >   Signed-off-by: Mark Struberg >struberg@yahoo.de> > > * The .gitignore pattern could be further limited to > "target/" ... > but you seem to change this to /target later. > > In org.spearce.jgit/pom.xml: > >     * The use of maven-surefire-plugin should be > removed. This module > does not have any tests. > >     * Shouldn't we retain the original > ${groupId}:${artifactId} naming > convention, being org.spearce:jgit? > > In org.spearce.jgit.test/pom.xml: > >     * Dependency on jsch is unecessary since it > is derived from > org.spearce.jgit. > >     * Maybe name as org.spearce:jgit-test? > > In org.spearce.jgit.pgm/pom.xml: > >     * Maybe name as org.spearce:jgit-pgm? > > Commit: "mavenizing step 2: move the core libs from src to > src/main/java" > > * Please also add an empty line to this commit message. > > * You might as well squash the whitespace fixes into the > first commit. > > Commit: "mavenizing step 3: moving all core tests into the > core module" > > * The commit message wrongly states: >     org.spearce.jgit.test/tst/ -> > org.spearce.jgit/src/test/java/tst/ >    Should be: >     org.spearce.jgit.test/tst/ -> > org.spearce.jgit/src/test/java/ > > Commit: "mavenizing step 4: moving some license files and > META-INF" > > * Shouldn't the commit message rather say "remove JSch"? >    Then the moving of META-INF can be put in > its own commit. > > * The new NOTICE file has a few typos and the info could > fit into the README > > Then I got a bit lost in a huge reformatting. > > -- > Jonas Fonseca > -- > To unsubscribe from this list: send the line "unsubscribe > git" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at  http://vger.kernel.org/majordomo-info.html > ``` ## Gabe, 2009-09-04 23:47 Subject: Re: [JGIT] Request for help Message-ID: <524457d10909041647u562601d5q69142eefe894ac5b@mail.gmail.com> URL: https://gitlist.dev/e/524457d10909041647u562601d5q69142eefe894ac5b%40mail.gmail.com In-Reply-To: <658028.86274.qm@web27804.mail.ukl.yahoo.com> ``` On Fri, Sep 4, 2009 at 10:28 AM, Mark Struberg wrote: > Hi! > > Work has been done at > > http://github.com/sonatype/JGit/tree/mavenize > > Please feel free to pull/fork and share your changes! I'd be happy to pull it in. > > @Gabe: your patch seems to got filtered by the list, I think sharing such big things is easier by using github. Would be cool if you could help us! Ok, I'll fork and send a patch request shortly. I was thinking about it earlier, and I may add a couple of features that all OS projects should follow (e.g. License in the jar, etc.). As to a few questions that have been raised: 1) I pick the 'sources' folder because it's good metadata management. Everything in the root folder should be about or related to managing the project. No direct source folders, as it clutters the layout. Best to be perfectly clear where all the action is happening. It's a simple convention I wished more projects followed. 2) I haven't worked with the find-bugs plugin. I looked it up, but it seems to only generate documents in the 'site'/reporting profile. Thus it wouldn't necessarily affect the building of the software. It would really only be useful if you had something like a Hudson CI infrastructure or site generation going on to build a website and show the reports. I could certainly add that, though, if you like. 3) The LICENSE file can be at the top level. Not really an issue for me one way or another. Just a personal preference on how I have structured all of my previous Maven projects. -Gabe > > LieGrue, > strub > > --- On Fri, 9/4/09, Mark Struberg wrote: > >> From: Mark Struberg >> Subject: Re: [JGIT] Request for help >> To: "Douglas Campos" >> Cc: "Jonas Fonseca" , git@vger.kernel.org, "Gabe McArthur" >> Date: Friday, September 4, 2009, 4:49 PM >> Hi Douglas! >> >> http://github.com/sonatype/JGit >> >> The branch will be called mavenizing or so. >> >> Will post this after I got the tests running. >> >> LieGrue, >> strub >> >> --- On Fri, 9/4/09, Douglas Campos >> wrote: >> >> > From: Douglas Campos >> > Subject: Re: [JGIT] Request for help >> > To: "Mark Struberg" >> > Cc: "Jonas Fonseca" , >> git@vger.kernel.org, >> "Gabe McArthur" >> > Date: Friday, September 4, 2009, 4:44 PM >> > On Fri, Sep 4, 2009 >> > at 9:47 AM, Mark Struberg >> > wrote: >> > >> > >> > as an old saying tells us: how to climb a mountain? >> step >> > after step! ;) >> > >> > >> > >> > I suggest we create a fresh branch based on the >> Shawns >> > current version and add all the features >> incrementally. >> > >> > >> > >> > please point out where this branch will happen, I want >> to >> > give some help too. >> > >> > >> >> >> >> -- >> To unsubscribe from this list: send the line "unsubscribe >> git" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at  http://vger.kernel.org/majordomo-info.html >> > > > > ``` ## Douglas Campos, 2009-09-05 00:06 Subject: Re: [JGIT] Request for help Message-ID: URL: https://gitlist.dev/e/ed88cb980909041706n26e50107m2343d4d922788459%40mail.gmail.com In-Reply-To: <524457d10909041647u562601d5q69142eefe894ac5b@mail.gmail.com> ``` > Ok, I'll fork and send a patch request shortly.  I was thinking about > it earlier, and I may add a couple of features that all OS projects > should follow (e.g. License in the jar, etc.). Gabe, is there some task that you want to share with me? I have a short timeframe of 4hours to invest on mavenization. Cheers Douglas Campos (qmx) ``` ## Gabe McArthur, 2009-09-05 01:29 Subject: Re: [JGIT] Request for help Message-ID: <63CD88F1-B28B-42B8-A78B-485DB61487D7@gmail.com> URL: https://gitlist.dev/e/63CD88F1-B28B-42B8-A78B-485DB61487D7%40gmail.com In-Reply-To: ``` I'll post the pull request to github within 3-4 hours. Is that what you mean by invest? My patch will contain everything I submitted before plus a bit more. That patch set should contain everything necessary to build, plus any refinements to whatever is already in the 'mavenize' branch. -Gabe On Sep 4, 2009, at 5:06 PM, Douglas Campos wrote: >> Ok, I'll fork and send a patch request shortly. I was thinking about >> it earlier, and I may add a couple of features that all OS projects >> should follow (e.g. License in the jar, etc.). > > Gabe, is there some task that you want to share with me? I have a > short timeframe of 4hours to invest on mavenization. > > Cheers > Douglas Campos (qmx) ``` ## Robin Rosenberg, 2009-09-05 16:25 Subject: Re: [JGIT] Request for help Message-ID: <200909051825.49619.robin.rosenberg.lists@dewire.com> URL: https://gitlist.dev/e/200909051825.49619.robin.rosenberg.lists%40dewire.com In-Reply-To: <658028.86274.qm@web27804.mail.ukl.yahoo.com> ``` fredag 04 september 2009 19:28:39 skrev Mark Struberg : > Hi! > > Work has been done at > > http://github.com/sonatype/JGit/tree/mavenize > > Please feel free to pull/fork and share your changes! I'd be happy to pull it in. > Why does this new mvn test only execute 1024 tests here, while the old maven setup does 1108 ones? It seems the classes that don't match *Test.java are omitted. In both cases I invoke with "mvn clean test" -- robin ``` ## Mark Struberg, 2009-09-05 16:40 Subject: Re: [JGIT] Request for help Message-ID: <513639.15699.qm@web27805.mail.ukl.yahoo.com> URL: https://gitlist.dev/e/513639.15699.qm%40web27805.mail.ukl.yahoo.com In-Reply-To: <200909051825.49619.robin.rosenberg.lists@dewire.com> ``` Haven't counted it, but I will check it. Please note that for running the tests previously in 'exttest' you have to activate the tck profile: $> mvn test -Ptck And yes, we currently only run *Test.java. Any other patterns/files to include? txs and LieGrue, strub --- On Sat, 9/5/09, Robin Rosenberg wrote: > From: Robin Rosenberg > Subject: Re: [JGIT] Request for help > To: "Mark Struberg" > Cc: "Douglas Campos" , "Jonas Fonseca" , git@vger.kernel.org, "Gabe McArthur" > Date: Saturday, September 5, 2009, 6:25 PM > fredag 04 september 2009 19:28:39 > skrev Mark Struberg : > > Hi! > > > > Work has been done at > > > > http://github.com/sonatype/JGit/tree/mavenize > > > > Please feel free to pull/fork and share your changes! > I'd be happy to pull it in. > > > > Why does this new mvn test only execute 1024 tests here, > while the old maven setup > does 1108 ones? It seems the classes that don't match > *Test.java are omitted. > > In both cases I invoke with "mvn clean test" > > -- robin > ```