git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jonas Fonseca <jonas.fonseca@gmail.com>
To: "Shawn O. Pearce" <spearce@spearce.org>
Cc: Johannes Schindelin <Johannes.Schindelin@gmx.de>,
	Nasser Grainawi <nasser@codeaurora.org>,
	Git Mailing List <git@vger.kernel.org>
Subject: Re: [JGIT] Request for help
Date: Thu, 3 Sep 2009 11:38:47 -0400	[thread overview]
Message-ID: <2c6b72b30909030838q10b88705gb8c553f2c2d74379@mail.gmail.com> (raw)
In-Reply-To: <20090903144227.GH1033@spearce.org>

On Thu, Sep 3, 2009 at 10:42, Shawn O. Pearce<spearce@spearce.org> wrote:
> Jonas Fonseca <jonas.fonseca@gmail.com> wrote:
>> BTW, what is your opinion of making it a bit easier to import and use
>> the Maven configuration by putting a pom.xml in the top-level
>> directory? The actual pom.xml file responsible for building the jgit
>> library can still live on in jgit-maven/ if that is preferable.
>>
>> I am also thinking about "mavenizing" the .pgm subproject to make it
>> easier to browse and search the code from within NetBeans.
>
> Actually, now that we have forked out of the egit.git repository,
> I want to refactor the layout of the JGit project to be more maven
> like, and have a proper top-level pom to build things.

What kind of module structure do you have in mind? Do you want to move
some of the modules/subdirectories?
Some refactoring of the maven setup for JGit back was done back in
April in sonatype's (a maven company) JGit clone. It is not
signed-off, but can serve as a reference.

 - http://github.com/sonatype/JGit/commit/641ae523c496f381a7673f4acfa0acdff9d3913e

The Maven layout in the sonatype clone simply uses the Eclipse project layout.

pom.xml: JGit :: Parent
 |- org.spearce.jgit/pom.xml: JGit :: Core
 |- org.spearce.jgit.pgm/pom.xml: JGit :: Programs
 `- org.spearce.jgit.test/pom.xml: JGit :: Test

However, having tests in a separate module can be both good/bad. For
example, they will not automatically get run when you only build the
Core module.

Anyway, I would like to help.

-- 
Jonas Fonseca

  reply	other threads:[~2009-09-03 15:39 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-02 23:28 [JGIT] Request for help Nasser Grainawi
2009-09-03  0:04 ` Johannes Schindelin
2009-09-03  1:22   ` Shawn O. Pearce
2009-09-03  6:55     ` jgit diff, was " Johannes Schindelin
2009-09-03 10:45       ` [JGIT PATCH 0/5] jgit diff Johannes Schindelin
2009-09-03 10:46         ` [JGIT PATCH 1/5] Add set to IntList Johannes Schindelin
2009-09-03 10:46         ` [JGIT PATCH 2/5] Add Myers' algorithm to generate diff scripts Johannes Schindelin
2009-09-03 10:46         ` [JGIT PATCH 3/5] Add a test class for Myers' diff algorithm Johannes Schindelin
2009-09-03 10:47         ` [JGIT PATCH 4/5] Prepare RawText for diff-index and diff-files Johannes Schindelin
2009-09-03 10:47         ` [JGIT PATCH 5/5] Add the "jgit diff" command Johannes Schindelin
2009-09-03 10:48         ` [JGIT PATCH 0/5] jgit diff Johannes Schindelin
2009-10-01 18:47         ` Shawn O. Pearce
2009-10-01 21:47           ` Johannes Schindelin
2009-09-03 15:54       ` jgit diff, was Re: [JGIT] Request for help Christian Halstrick
2009-09-03 12:45     ` Jonas Fonseca
2009-09-03 14:42       ` Shawn O. Pearce
2009-09-03 15:38         ` Jonas Fonseca [this message]
2009-09-03 15:52           ` Shawn O. Pearce
2009-09-04  5:00             ` Gabe McArthur
2009-09-04  7:33               ` Mark Struberg
2009-09-04 12:22                 ` Jonas Fonseca
2009-09-04 12:27                   ` Mark Struberg
2009-09-04 12:41                 ` Jonas Fonseca
2009-09-04 12:47                   ` Mark Struberg
2009-09-03  1:23 ` Shawn O. Pearce
2009-09-03 19:46   ` Nasser Grainawi
2009-09-03 19:49     ` Shawn O. Pearce
2009-09-03 21:09       ` Nasser Grainawi
  -- strict thread matches above, loose matches on Subject: below --
2009-09-04  7:12 Mark Struberg
     [not found] <ed88cb980909040744k2fa372fapb7ee457c745b9aa0@mail.gmail.com>
2009-09-04 14:49 ` Mark Struberg
2009-09-04 17:28   ` Mark Struberg
2009-09-04 18:50     ` Jonas Fonseca
2009-09-04 18:54       ` Mark Struberg
2009-09-04 19:51       ` Mark Struberg
2009-09-04 23:47     ` Gabe
2009-09-05  0:06       ` Douglas Campos
2009-09-05  1:29         ` Gabe McArthur
2009-09-05 16:25     ` Robin Rosenberg
2009-09-05 16:40       ` Mark Struberg

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=2c6b72b30909030838q10b88705gb8c553f2c2d74379@mail.gmail.com \
    --to=jonas.fonseca@gmail.com \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=nasser@codeaurora.org \
    --cc=spearce@spearce.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).