Re: [maven-jenkins-lib] branch master updated (50410a9 -> bb8b8c8)

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: [maven-jenkins-lib] branch master updated (50410a9 -> bb8b8c8)

olamy
I have disabled it as there were some errors (not sure what) with using the
configfile provider plugin.
But yup good catch we definitely need to do it only for one jdk...
not sure how to do the deploy at end as it was done previously by the maven
plugin. (probably someone need to write a plugin for this :-) )

On Sat, 7 Apr 2018 at 22:09, Robert Scholte <[hidden email]> wrote:

> Hi Olivier,
>
> doesn't this mean that we get about 8 deployments (2 oses times 4 jdks) of
> the same artifacts?
> This is a potential problem for the metadata files.
> Is it possible to do it really at the end by Jenkins?
>
> thanks,
> Robert
>
> On Sat, 07 Apr 2018 13:19:23 +0200, <[hidden email]> wrote:
>
> > This is an automated email from the ASF dual-hosted git repository.
> >
> > olamy pushed a change to branch master
> > in repository https://gitbox.apache.org/repos/asf/maven-jenkins-lib.git.
> >
> >
> >     from 50410a9  Add JDK 10 to the jdk default
> >      add 96afe26  auto deploy if we are currently on master branch
> >      new bb8b8c8  Merge pull request #1 from olamy/master
> >
> > The 1 revisions listed above as "new" are entirely new to this
> > repository and will be described in separate emails.  The revisions
> > listed as "add" were already present in the repository and have only
> > been added to this reference.
> >
> >
> > Summary of changes:
> >  vars/asfMavenTlpStdBuild.groovy | 30 +++++++++++++++++++-----------
> >  1 file changed, 19 insertions(+), 11 deletions(-)
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>

--
Olivier Lamy
http://twitter.com/olamy | http://linkedin.com/in/olamy
Reply | Threaded
Open this post in threaded view
|

Re: [maven-jenkins-lib] branch master updated (50410a9 -> bb8b8c8)

olamy
On Sun, 8 Apr 2018 at 19:31, Karl Heinz Marbaise <[hidden email]> wrote:

> Hi,
>
> On 08/04/18 02:04, Olivier Lamy wrote:
> > I have disabled it as there were some errors (not sure what) with using
> the
> > configfile provider plugin.
> > But yup good catch we definitely need to do it only for one jdk...
>
> would be better...
>
> > not sure how to do the deploy at end as it was done previously by the
> maven
> > plugin. (probably someone need to write a plugin for this :-) )
>
> did I miss something ? What do you mean by this?
>

ideally I'd like to get the produced/attached artifacts of a maven build
and deploy them.
But I'm not sure if with withMaven we can know which artifacts has been
produced by a jdk....

So looks better to activate deploy goal only jdk=jdk7.
I don't like the idea about being dependant of a label (and what about the
future when some part will be build with only jdk8+)
if any better idea?


>
> Kind regards
> Karl  Heinz Marbaise
>
> >
> > On Sat, 7 Apr 2018 at 22:09, Robert Scholte <[hidden email]>
> wrote:
> >
> >> Hi Olivier,
> >>
> >> doesn't this mean that we get about 8 deployments (2 oses times 4 jdks)
> of
> >> the same artifacts?
> >> This is a potential problem for the metadata files.
> >> Is it possible to do it really at the end by Jenkins?
> >>
> >> thanks,
> >> Robert
> >>
> >> On Sat, 07 Apr 2018 13:19:23 +0200, <[hidden email]> wrote:
> >>
> >>> This is an automated email from the ASF dual-hosted git repository.
> >>>
> >>> olamy pushed a change to branch master
> >>> in repository
> https://gitbox.apache.org/repos/asf/maven-jenkins-lib.git.
> >>>
> >>>
> >>>      from 50410a9  Add JDK 10 to the jdk default
> >>>       add 96afe26  auto deploy if we are currently on master branch
> >>>       new bb8b8c8  Merge pull request #1 from olamy/master
> >>>
> >>> The 1 revisions listed above as "new" are entirely new to this
> >>> repository and will be described in separate emails.  The revisions
> >>> listed as "add" were already present in the repository and have only
> >>> been added to this reference.
> >>>
> >>>
> >>> Summary of changes:
> >>>   vars/asfMavenTlpStdBuild.groovy | 30 +++++++++++++++++++-----------
> >>>   1 file changed, 19 insertions(+), 11 deletions(-)
>


--
Olivier Lamy
http://twitter.com/olamy | http://linkedin.com/in/olamy