I agree that neither is correct and that this is yet another manifestation of bad Maven design.

But...

This is a POM. It is specifically a Maven artifact. In my opinion this needs to follow Maven conventions. That seems to be the way you are leaning as well given the plan you are describing below.

I'd really like to stick with the Publication DSL. It is much simpler/better. I updated my plugins to work with it. And I'd like to keep testing it for y'all. It is not clear from the documentation how one can use the pom.withXml approach to over-write values already in the POM from Gradle. I tried what seemed logical (though granted, brute force) to me:

pom.withXml {
    asNode()...

    final String runtimeScope = '<scope>runtime</scope>';
    final int replacementLength = runtimeScope.length();
    final String compileScope = '<scope>compile</scope>';
    final StringBuilder stringBuilder = asString();
    int index = stringBuilder.indexOf( runtimeScope );
    while ( index > -1 ) {
stringBuilder.replace( index, index+replacementLength, compileScope );
        index = stringBuilder.indexOf( runtimeScope );
    }
}

It works (though quite ugly imo).


On 05/05/2013 03:13 PM, Adam Murdoch wrote:

On 03/05/2013, at 11:28 PM, Steve Ebersole <st...@hibernate.org <mailto:st...@hibernate.org>> wrote:

After updating Hibernate to use the new publishing stuff we now see all dependencies in the generated POM as 'runtime' whereas previously the generated pom used 'compile'. Is that expected?

If so, I understand the logic behind using 'runtime' instead except for the fact of the footnote on Maven's own dependency primer[1[, I quote: " it is intended that this should be runtime scope instead, so that all compile dependencies must be explicitly listed - however, there is the case where the library you depend on extends a class from another library, forcing you to have available at compile time. For this reason, compile time dependencies remain as compile scope even when they are transitive."

Given that this is generating something to explicitly plug in to Maven builds, I think 'compile' should still have been the choice.

The problem is that both choices are wrong. Some of the things you compile against form part of your API, and should be made available to a consumer when they are compiling against your component. And some of the things you compile against form part of your private implementation, and should not be made available to a consumer at compile time.

Our plan is to add a mechanism that allows you to declare the dependencies of your API. These will be made available to both you and your consumers at compile time. When you're publishing to Maven, these will end up in the `compile` scope. Everything else will end up in the `runtime` scope.

There will also be a DSL on the publication object for messing with the scopes, if you don't like whatever the defaults happen to be. We've done this for the artefacts of the publication, but haven't tackled it for the scopes and dependencies yet.


[1] http://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html


On Tue 16 Apr 2013 06:11:54 PM CDT, Steve Ebersole wrote:
That plugin code on master works.  I just integrated it into Hibernate
build and updated Hibernate to use this Publication DSL and was able
to perform a publish!

In terms of designing a generic solution, not sure how much help I
will be.  Gradle has moved on dramatically from the last time I was
"in the code".  But I do think the
org.hibernate.build.gradle.publish.auth.maven.Credentials/CredentialsProvider/CredentialsProviderRegistry
stuff is pretty generic solution for managing credentials.   But I'd
not really be sure the best way to tie it in to Gradle to apply
credentials to "things" that need authentication.

Anyway, I'll be pushing (ahem, *publishing*) a 2.0.0 release of this
plugin later today that will support its feature set against the
Publication DSL.

Thanks to Luke, Daz and Adam for all the help getting me on the right
path there!


On Tue 16 Apr 2013 04:19:41 PM CDT, Adam Murdoch wrote:

On 16/04/2013, at 10:41 PM, Steve Ebersole <st...@hibernate.org <mailto:st...@hibernate.org>
<mailto:st...@hibernate.org>> wrote:

Maybe an even better question...

This upload credential plugin is really in my opinion a work around
for the fact that Gradle has no built-in support for externalized
credential declaration.  As the developer of an OSS project, it is
utterly impossible for me to put my username/password directly into
my build scripts.

While forcing each project  to define username/password project
variables and do the whole hasProperty checking stuff "works", it is
(again in my opinion) not ideal.

Absolutely. I agree with this.


So...  Is there any externalized credential management in place or
planned?  Pretty sure there is nothing in place as of now.  Is
anything of the sort planned?

It's planned. Just a matter of finding the time. As always, if someone
from the community is interested in helping out with this, we'd really
appreciate it. One place to start would be to get Steve's plugin
merged into Gradle and we can evolve it from there.


On Tue 16 Apr 2013 07:30:57 AM CDT, Steve Ebersole wrote:
Sigh.  Daz mentioned that credentials get copied from the upload repo
definitions.

Look, I just want to know what I need to do in order to apply a set of
credentials to a maven repository to be used for upload.  Is that
possible?  If so how?  Been asking that for almost 2 weeks now ;)

P.S.  I am also not seeing a way to define a split between
"production" and "snapshot" repo urls for publishing the way we used
to be able to do with uploadArchives.repositories.mavenDeployer.  Is
that also no longer supported?  If not, I guess that is something I
need to handle "manually" in the script to conditionally set the url?


On Tue 16 Apr 2013 07:25:04 AM CDT, Luke Daley wrote:

On 16/04/2013, at 1:19 PM, Steve Ebersole <st...@hibernate.org <mailto:st...@hibernate.org>
<mailto:st...@hibernate.org>> wrote:

So am I understanding that the only way to get this (applying
"publish repo" credentials) is to add credentials to a
MavenArtifactRepository?

What I am not understanding is what to do in the (seemingly normal)
use case where the publish repo is not an artifact (download) repo.

The set of consumption repositories is a different set to the
publication repositories.



On Sat 06 Apr 2013 08:47:15 PM CDT, Daz DeBoer wrote:
If you didn't find it, the DSL docs for the Publishing Extension
and
related model elements might be useful:
http://www.gradle.org/docs/nightly/dsl/org.gradle.api.publish.PublishingExtension.html.


For some classes (eg PublicationContainer) the javadoc is probably
more useful than the DSL reference (click the "API Documentation"
link
from the DSL page).

Other than that, the best way to understand what's going on is to
consult the sources.

One thing that is not well documented is the new, experimental
support
for deferred configuration that is leveraged by the
PublishingExtension. This  extension is a {@link
org.gradle.api.plugins.DeferredConfigurable} model element, meaning that extension will not be configured until it is first accessed in
the build. So any configuration blocks are not executed until
either:

1. The project is about to execute
or
2. The publishing extension is referenced as an instance, as
opposed
to a configuration closure. ie:
 publishing.publications { ... }
 publishing.repositories.maven { ... }
You can read the rationale behind deferred configuration in the
design
doc:
https://github.com/gradle/gradle/blob/master/design-docs/lazy-configuration.md.


We're interested in feedback and ideas of how we can make this
clearer.

If you were not aware of this, it's quite possible that you were
causing the publishing extension to be configured early which could
lead to hard-to-understand behaviour.

I'm happy to help get things working, do you have some code to
share?

cheers
Daz





On 5 April 2013 15:47, Steve Ebersole <st...@hibernate.org <mailto:st...@hibernate.org>
<mailto:st...@hibernate.org>
<mailto:st...@hibernate.org>> wrote:

  I realize that this is an incubating feature, but given the
  current documentation I am really not able to get this to work.
   It works if I have something simple (no pom customization, no
  snapshot/releases repository distinction, etc).  Is there some
  better documentation I shuld look at rather than

http://www.gradle.org/docs/__current/userguide/publishing___maven.html


<http://www.gradle.org/docs/current/userguide/publishing_maven.html>

  and the dsl docs for MavenPom/MavenPublication?


  On Fri 05 Apr 2013 02:50:42 PM CDT, Steve Ebersole wrote:

      Hey Daz,

      The "upload auth" plugin[1] reads credentials from maven
      settings.xml
      etc and applies them to any same-named repositories
referenced
      in a
      Gradle build through an Upload task.  So, as I understand
it, that
      would not apply here.

      Applying the credentials to the resolution repos would be a
      good thing
      too.

      [1] https://github.com/sebersole/__gradle-upload-auth-plugin
      <https://github.com/sebersole/gradle-upload-auth-plugin>


      On Fri 05 Apr 2013 02:21:07 PM CDT, Daz DeBoer wrote:

          On 5 April 2013 07:16, Steve Ebersole
<st...@hibernate.org <mailto:st...@hibernate.org>
          <mailto:st...@hibernate.org>
          <mailto:st...@hibernate.org
<mailto:st...@hibernate.org>>>
          wrote:

              I am just upgrading Hibernate to Gradle 1.5 and am
          reading about
              the new publications stuff.  I really like the
new DSL
          much, much
              better.

              I did have one question though that was not
addressed
          in the docs.
              For the old style of "uploading" I had developed a
          plugin that
              provided authorization based on users local Maven
          install.  From
              my recollection the intent in the new maven
          publication code was
              to provide this behavior out-of-the-box.  So I am
          curious if that
              code ever made it into the
              MavenArtifactRepository/____MavenPublication
code.  Or
          do I need to
              update my gradle-upload-auth-plugin to handle
this new
          API?


          The new Publication support copies the credentials from
the
          MavenArtifactRepository (see the poorly named

org.gradle.api.publish.maven.__internal.publisher.__MavenDeployerConfigurer).



          So any plugin code that you use to add credentials to
maven
          repositories used for resolution should also work for
          publication.
          --
          Darrell (Daz) DeBoer
          Principal Engineer, Gradleware
http://www.gradleware.com <http://www.gradleware.com/>
          Join us at the Gradle Summit 2013, June 13th and 14th in
          Santa Clara,
          CA: http://www.gradlesummit.com
<http://www.gradlesummit.com/>




--
Darrell (Daz) DeBoer
Principal Engineer, Gradleware
http://www.gradleware.com <http://www.gradleware.com/>
Join us at the Gradle Summit 2013, June 13th and 14th in Santa
Clara,
CA: http://www.gradlesummit.com <http://www.gradlesummit.com/>

---------------------------------------------------------------------

To unsubscribe from this list, please visit:

http://xircles.codehaus.org/manage_email




---------------------------------------------------------------------
To unsubscribe from this list, please visit:

http://xircles.codehaus.org/manage_email




--
Adam Murdoch
Gradle Co-founder
http://www.gradle.org
VP of Engineering, Gradleware Inc. - Gradle Training, Support,
Consulting
http://www.gradleware.com

Join us at the Gradle Summit 2013, June 13th and 14th in Santa Clara,
CA: http://www.gradlesummit.com



--
Adam Murdoch
Gradle Co-founder
http://www.gradle.org
VP of Engineering, Gradleware Inc. - Gradle Training, Support, Consulting
http://www.gradleware.com

Join us at the Gradle Summit 2013, June 13th and 14th in Santa Clara, CA: http://www.gradlesummit.com


Reply via email to