Hi Yusuke,

On Thu, Feb 21, 2013 at 09:04:45PM +0900, yusuke iida wrote:
> Hi, Dejan
> 
> I also tested by rhel6.3 and fedora17.
> Since there is no environment, centos is not tested.
> 
> The point worried below is shown:
> - I think that %{?fedora_version} and %{?rhel_version} are macro not to exist.

Those macros work in OBS when rhel6 packages are built. I wonder
if that's some build service extension.

> - pssh is not provided in rhel.
>   I think that you should not put it in Requires.

OK, but currently the only RPM built is the one in OBS where the
repository includes pssh RPMs for rhel/centos too. See for
instance:

http://download.opensuse.org/repositories/network:/ha-clustering/RedHat_RHEL-6/x86_64/

> I made a patch to solve the above.

Note that the .spec file in the upstream may not be perfect or
even work on particular distribution. However, it should advise
packagers on what it should contain. The pssh requirement is
there because history would not work well without it. It is
further rather unfortunate that that feature is used very seldom
and that it got so little attention.

Therefore, I'm reluctant to apply the pssh part of the patch.

Cheers,

Dejan

> Regards,
> Yusuke
> 
> 2013/2/19 Dejan Muhamedagic <de...@suse.de>:
> > On Tue, Feb 19, 2013 at 11:03:53AM +0100, Dejan Muhamedagic wrote:
> >> On Fri, Feb 15, 2013 at 10:19:41PM +0100, Dejan Muhamedagic wrote:
> >> > Hi,
> >> >
> >> > On Fri, Feb 15, 2013 at 02:25:41PM +0900, yusuke iida wrote:
> >> > > Hi, Dejan
> >> > >
> >> > > I made a patch for spec file to make rpm of crmsh in rhel environment.
> >> > > I want a crmsh repository to merge it if I do not have any problem.
> >>
> >> This is a problem which I ran into earlier too. Something
> >> (probably one of the rpm macros) does a 'rm -rf' of the doc
> >> directory _after_ the files got installed:
> >>
> >> [   29s] test -z "/usr/share/doc/packages/crmsh" || /bin/mkdir -p 
> >> "/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh"
> >> [   29s]  /usr/bin/install -c -m 644 'AUTHORS' 
> >> '/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh/AUTHORS'
> >> [   29s]  /usr/bin/install -c -m 644 'COPYING' 
> >> '/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh/COPYING'
> >> ...
> >> [   30s] Executing(%doc): /bin/sh -e /var/tmp/rpm-tmp.6245
> >> [   30s] + umask 022
> >> [   30s] + cd /usr/src/packages/BUILD
> >> [   30s] + cd crmsh
> >> [   30s] + DOCDIR=/var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >> [   30s] + export DOCDIR
> >> [   30s] + rm -rf /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >> [   30s] + /bin/mkdir -p 
> >> /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >> [   30s] + cp -pr ChangeLog 
> >> /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh
> >> ...
> >> [   32s] error: create archive failed on file 
> >> /var/tmp/crmsh-1.2.5-build/usr/share/doc/packages/crmsh/AUTHORS: cpio: 
> >> open failed - Bad file descriptor
> >>
> >> If somebody can shed some light or suggest how to deal with
> >> this ...
> >
> > OK. I think I managed to fix it. The result is already upstream.
> > I tested it with rhel6, centos6, fedora 17 and 18. Can you
> > please test too.
> >
> > Thanks,
> >
> > Dejan
> >
> >> Thanks,
> >>
> >> Dejan
> >>
> >>
> >> > No problem. Will test the patch. BTW, did you notice that there
> >> > are packages for rhel too at OBS (see the latest news item at
> >> > https://savannah.nongnu.org/projects/crmsh/).
> >> >
> >> > Cheers,
> >> >
> >> > Dejan
> >> >
> >> > >
> >> > > Best regards,
> >> > > Yusuke
> >> > > --
> >> > > ----------------------------------------
> >> > > METRO SYSTEMS CO., LTD
> >> > >
> >> > > Yusuke Iida
> >> > > Mail: yusk.i...@gmail.com
> >> > > ----------------------------------------
> >> >
> >> >
> >> > _______________________________________________________
> >> > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> >> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> >> > Home Page: http://linux-ha.org/
> >> _______________________________________________________
> >> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> >> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> >> Home Page: http://linux-ha.org/
> > _______________________________________________________
> > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > Home Page: http://linux-ha.org/
> 
> 
> 
> --
> ----------------------------------------
> METRO SYSTEMS CO., LTD
> 
> Yusuke Iida
> Mail: yusk.i...@gmail.com
> ----------------------------------------


_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to