Re: [Linux-ha-dev] A patch for crmsh.spec

2013-07-08 Thread yusuke iida
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

2013-07-02 Thread Dejan Muhamedagic
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

2013-07-01 Thread yusuke iida
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

2013-06-03 Thread yusuke iida
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

2013-05-29 Thread yusuke iida
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

2013-05-09 Thread yusuke iida
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

2013-04-22 Thread Dejan Muhamedagic
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

2013-04-21 Thread yusuke iida
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

2013-04-18 Thread Dejan Muhamedagic
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

2013-04-15 Thread yusuke iida
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

2013-03-19 Thread yusuke iida
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

2013-03-19 Thread yusuke iida
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

2013-03-19 Thread Lars Marowsky-Bree
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

2013-03-19 Thread yusuke iida
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

2013-02-21 Thread yusuke iida
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

2013-02-19 Thread Dejan Muhamedagic
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

2013-02-19 Thread Dejan Muhamedagic
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

2013-02-15 Thread Dejan Muhamedagic
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

2013-02-14 Thread yusuke iida
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/