<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 2, 2015 at 4:07 PM, Alon Bar-Lev <span dir="ltr"><<a href="mailto:alonbl@redhat.com" target="_blank">alonbl@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
We soon need to build entire product using xmvn, this includes tests.<br>
So these argument is incorrect.<br>
We should have no dependencies (especially new ones) that fedora does not provide.<br>
Sandro, please enforce.<br>
<span class="im HOEnZb"><br></span></blockquote><div><br></div><div>Not that I have super powers for enforcing that, but yes I toatlly agree with Alon.</div><div>If we can't really live without such dependency, please package it for Fedora first.</div><div>We want to be able to adhere to fedora guidelines now that we can drop el6 / mead build.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="im HOEnZb">
----- Original Message -----<br>
> From: "Roy Golan" <<a href="mailto:rgolan@redhat.com">rgolan@redhat.com</a>><br>
</span><div class="HOEnZb"><div class="h5">> To: "Alon Bar-Lev" <<a href="mailto:alonbl@redhat.com">alonbl@redhat.com</a>><br>
> Cc: "Eyal Edri" <<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>>, "Roman Mohr" <<a href="mailto:rmohr@redhat.com">rmohr@redhat.com</a>>, "infra" <<a href="mailto:infra@ovirt.org">infra@ovirt.org</a>><br>
> Sent: Wednesday, September 2, 2015 5:04:54 PM<br>
> Subject: Re: Add AssertJ to artifactory<br>
><br>
> its a test depency, its not needed for packaging<br>
><br>
> On Wed, Sep 2, 2015 at 4:56 PM, Alon Bar-Lev <<a href="mailto:alonbl@redhat.com">alonbl@redhat.com</a>> wrote:<br>
><br>
> > needless to say that this assertj is not available in fedora and if not<br>
> > actually required for production use and actual important use case should<br>
> > not be added as dependency to our project.<br>
> ><br>
> > ----- Original Message -----<br>
> > > From: "Roy Golan" <<a href="mailto:rgolan@redhat.com">rgolan@redhat.com</a>><br>
> > > To: "Eyal Edri" <<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>><br>
> > > Cc: "Roman Mohr" <<a href="mailto:rmohr@redhat.com">rmohr@redhat.com</a>>, "infra" <<a href="mailto:infra@ovirt.org">infra@ovirt.org</a>><br>
> > > Sent: Wednesday, September 2, 2015 4:52:20 PM<br>
> > > Subject: Re: Add AssertJ to artifactory<br>
> > ><br>
> > > Thanks, its there.<br>
> > ><br>
> > > On Wed, Sep 2, 2015 at 3:40 PM, Eyal Edri < <a href="mailto:eedri@redhat.com">eedri@redhat.com</a> > wrote:<br>
> > ><br>
> > ><br>
> > ><br>
> > > you're looking at the wrong artifactory.<br>
> > > our mirror in in <a href="http://artifactory.ovirt.org/artifactory/ovirt-mirror" rel="noreferrer" target="_blank">http://artifactory.ovirt.org/artifactory/ovirt-mirror</a> .<br>
> > ><br>
> > > e.<br>
> > ><br>
> > > On Wed, Sep 2, 2015 at 1:42 PM, Roy Golan < <a href="mailto:rgolan@redhat.com">rgolan@redhat.com</a> > wrote:<br>
> > ><br>
> > ><br>
> > ><br>
> > > I would expect the artifactory to find it on a search query but it didn't<br>
> > ><br>
> > <a href="http://artifactory.eng.lab.tlv.redhat.com:8081/artifactory/webapp/search/artifact/?3&q=assertj" rel="noreferrer" target="_blank">http://artifactory.eng.lab.tlv.redhat.com:8081/artifactory/webapp/search/artifact/?3&q=assertj</a><br>
> > ><br>
> > > On Wed, Sep 2, 2015 at 1:37 PM, Eyal Edri < <a href="mailto:eedri@redhat.com">eedri@redhat.com</a> > wrote:<br>
> > ><br>
> > ><br>
> > ><br>
> > > adding infra.<br>
> > > any reason why this isn't on maven central?<br>
> > ><br>
> > > e.<br>
> > ><br>
> > > On Wed, Sep 2, 2015 at 12:35 PM, Roy Golan < <a href="mailto:rgolan@redhat.com">rgolan@redhat.com</a> > wrote:<br>
> > ><br>
> > ><br>
> > ><br>
> > > We want to add a test jar dependency to the project and want to make sure<br>
> > > artifactory has that.<br>
> > ><br>
> > > Can you add this?<br>
> > ><br>
> > > <a href="http://mvnrepository.com/artifact/org.assertj/assertj-core/2.1.0" rel="noreferrer" target="_blank">http://mvnrepository.com/artifact/org.assertj/assertj-core/2.1.0</a><br>
> > ><br>
> > ><br>
> > > <dependency><br>
> > > <groupId>org.assertj</groupId><br>
> > > <artifactId>assertj-core</artifactId><br>
> > > <version>2.1.0</version><br>
> > > </dependency><br>
> > ><br>
> > ><br>
> > > Thanks<br>
> > ><br>
> > ><br>
> > ><br>
> > > --<br>
> > > Eyal Edri<br>
> > > Supervisor, RHEV CI<br>
> > > EMEA ENG Virtualization R&D<br>
> > > Red Hat Israel<br>
> > ><br>
> > > phone: <a href="tel:%2B972-9-7692018" value="+97297692018">+972-9-7692018</a><br>
> > > irc: eedri (on #tlv #rhev-dev #rhev-integ)<br>
> > ><br>
> > ><br>
> > ><br>
> > ><br>
> > > --<br>
> > > Eyal Edri<br>
> > > Supervisor, RHEV CI<br>
> > > EMEA ENG Virtualization R&D<br>
> > > Red Hat Israel<br>
> > ><br>
> > > phone: <a href="tel:%2B972-9-7692018" value="+97297692018">+972-9-7692018</a><br>
> > > irc: eedri (on #tlv #rhev-dev #rhev-integ)<br>
> > ><br>
> > ><br>
> > > _______________________________________________<br>
> > > Infra mailing list<br>
> > > <a href="mailto:Infra@ovirt.org">Infra@ovirt.org</a><br>
> > > <a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/infra</a><br>
> > ><br>
> ><br>
><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Sandro Bonazzola<br>Better technology. Faster innovation. Powered by community collaboration.<br>See how it works at <a href="http://redhat.com" target="_blank">redhat.com</a><br></div></div></div></div>
</div></div>