Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan Thank you for merging a patch! 2013/7/2 Dejan Muhamedagic de...@suse.de: Hi Yusuke-san, On Tue, Jul 02, 2013 at 01:08:18PM +0900, yusuke iida wrote: Hi, Dejan Could you incorporate this patch? Oh, somehow missed this one. Sorry about that. Yes, I'll apply this patch. Anyway, why is it so difficult to install pssh? Or did I already forget you reasoning? In the environment which we offer, the package offered from RHEL and a package except Pacemaker which we built cannot be supported. We are also building crmsh. Because want to build crmsh.spec as it is, we want to remove the dependency of pssh from Require if possible. For example, it is also good in such a way switch option. # rpmbuild -bb --without pssh crmsh.spec Do you will not be able to consider such a scheme? Regards, Yusuke Cheers, Dejan I want some messages. Regards, Yusuke 2013/6/3 yusuke iida yusk.i...@gmail.com: Hi, Dejan 2013/4/18 Dejan Muhamedagic de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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. I made sure the source code. In the environment where pssh is not supported, history was collecting information using the crm_report command. Furthermore, the processing which is using pssh was found. It is rsctest. This processing serves as an error of python in the environment where pssh is not supported. Probing resources .Traceback (most recent call last): File /usr/sbin/crm, line 44, in module main.run() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 413, in run do_work() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 323, in do_work if parse_line(levels,shlex.split(' '.join(l))): File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 149, in parse_line rv = d() # execute the command File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 148, in lambda d = lambda: cmd[0](*args) File /usr/lib64/python2.6/site-packages/crmsh/ui.py, line 1945, in rsc_test return test_resources(rsc_l, node_l, all_nodes) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 300, in test_resources if not are_all_stopped(rsc_l, all_nodes_l): File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 250, in are_all_stopped drv.runop(probe) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 143, in runop from crm_pssh import do_pssh_cmd
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi Yusuke-san, On Tue, Jul 02, 2013 at 01:08:18PM +0900, yusuke iida wrote: Hi, Dejan Could you incorporate this patch? Oh, somehow missed this one. Sorry about that. Yes, I'll apply this patch. Anyway, why is it so difficult to install pssh? Or did I already forget you reasoning? Cheers, Dejan I want some messages. Regards, Yusuke 2013/6/3 yusuke iida yusk.i...@gmail.com: Hi, Dejan 2013/4/18 Dejan Muhamedagic de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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. I made sure the source code. In the environment where pssh is not supported, history was collecting information using the crm_report command. Furthermore, the processing which is using pssh was found. It is rsctest. This processing serves as an error of python in the environment where pssh is not supported. Probing resources .Traceback (most recent call last): File /usr/sbin/crm, line 44, in module main.run() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 413, in run do_work() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 323, in do_work if parse_line(levels,shlex.split(' '.join(l))): File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 149, in parse_line rv = d() # execute the command File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 148, in lambda d = lambda: cmd[0](*args) File /usr/lib64/python2.6/site-packages/crmsh/ui.py, line 1945, in rsc_test return test_resources(rsc_l, node_l, all_nodes) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 300, in test_resources if not are_all_stopped(rsc_l, all_nodes_l): File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 250, in are_all_stopped drv.runop(probe) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 143, in runop from crm_pssh import do_pssh_cmd File /usr/lib64/python2.6/site-packages/crmsh/crm_pssh.py, line 24, in module from psshlib import psshutil ImportError: No module named psshlib Since I thought that this was a problem, I added the processing which checks support of pssh. If there is no problem, I want you to apply this patch. Regards, Yusuke 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,
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan Could you incorporate this patch? I want some messages. Regards, Yusuke 2013/6/3 yusuke iida yusk.i...@gmail.com: Hi, Dejan 2013/4/18 Dejan Muhamedagic de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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. I made sure the source code. In the environment where pssh is not supported, history was collecting information using the crm_report command. Furthermore, the processing which is using pssh was found. It is rsctest. This processing serves as an error of python in the environment where pssh is not supported. Probing resources .Traceback (most recent call last): File /usr/sbin/crm, line 44, in module main.run() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 413, in run do_work() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 323, in do_work if parse_line(levels,shlex.split(' '.join(l))): File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 149, in parse_line rv = d() # execute the command File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 148, in lambda d = lambda: cmd[0](*args) File /usr/lib64/python2.6/site-packages/crmsh/ui.py, line 1945, in rsc_test return test_resources(rsc_l, node_l, all_nodes) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 300, in test_resources if not are_all_stopped(rsc_l, all_nodes_l): File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 250, in are_all_stopped drv.runop(probe) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 143, in runop from crm_pssh import do_pssh_cmd File /usr/lib64/python2.6/site-packages/crmsh/crm_pssh.py, line 24, in module from psshlib import psshutil ImportError: No module named psshlib Since I thought that this was a problem, I added the processing which checks support of pssh. If there is no problem, I want you to apply this patch. Regards, Yusuke 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 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
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan 2013/4/18 Dejan Muhamedagic de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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. I made sure the source code. In the environment where pssh is not supported, history was collecting information using the crm_report command. Furthermore, the processing which is using pssh was found. It is rsctest. This processing serves as an error of python in the environment where pssh is not supported. Probing resources .Traceback (most recent call last): File /usr/sbin/crm, line 44, in module main.run() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 413, in run do_work() File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 323, in do_work if parse_line(levels,shlex.split(' '.join(l))): File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 149, in parse_line rv = d() # execute the command File /usr/lib64/python2.6/site-packages/crmsh/main.py, line 148, in lambda d = lambda: cmd[0](*args) File /usr/lib64/python2.6/site-packages/crmsh/ui.py, line 1945, in rsc_test return test_resources(rsc_l, node_l, all_nodes) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 300, in test_resources if not are_all_stopped(rsc_l, all_nodes_l): File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 250, in are_all_stopped drv.runop(probe) File /usr/lib64/python2.6/site-packages/crmsh/rsctest.py, line 143, in runop from crm_pssh import do_pssh_cmd File /usr/lib64/python2.6/site-packages/crmsh/crm_pssh.py, line 24, in module from psshlib import psshutil ImportError: No module named psshlib Since I thought that this was a problem, I added the processing which checks support of pssh. If there is no problem, I want you to apply this patch. Regards, Yusuke 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 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
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan I made sure the fix. Thanks, Yusuke 2013/5/29 Dejan Muhamedagic de...@suse.de: 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 de...@suse.de: 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 de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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 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]
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan Thank you for applying a patch. However, since it has still mistaken, please correct once again. Regards, Yusuke 2013/4/22 Dejan Muhamedagic de...@suse.de: 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 de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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 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 -
Re: [Linux-ha-dev] A patch for crmsh.spec
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 de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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 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
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan Thank you for merging a patch. However, since there is typo in part, please correct again. Regards, Yusuke 2013/4/18 Dejan Muhamedagic de...@suse.de: Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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 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
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi Yusuke-san, On Tue, Apr 16, 2013 at 02:55:40PM +0900, yusuke iida wrote: Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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 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 --
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan 2013/4/4 Dejan Muhamedagic de...@suse.de: 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.specTue Mar 26 11:44:17 2013 +0100 +++ b/crmsh.specTue 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 - 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? If it is possible, can it not exclude pssh from Requires? Regards, Yusuke 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:
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Dejan Can you not yet cope about this matter? Regards, Yusuke 2013/2/21 yusuke iida yusk.i...@gmail.com: 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. - pssh is not provided in rhel. I think that you should not put it in Requires. I made a patch to solve the above. 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 -- 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/
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Lars 2013/2/21 yusuke iida yusk.i...@gmail.com: 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. - pssh is not provided in rhel. I think that you should not put it in Requires. I made a patch to solve the above. Dejan seems to be busy. Would you cope with this instead? Best Regards, Yusuke 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 -- 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/
Re: [Linux-ha-dev] A patch for crmsh.spec
On 2013-03-19T15:28:05, yusuke iida yusk.i...@gmail.com wrote: I made a patch to solve the above. Dejan seems to be busy. Would you cope with this instead? Dejan is just on vacation this week. Let's wait for him to return ;-) Regards, Lars -- Architect Storage/HA SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 21284 (AG Nürnberg) Experience is the name everyone gives to their mistakes. -- Oscar Wilde ___ 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/
Re: [Linux-ha-dev] A patch for crmsh.spec
Hi, Lars OK. Thank you for the information. I wait for dejan to return. Thanks! Yusuke 2013/3/19 Lars Marowsky-Bree l...@suse.com: On 2013-03-19T15:28:05, yusuke iida yusk.i...@gmail.com wrote: I made a patch to solve the above. Dejan seems to be busy. Would you cope with this instead? Dejan is just on vacation this week. Let's wait for him to return ;-) Regards, Lars -- Architect Storage/HA SUSE LINUX Products GmbH, GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer, HRB 21284 (AG Nürnberg) Experience is the name everyone gives to their mistakes. -- Oscar Wilde ___ 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/
Re: [Linux-ha-dev] A patch for crmsh.spec
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. - pssh is not provided in rhel. I think that you should not put it in Requires. I made a patch to solve the above. 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 crmsh.spec.patch Description: Binary data ___ 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/
Re: [Linux-ha-dev] A patch for crmsh.spec
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 ... 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/
Re: [Linux-ha-dev] A patch for crmsh.spec
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/
Re: [Linux-ha-dev] A patch for crmsh.spec
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. 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] A patch for crmsh.spec
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. Best regards, Yusuke -- METRO SYSTEMS CO., LTD Yusuke Iida Mail: yusk.i...@gmail.com crmsh.spec.patch Description: Binary data ___ 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/