On Wed, Jul 7, 2010 at 9:57 PM, Pete Muir <pmuir(a)redhat.com> wrote:
On 7 Jul 2010, at 18:52, Dan Allen wrote:
> Thanks for the feedback.
>
> On Wed, Jul 7, 2010 at 8:17 PM, Pete Muir <pmuir(a)redhat.com> wrote:
>
> On 6 Jul 2010, at 13:31, Dan Allen wrote:
>
> > Stuart also suggested renaming modules to trunk, or as I suggested
making a SVN-style symlink using svn:externals. The purpose would be to make
it more obvious to people what to checkout.
> >
> > /trunk -> /modules
>
> No. This is completely counterintuitive and breaks all conventions around
SVN.
>
> Okay. I agree.
>
> > I'd like to nuke the following directories from the Seam repo, which I
don't believe are currently being used.
> >
> >
http://anonsvn.jboss.org/repos/seam/trunk/ (each component has it's
own trunk in Seam 3, so this trunk is erroneous; all Seam 2 stuff is under a
branch)
> >
http://anonsvn.jboss.org/repos/seam/seam-gen/ (the real encore is in
/sandbox)
>
> Agreed.
>
> Cool.
Done.
http://anonsvn.jboss.org/repos/seam/about.txt
>
>
> >
> > I would like to consolidate /build and /dist under a single folder.
They are both build/packaging related and I think they should be seen as a
single top-level unit. They are this way in Weld.
>
> No. These need to remain separate, they have different release cycles and
thus different trunk, branch, tag.
>
> I'm assuming that's because the idea of a dist in Seam 3 is much
different than that of Weld. So the dist needs to be a separate entity. I'll
include your points in the readme.
It is - I actually screed this up in Weld a bit.
The idea behind a dist is that it represents the aggregation of released
modules, whilst build is something that each module depends on.
Looks like I never wrote this stuff up. It's back on the todo list.
Feel free to update/revise the about.txt file.
-Dan
--
Dan Allen
Senior Software Engineer, Red Hat | Author of Seam in Action
Registered Linux User #231597
http://mojavelinux.com
http://mojavelinux.com/seaminaction
http://www.google.com/profiles/dan.j.allen