Re: [Spacewalk-devel] [PATCH] updateinfo to Errata import in spacewalk-repo-sync

2011-03-10 Thread Michael Calmer
Hi, Am Mittwoch, 9. März 2011, 11:29:45 schrieb Jan Pazdziora: On Mon, Mar 07, 2011 at 02:28:48PM +0100, Michael Calmer wrote: [...] when running git am, I get the following whitespace-related warnings. Could you please polish the patch to get rid of them? Sure, I will work on all the issues

Re: [Spacewalk-devel] [PATCH] SUSE/AutoYaST Support

2011-03-10 Thread Uwe Gansert
on Tuesday 08 March 2011 Miroslav Suchý wrote: 0002-update-database-to-support-SUSE-distributions.patch update the database with a SUSE breed to make SUSE an option for a kickstartable tree You need to add this INSERT to schema/spacewalk/common/data/rhnKSInstallType.sql as well. yes,

Re: [Spacewalk-devel] [PATCH] updateinfo to Errata import in spacewalk-repo-sync

2011-03-10 Thread Jan Pazdziora
On Mon, Mar 07, 2011 at 02:28:48PM +0100, Michael Calmer wrote: Hello, the attached patch set implements updateinfo to errata import in spacewalk- repo-sync. This patch also requires two small changes to yum. Patches are also attached. This patch include also two enhancements. It adds

Re: [Spacewalk-devel] [PATCH] updateinfo to Errata import in?spacewalk-repo-sync

2011-03-10 Thread Jan Pazdziora
On Tue, Mar 08, 2011 at 05:57:05PM +0100, Michael Calmer wrote: * fix errata override bug. Create a unique advisory_name This was about the id element in the updateinfo which is not unique. Not unique where, across what domain? When I take

Re: [Spacewalk-devel] [PATCH] updateinfo to Errata import in spacewalk-repo-sync

2011-03-10 Thread Jan Pazdziora
On Mon, Mar 07, 2011 at 02:28:48PM +0100, Michael Calmer wrote: Hello, the attached patch set implements updateinfo to errata import in spacewalk- repo-sync. [...] diff --git a/backend/satellite_tools/reposync.py b/backend/satellite_tools/reposync.py index 2fc46a0..ee3be6e 100644 ---

Re: [Spacewalk-devel] [PATCH] SUSE/AutoYaST Support

2011-03-10 Thread Uwe Gansert
on Thursday 10 March 2011 Jan Pazdziora wrote: we hit the You are not authorized to access bug #674344. for bnc#674344 here. As the patch is trivial, I'm leaning towards dropping the bnc number from the commit message, so that you don't have to go through the trouble of creating