are there seconders for
http://git-wip-us.apache.org/repos/asf/maven-resolver/commit/17f804d7
(aka "option 2")?

Regards,

Hervé

Le samedi 27 mai 2017, 19:05:27 CEST Hervé BOUTEMY a écrit :
> good links
> yes, with this in mind, "api" is required for artifactId but should not be
> added to module name: good catch, and good experience to share because that
> was not so obvious
> 
> Regards,
> 
> Hervé
> 
> Le samedi 27 mai 2017, 18:43:22 CEST Robert Scholte a écrit :
> > There's no experience with this yet.
> > 
> > Stephen Colebourne has written to related blogs: module naming[1] and
> > modules are not artifacts[2]
> > which might suggest that "api" should not be added.
> > I do understand the addition of "api". And to make it worse, this is
> > probably the most important artifact needing a module name. In most cases
> > developers will only need this one: frameworks will handle the
> > implementation part. :)
> > 
> > Robert
> > 
> > [1] http://blog.joda.org/2017/04/java-se-9-jpms-module-naming.html
> > [2]
> > http://blog.joda.org/2017/04/java-se-9-jpms-modules-are-not-artifacts.html
> > 
> > 
> > On Sat, 27 May 2017 17:48:24 +0200, Hervé BOUTEMY <herve.bout...@free.fr>
> > 
> > wrote:
> > > second option committed in another branch:
> > > 
> > > option 1:
> > > http://git-wip-us.apache.org/repos/asf/maven-resolver/commit/d1724eb7
> > > 
> > > option 2:
> > > http://git-wip-us.apache.org/repos/asf/maven-resolver/commit/17f804d7
> > > 
> > > The only part that I'm not sure in option 2 is
> > > org.apache.maven.resolver.api >
> > > org.apache.maven.resolver: is it better to be explicit on the api or
> > > implicit?
> > > 
> > > Regards,
> > > 
> > > Hervé
> > > 
> > > Le samedi 27 mai 2017 17:37:03 CEST, vous avez écrit :
> > >> I think I would change the following 2:
> > >> 
> > >> org.apache.maven.resolver.connector_basic >
> > >> org.apache.maven.resolver.connector.basic (in line with transport)
> > >> org.apache.maven.resolver.test_util >
> > >> org.apache.maven.resolver.testutil
> > >> 
> > >> it's a matter of taste: the underscores look kind of weird, but that's
> > >> probably caused because we've never used them as package names.
> > >> 
> > >> And wondering if "api" should be changed, since this is the
> > >> access-module
> > >> and we don't use api in our packages:
> > >> org.apache.maven.resolver.api > org.apache.maven.resolver
> > >> 
> > >> Using a property makes it easier to configure the maven-jar-plugin, but
> > >> it
> > >> also makes these constants turn into variables, i.e. you could set them
> > >> via system properties or cmdline args.
> > >> If only we supported something like
> > >> <Automatic-Module-Name>${project.properties["AutomaticModuleName"]}</Au
> > >> to
> > >> mat ic-Module-Name>
> > >> 
> > >> for the rest it's looking good.
> > >> 
> > >> thanks
> > >> Robert
> > >> 
> > >> 
> > >> On Sat, 27 May 2017 17:20:15 +0200, Hervé BOUTEMY
> > >> <herve.bout...@free.fr>
> > >> 
> > >> wrote:
> > >> > please review and second if you think it's ok:
> > >> > http://git-wip-us.apache.org/repos/asf/maven-resolver/commit/d1724eb7
> > >> > 
> > >> > Regards,
> > >> > 
> > >> > Hervé
> > >> > 
> > >> > Le samedi 27 mai 2017, 13:24:47 CEST Hervé BOUTEMY a écrit :
> > >> >> he he, Java 9 is really coming, with associated real world
> > >> >> questions.
> > >> >> 
> > >> >> Maven Artifact Resolver is one of rare Maven components that has a
> > >> >> chance to
> > >> >> become a collection Java 9 modules, since it was written quite
> > >> 
> > >> recently
> > >> 
> > >> >> and
> > >> >> is pure new code as a result of Maven 3 refactoring, then does not
> > >> 
> > >> have
> > >> 
> > >> >> shared package names issues we have with Maven core itself.
> > >> >> 
> > >> >> And since it is expected to be a lib for easy embedding of artifact
> > >> >> resolution, making it a collection of Java 9 modules would be not
> > >> 
> > >> only a
> > >> 
> > >> >> great opportunity to test Java 9 modules, but it could be useful for
> > >> >> people
> > >> >> using it.
> > >> >> 
> > >> >> Then I'm highly in favor of trying.
> > >> >> 
> > >> >> Adding Automatic-Module-Name to the MANIFEST.MF looks feasible right
> > >> >> now,
> > >> >> without waiting much: I'm pretty sure module names will be obvious,
> > >> 
> > >> and
> > >> 
> > >> >> much
> > >> >> better if we define them instead of waiting for automatic names.
> > >> 
> > >> Let's
> > >> 
> > >> >> start! MRESOLVER-26 created [1]
> > >> >> 
> > >> >> Then there is the question of making real modules: is it feasible
> > >> 
> > >> right
> > >> 
> > >> >> now?
> > >> >> Or would we need a delay to tweak the module descriptors? And that
> > >> 
> > >> would
> > >> 
> > >> >> mean that we need Java 9 to build, even if the generated bytecode
> > >> >> remains
> > >> >> Java 7 compatible, isn't it? Is Maven tooling ready to it?
> > >> >> MRESOLVER-27 created to track the issue [2], but I'm not sure this
> > >> >> is
> > >> >> the
> > >> >> right time to do this job, but for the next release after this 1.1.0
> > >> >> 
> > >> >> Regards,
> > >> >> 
> > >> >> Hervé
> > >> >> 
> > >> >> [1] https://issues.apache.org/jira/browse/MRESOLVER-26
> > >> >> 
> > >> >> [2] https://issues.apache.org/jira/browse/MRESOLVER-27
> > >> >> 
> > >> >> Le samedi 27 mai 2017, 11:58:43 CEST Robert Scholte a écrit :
> > >> >> > Hi,
> > >> >> > 
> > >> >> > I've got a question from Remi Forax if we could add Java9 module
> > >> >> > descriptors to this project.
> > >> >> > This will be one of the first which can provide such descriptors
> > >> >> 
> > >> >> since it
> > >> >> 
> > >> >> > has no required dependencies other then its own and its package
> > >> >> 
> > >> >> structure
> > >> >> 
> > >> >> > seems valid with the new Java9 rules.
> > >> >> > 
> > >> >> > We haven't discussed this in general yet, but we have several
> > >> 
> > >> projects
> > >> 
> > >> >> > which are at the bottom of the dependency tree which should
> > >> >> > provide
> > >> >> 
> > >> >> either
> > >> >> 
> > >> >> > a module name or module descriptor when possible.
> > >> >> > 
> > >> >> > Do we want to help the community by having already several
> > >> 
> > >> libraries
> > >> 
> > >> >> with
> > >> >> 
> > >> >> > a module descriptor?
> > >> >> > 
> > >> >> > Or we could add a Automatic-Module-Name to the MANIFEST.MF, so
> > >> 
> > >> others
> > >> 
> > >> >> can
> > >> >> 
> > >> >> > refer to it by its official module name and we can add the
> > >> 
> > >> descriptor
> > >> 
> > >> >> once
> > >> >> 
> > >> >> > Java9 has officially been released. (pro: doesn't require Java 9
> > >> :
> > >> :) )
> > >> :
> > >> >> > Or do nothing yet...
> > >> >> > 
> > >> >> > thanks,
> > >> >> > Robert
> > >> >> > 
> > >> >> > On Sat, 27 May 2017 11:42:32 +0200, Hervé BOUTEMY
> > >> >> 
> > >> >> <herve.bout...@free.fr>
> > >> >> 
> > >> >> > wrote:
> > >> >> > > Hi,
> > >> >> > > 
> > >> >> > > No objection from me, thanks for keeping the ball rolling.
> > >> >> > > 
> > >> >> > > I tried to improve documentation by adding some useful links to
> > >> >> 
> > >> >> other
> > >> >> 
> > >> >> > > related
> > >> >> > > components [1]: I think the current state is better and ok for a
> > >> >> > > release.
> > >> >> > > 
> > >> >> > > One key question now is about Aether wiki content [2]: should we
> > >> >> 
> > >> >> copy
> > >> >> 
> > >> >> > > it? In a
> > >> >> > > wiki or in components sources?
> > >> >> > > I suppose wiki source format is supported by Doxia, then it
> > >> 
> > >> could be
> > >> 
> > >> >> > > imported
> > >> >> > > quite easily in sources.
> > >> >> > > 
> > >> >> > > And of course, there is the final question: should we do it
> > >> 
> > >> before
> > >> 
> > >> >> the
> > >> >> 
> > >> >> > > release?
> > >> >> > > 
> > >> >> > > Regards,
> > >> >> > > 
> > >> >> > > Hervé
> > >> >> > > 
> > >> >> > > [1] http://maven.apache.org/resolver-archives/resolver-LATEST/
> > >> >> > > 
> > >> >> > > [2] http://wiki.eclipse.org/Aether
> > >> >> > > 
> > >> >> > > Le vendredi 26 mai 2017, 16:18:02 CEST Michael Osipov a écrit :
> > >> >> > >> Hi folks,
> > >> >> > >> 
> > >> >> > >> is there anything holding us back from MRESOLVER 1.1.0?
> > >> >> > >> I'd like to start the release by the end of the week and have
> > >> >> > >> it
> > >> >> > >> integrated into Maven 3.5.1.
> > >> >> > >> 
> > >> >> > >> Any objections?
> > >> >> > >> 
> > >> >> > >> Michael
> > >> >> 
> > >> >> --------------------------------------------------------------------
> > >> >> -
> > >> >> 
> > >> >> > >> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > >> >> > >> For additional commands, e-mail: dev-h...@maven.apache.org
> > >> >> 
> > >> >> --------------------------------------------------------------------
> > >> >> -
> > >> >> 
> > >> >> > > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > >> >> > > For additional commands, e-mail: dev-h...@maven.apache.org
> > >> 
> > >> ---------------------------------------------------------------------
> > >> 
> > >> >> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > >> >> > For additional commands, e-mail: dev-h...@maven.apache.org
> > >> >> 
> > >> >> --------------------------------------------------------------------
> > >> >> -
> > >> >> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > >> >> For additional commands, e-mail: dev-h...@maven.apache.org
> > >> > 
> > >> > ---------------------------------------------------------------------
> > >> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > >> > For additional commands, e-mail: dev-h...@maven.apache.org
> > > 
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > > For additional commands, e-mail: dev-h...@maven.apache.org
> > 
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> > For additional commands, e-mail: dev-h...@maven.apache.org
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Reply via email to