Add AssertJ to artifactory

[Roman Mohr]

On Wed, Sep 2, 2015 at 5:21 PM, Sandro Bonazzola <sbonazzo at redhat.com>
wrote:

>
>
> On Wed, Sep 2, 2015 at 4:07 PM, Alon Bar-Lev <alonbl at 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.
>

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).
And more easy test-writing means more happy test-writing developers.

Here are the packages:
https://bugzilla.redhat.com/show_bug.cgi?id=1259852
https://bugzilla.redhat.com/show_bug.cgi?id=1259851

The second package is just a test dependency so that I can run the assertj
unit tests when building the package.

Some of you are package maintainers so I would be delighted if someone
wants to review them (they are easy reviews).
Otherwise I am sure I will find someone on the fedora mailing list for some
review exchange.



>
>

>
>> ----- Original Message -----
>> > From: "Roy Golan" <rgolan at redhat.com>
>> > To: "Alon Bar-Lev" <alonbl at redhat.com>
>> > Cc: "Eyal Edri" <eedri at redhat.com>, "Roman Mohr" <rmohr at redhat.com>,
>> "infra" <infra at 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 at 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 at redhat.com>
>> > > > To: "Eyal Edri" <eedri at redhat.com>
>> > > > Cc: "Roman Mohr" <rmohr at redhat.com>, "infra" <infra at 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 at 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 at 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 at 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 at 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 at 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
>

Have a nice weekend :)

Roman
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20150904/22c28fa8/attachment.html>