On 6/4/07, Brett Porter <[EMAIL PROTECTED]> wrote:
On 05/06/2007, at 11:01 AM, Carlos Sanchez wrote:
> I noticed the issue with duplicated packages while playing with OSGi
> but is not directly related.
> The fact that we have same packages in different modules is just a bad
> practice, for architectural and easier development issues. If I see an
> org.apache.maven.project class I'd look into maven-project without
> having to search all the sources
Thinking purely from a packaging perspective, I agree with the
principle but not the implementation. I didn't like the look of a
'core' package and was planning on commenting myself. (I only had a
brief look at the commits didn't spend any more time with it after it
was reverted until the discussion came up).
I'm open to suggestions for package names for maven-core, but the
other ones are pretty obvious
I think we need to take a more critical look at how the packages are
structured to make sure each module is properly self-contained and
has a well-defined public API. I think this should this be an
architectural goal for 2.1.
ok so what are we going to do to keep backwards compatibility without
adding complexity to patching
- just make new classes in the new packages and old ones extend them
- merging that to the 2.0.x branch too to reduce differences between them
- make the new classes extend the old ones so all the code is in the old ones
I think it's great that we have things like the embedder which can be
a simple, single point of entry for someone that uses all of Maven,
but I don't ever want to lose the ability to consume discreet pieces
of functionality like the artifact mechanism.
we agree
- Brett
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
--
I could give you my word as a Spaniard.
No good. I've known too many Spaniards.
-- The Princess Bride
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
