Hi Yusuke-san, On Fri, May 10, 2013 at 02:38:10PM +0900, yusuke iida wrote: > Hi, Dejan > > Thank you for applying a patch. > However, since it has still mistaken, please correct once again.
:D I couldn't find a typo in this line if my life depended on it. Bill Durr also reported the issue with a patch attached in: https://savannah.nongnu.org/bugs/?38950 Cheers, Dejan > > Regards, > Yusuke > > 2013/4/22 Dejan Muhamedagic <[email protected]>: > > On Mon, Apr 22, 2013 at 11:27:49AM +0900, yusuke iida wrote: > >> Hi, Dejan > >> > >> Thank you for merging a patch. > >> > >> However, since there is typo in part, please correct again. > > > > Oops. Sorry for the typo. Fixed now. > > > > Cheers, > > > > Dejan > > > >> Regards, > >> Yusuke > >> > >> 2013/4/18 Dejan Muhamedagic <[email protected]>: > >> > Hi Yusuke-san, > >> > > >> > On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: > >> >> Hi, Dejan > >> >> > >> >> 2013/4/4 Dejan Muhamedagic <[email protected]>: > >> >> > 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. > >> >> > >> >> In my environment, the macro of rpmbuild is as follows. > >> >> > >> >> rhel6.3 > >> >> # rpmbuild --showrc | grep rhel > >> >> -14: rhel 6 > >> >> > >> >> fedora18 > >> >> # rpmbuild --showrc | grep fedora > >> >> -14: fedora 18 > >> >> > >> >> So I want you to revise it as follows at least. > >> >> > >> >> # hg diff > >> >> diff -r da93d3523e6a crmsh.spec > >> >> --- a/crmsh.spec Tue Mar 26 11:44:17 2013 +0100 > >> >> +++ b/crmsh.spec Tue Apr 16 13:08:37 2013 +0900 > >> >> @@ -6,7 +6,7 @@ > >> >> %global upstream_version tip > >> >> %global upstream_prefix crmsh > >> >> > >> >> -%if 0%{?fedora_version} || 0%{?centos_version} || 0%{?rhel_version} > >> >> +%if 0%{?fedora_version} || 0%{?centos_version} || 0%{?rhel_version} > >> >> || 0%{?rhel} || 0%{?fedora} > >> >> %define pkg_group System Environment/Daemons > >> >> %else > >> >> %define pkg_group Productivity/Clustering/HA > >> > > >> > Patch applied. Thanks! > >> > > >> >> >> - 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. > >> >> > >> >> hmm ... > >> >> For example, can't it change so that the function in which pssh is > >> >> used may be disabled by the configure option? > >> > > >> > The functionality is still there, even without pssh. For > >> > instance, static reports can also be examined. It's just that the > >> > live updates are going to be quite a bit slower, if somebody > >> > wants to use the history feature to examine changes happening in > >> > the cluster. > >> > > >> >> If it is possible, can it not exclude pssh from Requires? > >> > > >> > I already reasoned in my previous message (quoted above) why I'm > >> > reluctant to do that. > >> > > >> > Cheers, > >> > > >> > Dejan > >> > > >> >> Regards, > >> >> Yusuke > >> >> > > >> >> > Cheers, > >> >> > > >> >> > Dejan > >> >> > > >> >> >> Regards, > >> >> >> Yusuke > >> >> >> > >> >> >> 2013/2/19 Dejan Muhamedagic <[email protected]>: > >> >> >> > 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: [email protected] > >> >> >> >> > > ---------------------------------------- > >> >> >> >> > > >> >> >> >> > > >> >> >> >> > _______________________________________________________ > >> >> >> >> > Linux-HA-Dev: [email protected] > >> >> >> >> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > >> >> >> >> > Home Page: http://linux-ha.org/ > >> >> >> >> _______________________________________________________ > >> >> >> >> Linux-HA-Dev: [email protected] > >> >> >> >> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > >> >> >> >> Home Page: http://linux-ha.org/ > >> >> >> > _______________________________________________________ > >> >> >> > Linux-HA-Dev: [email protected] > >> >> >> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > >> >> >> > Home Page: http://linux-ha.org/ > >> >> >> > >> >> >> > >> >> >> > >> >> >> -- > >> >> >> ---------------------------------------- > >> >> >> METRO SYSTEMS CO., LTD > >> >> >> > >> >> >> Yusuke Iida > >> >> >> Mail: [email protected] > >> >> >> ---------------------------------------- > >> >> > > >> >> > > >> >> > >> >> > >> >> > >> >> -- > >> >> ---------------------------------------- > >> >> METRO SYSTEMS CO., LTD > >> >> > >> >> Yusuke Iida > >> >> Mail: [email protected] > >> >> ---------------------------------------- > >> > _______________________________________________________ > >> > Linux-HA-Dev: [email protected] > >> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > >> > Home Page: http://linux-ha.org/ > >> > >> > >> > >> -- > >> ---------------------------------------- > >> METRO SYSTEMS CO., LTD > >> > >> Yusuke Iida > >> Mail: [email protected] > >> ---------------------------------------- > > > > > > _______________________________________________________ > > Linux-HA-Dev: [email protected] > > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > > Home Page: http://linux-ha.org/ > > > > -- > ---------------------------------------- > METRO SYSTEMS CO., LTD > > Yusuke Iida > Mail: [email protected] > ---------------------------------------- _______________________________________________________ Linux-HA-Dev: [email protected] http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev Home Page: http://linux-ha.org/
