zip command is missing on EL6 host

[Alon Bar-Lev]

----- Original Message -----
> From: "Alon Bar-Lev" <alonbl at redhat.com>
> To: "Martin Perina" <mperina at redhat.com>
> Cc: "Eyal Edri" <eedri at redhat.com>, infra at ovirt.org
> Sent: Thursday, May 7, 2015 2:45:38 PM
> Subject: Re: zip command is missing on EL6 host
> 
> 
> 
> ----- Original Message -----
> > From: "Martin Perina" <mperina at redhat.com>
> > To: "Eyal Edri" <eedri at redhat.com>
> > Cc: infra at ovirt.org
> > Sent: Thursday, May 7, 2015 2:44:37 PM
> > Subject: Re: zip command is missing on EL6 host
> > 
> > 
> > 
> > ----- Original Message -----
> > > From: "Eyal Edri" <eedri at redhat.com>
> > > To: "Martin Perina" <mperina at redhat.com>
> > > Cc: infra at ovirt.org
> > > Sent: Thursday, May 7, 2015 12:40:04 PM
> > > Subject: Re: zip command is missing on EL6 host
> > > 
> > > Hi Martin,
> > > 
> > > Unfortunately infra teams has been handling multiple urgent issues in
> > > these
> > > past days,
> > > and this wasn't picked up yet.
> > 
> > No problem, I just didn't know if somebody read it already or not :-)
> > 
> > > 
> > > how come the job needs zip if it uses mock? maybe you're missing an el6
> > > repo
> > > with it
> > > for the job to work?
> > 
> > Because the zip is needed before executing the build of SRPMS archive. We
> > need
> > to upgrade hibernate-validator inside original WildFly 8.2 distribution. I
> > didn't
> > want to move this logic into spec file, because there's high probability
> > that
> > WildFly 9.0 will contain the fix, so it seems easier for me just to remove
> > few
> > lines from build.sh than change logic in spec file.
> 
> please move your logic into the spec file, it belongs there.

well, this is in principal, while in this specific solution you should probably use module overlay if possible and not modify base package.

> 
> > 
> > Thanks
> > 
> > Martin
> > 
> > > 
> > > e.
> > > 
> > > 
> > > ----- Original Message -----
> > > > From: "Martin Perina" <mperina at redhat.com>
> > > > To: infra at ovirt.org
> > > > Sent: Thursday, May 7, 2015 12:37:50 PM
> > > > Subject: Re: zip command is missing on EL6 host
> > > > 
> > > > Hi,
> > > > 
> > > > any progress with the issue?
> > > > 
> > > > Thanks
> > > > 
> > > > Martin
> > > > 
> > > > ----- Original Message -----
> > > > > From: "Martin Perina" <mperina at redhat.com>
> > > > > To: infra at ovirt.org
> > > > > Sent: Wednesday, May 6, 2015 11:29:36 AM
> > > > > Subject: zip command is missing on EL6 host
> > > > > 
> > > > > Hi,
> > > > > 
> > > > > I submitted a patch to create WildFly RPM with upgraded
> > > > > hibernate-validator
> > > > > [1].
> > > > > Unfortunately build of this patch is failing on EL6 host [2], because
> > > > > zip
> > > > > command
> > > > > is missing. The zip command is needed during execution of build.sh
> > > > > command
> > > > > (it's
> > > > > not needed during SRPM creation by rpmbui;d, that's why it's not
> > > > > included
> > > > > in
> > > > > BuildRequires).
> > > > > 
> > > > > The build finished successfully on EL7 host [3], because zip is
> > > > > present
> > > > > on
> > > > > it.
> > > > > 
> > > > > How can this be fixed?
> > > > > 
> > > > > Thanks
> > > > > 
> > > > > Martin
> > > > > 
> > > > > 
> > > > > [1] https://gerrit.ovirt.org/40574
> > > > > [2]
> > > > > http://jenkins.ovirt.org/job/ovirt-engine-wildfly_master_create-rpms-el6-x86_64_created/2/
> > > > > [3]
> > > > > http://jenkins.ovirt.org/job/ovirt-engine-wildfly_master_create-rpms-el7-x86_64_created/2/
> > > > _______________________________________________
> > > > Infra mailing list
> > > > Infra at ovirt.org
> > > > http://lists.ovirt.org/mailman/listinfo/infra
> > > > 
> > > > 
> > > > 
> > > 
> > _______________________________________________
> > Infra mailing list
> > Infra at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/infra
> > 
>