Re: [Spacewalk-devel] [PATCHES] two patches for SUSE support

2012-01-17 Thread Jan Pazdziora
On Mon, Jan 16, 2012 at 04:48:17PM +0100, Michael Calmer wrote: Do you know where this getPlatform come from? It's in client/rhel/rhn-client-tools/src/up2date_client/platform.py Ahh, thanks. But this is bad. I would like to use the official platform module from python

Re: [Spacewalk-devel] [PATCHES] two patches for SUSE support

2012-01-17 Thread Michael Calmer
Hi, Am Dienstag, 17. Januar 2012, 10:54:45 schrieb Jan Pazdziora: On Mon, Jan 16, 2012 at 04:48:17PM +0100, Michael Calmer wrote: [...] Renamed in Spacewalk master and rhn-client-tools-1.7.5-1. Thanks for pointing this issue out. Thanks. I have a new patch attached which define

Re: [Spacewalk-devel] [PATCHES] two patches for SUSE support

2012-01-17 Thread Miroslav Suchý
On 01/17/2012 11:40 AM, Michael Calmer wrote: Hi, Am Dienstag, 17. Januar 2012, 10:54:45 schrieb Jan Pazdziora: On Mon, Jan 16, 2012 at 04:48:17PM +0100, Michael Calmer wrote: [...] Renamed in Spacewalk master and rhn-client-tools-1.7.5-1. Thanks for pointing this issue out. Thanks. I

Re: [Spacewalk-devel] [PATCHES] two patches for SUSE support

2012-01-16 Thread Michael Calmer
Hi, Am Donnerstag, 5. Januar 2012, 16:46:04 schrieb Miroslav Suchý: On 01/02/2012 04:30 PM, Michael Calmer wrote: Hi, happy new year. I have attached two small patches to imporove the SUSE client support. The first one is a general workaround for a not installed yum. Currently

Re: [Spacewalk-devel] [PATCHES] two patches for SUSE support

2012-01-16 Thread Jan Pazdziora
On Mon, Jan 16, 2012 at 12:15:26PM +0100, Michael Calmer wrote: easier fix. But I would most rather see there if getPlatform() == 'deb' or isThisSuse(): Ok, but looking into it more deeply I wonder how this can work. I cannot find a function called getPlatform() in the module

Re: [Spacewalk-devel] [PATCHES] two patches for SUSE support

2012-01-16 Thread Michael Calmer
Hi, Am Montag, 16. Januar 2012, 13:01:13 schrieb Jan Pazdziora: On Mon, Jan 16, 2012 at 12:15:26PM +0100, Michael Calmer wrote: easier fix. But I would most rather see there if getPlatform() == 'deb' or isThisSuse(): Ok, but looking into it more deeply I wonder how this can