On Wed, Sep 2, 2015 at 4:07 PM, Alon Bar-Lev <alonbl@redhat.com> wrote:

We soon need to build entire product using xmvn, this includes tests.
So these argument is incorrect.
We should have no dependencies (especially new ones) that fedora does not provide.
Sandro, please enforce.


Not that I have super powers for enforcing that, but yes I toatlly agree with Alon.
If we can't really live without such dependency, please package it for Fedora first.
We want to be able to adhere to fedora guidelines now that we can drop el6 / mead build.

 
----- Original Message -----
> From: "Roy Golan" <rgolan@redhat.com>
> To: "Alon Bar-Lev" <alonbl@redhat.com>
> Cc: "Eyal Edri" <eedri@redhat.com>, "Roman Mohr" <rmohr@redhat.com>, "infra" <infra@ovirt.org>
> Sent: Wednesday, September 2, 2015 5:04:54 PM
> Subject: Re: Add AssertJ to artifactory
>
> its a test depency, its not needed for packaging
>
> On Wed, Sep 2, 2015 at 4:56 PM, Alon Bar-Lev <alonbl@redhat.com> wrote:
>
> > needless to say that this assertj is not available in fedora and if not
> > actually required for production use and actual important use case should
> > not be added as dependency to our project.
> >
> > ----- Original Message -----
> > > From: "Roy Golan" <rgolan@redhat.com>
> > > To: "Eyal Edri" <eedri@redhat.com>
> > > Cc: "Roman Mohr" <rmohr@redhat.com>, "infra" <infra@ovirt.org>
> > > Sent: Wednesday, September 2, 2015 4:52:20 PM
> > > Subject: Re: Add AssertJ to artifactory
> > >
> > > Thanks, its there.
> > >
> > > On Wed, Sep 2, 2015 at 3:40 PM, Eyal Edri < eedri@redhat.com > wrote:
> > >
> > >
> > >
> > > you're looking at the wrong artifactory.
> > > our mirror in in http://artifactory.ovirt.org/artifactory/ovirt-mirror .
> > >
> > > e.
> > >
> > > On Wed, Sep 2, 2015 at 1:42 PM, Roy Golan < rgolan@redhat.com > wrote:
> > >
> > >
> > >
> > > I would expect the artifactory to find it on a search query but it didn't
> > >
> > http://artifactory.eng.lab.tlv.redhat.com:8081/artifactory/webapp/search/artifact/?3&q=assertj
> > >
> > > On Wed, Sep 2, 2015 at 1:37 PM, Eyal Edri < eedri@redhat.com > wrote:
> > >
> > >
> > >
> > > adding infra.
> > > any reason why this isn't on maven central?
> > >
> > > e.
> > >
> > > On Wed, Sep 2, 2015 at 12:35 PM, Roy Golan < rgolan@redhat.com > wrote:
> > >
> > >
> > >
> > > We want to add a test jar dependency to the project and want to make sure
> > > artifactory has that.
> > >
> > > Can you add this?
> > >
> > > http://mvnrepository.com/artifact/org.assertj/assertj-core/2.1.0
> > >
> > >
> > > <dependency>
> > > <groupId>org.assertj</groupId>
> > > <artifactId>assertj-core</artifactId>
> > > <version>2.1.0</version>
> > > </dependency>
> > >
> > >
> > > Thanks
> > >
> > >
> > >
> > > --
> > > Eyal Edri
> > > Supervisor, RHEV CI
> > > EMEA ENG Virtualization R&D
> > > Red Hat Israel
> > >
> > > phone: +972-9-7692018
> > > irc: eedri (on #tlv #rhev-dev #rhev-integ)
> > >
> > >
> > >
> > >
> > > --
> > > Eyal Edri
> > > Supervisor, RHEV CI
> > > EMEA ENG Virtualization R&D
> > > Red Hat Israel
> > >
> > > phone: +972-9-7692018
> > > irc: eedri (on #tlv #rhev-dev #rhev-integ)
> > >
> > >
> > > _______________________________________________
> > > Infra mailing list
> > > Infra@ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/infra
> > >
> >
>



--
Sandro Bonazzola
Better technology. Faster innovation. Powered by community collaboration.
See how it works at redhat.com