<div dir="ltr"><div><div>assertj is now in rawhide and updates for f22 and f23 are pending: <a href="https://bodhi.fedoraproject.org/updates/?packages=assertj-core">https://bodhi.fedoraproject.org/updates/?packages=assertj-core</a> :)<br><br></div>Have a nice evening,<br></div>Roman<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Sep 4, 2015 at 11:40 AM, Sandro Bonazzola <span dir="ltr"><<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Fri, Sep 4, 2015 at 11:28 AM, Roman Mohr <span dir="ltr"><<a href="mailto:rmohr@redhat.com" target="_blank">rmohr@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span>On Wed, Sep 2, 2015 at 5:21 PM, Sandro Bonazzola <span dir="ltr"><<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);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><br></span></blockquote><div><br></div></span><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.<br></div></div></div></div></blockquote><div><br></div></span><div>Well of course we can live without that dependency but it will greatly improve the way of writing meaningful assertions (having hamcrest is better than not having it but it is a pain to use and read its assertions).<br>And more easy test-writing means more happy test-writing developers.<br><br></div><div>Here are the packages:<br><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1259852" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1259852</a><br><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1259851" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1259851</a><br><br></div></div></div></div></blockquote><div><br></div></span><div>Requested Fedora Java SIG to review.</div><div><div class="h5"><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>The second package is just a test dependency so that I can run the assertj unit tests when building the package.<br><br></div><div>Some of you are package maintainers so I would be delighted if someone wants to review them (they are easy reviews).<br></div><div>Otherwise I am sure I will find someone on the fedora mailing list for some review exchange.<br></div><div><div><div><br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br> </div></div></div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><div><div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span>
----- Original Message -----<br>
> From: "Roy Golan" <<a href="mailto:rgolan@redhat.com" target="_blank">rgolan@redhat.com</a>><br>
</span><div><div>> To: "Alon Bar-Lev" <<a href="mailto:alonbl@redhat.com" target="_blank">alonbl@redhat.com</a>><br>
> Cc: "Eyal Edri" <<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>>, "Roman Mohr" <<a href="mailto:rmohr@redhat.com" target="_blank">rmohr@redhat.com</a>>, "infra" <<a href="mailto:infra@ovirt.org" target="_blank">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" target="_blank">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" target="_blank">rgolan@redhat.com</a>><br>
> > > To: "Eyal Edri" <<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>><br>
> > > Cc: "Roman Mohr" <<a href="mailto:rmohr@redhat.com" target="_blank">rmohr@redhat.com</a>>, "infra" <<a href="mailto:infra@ovirt.org" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">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" target="_blank">+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" target="_blank">+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" target="_blank">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></div></div><span><font color="#888888"><br><br clear="all"><div><br></div>-- <br><div><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>
</font></span></div></div>
</blockquote></div></div></div><br></div><div class="gmail_extra">Have a nice weekend :)<span><font color="#888888"><br><br></font></span></div><span><font color="#888888"><div class="gmail_extra">Roman<br></div></font></span></div>
</blockquote></div></div></div><div><div class="h5"><br><br clear="all"><div><br></div>-- <br><div><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></div></div>
</blockquote></div><br></div>