quite a few packages are still missing in the repos :-(

[Thorsten Leemhuis]

On 06.09.2008 20:03, David Woodhouse wrote:
> On Sat, 2008-09-06 at 15:15 +0200, Thorsten Leemhuis wrote:
>> On 06.09.2008 12:00, Hans de Goede wrote:
 >
>>> I have no objections to temporarily using ExcludeArch ppc*,
>> I don't like it very much -- it's quite easy to forget fixing ppc and 
>> reverting the ExcludeArch later. Then ppc might stays excluded for weeks 
>> or event months -- that's IMHO not what we want.
> We should set up ExcludeArch bugs for each architecture like we have for
> Fedora,  and impose the same rule that all exclusions _must_ have a bug
> filed (with appropriate explanation, etc.)

I tend to partly disagree and agree with this:

Here's why I do partly disagree: I don't like the idea with manually 
maintained "ExcludeArch tracker bugs" in Bugzilla much, as that IMHO is 
error prone and easily forgotten. Going to bugzilla and filing new bugs 
just for tracking is also something that a lot of people don't like much 
and thus forget on purpose or by accident.

OTOH those ExcludeArch tracker bugs help those interested in a 
particular arch to check for packages that need fixing. That is really 
important.

So we IMHO need those bugs, but also make sure that they are maintainted 
properly. Maybe the ExcludeArch bugs and tracker bugs work just fine as 
long as there is a script (started once a week by cron) that checks the 
packages for use of ExcludeArch or ExclusiveArch and yells if no bugs 
are found? Maybe something like this could work: Enforce a rule that the 
Bug number needs to be on the same line with a specific keyword. E.g. 
like this:

ExcludeArch: ppc64        # ExcludeArchBug: 734

That way two simple grep call from a full cvs checkout will find all the 
packages with a ExcludeArch/ExclusiveArch:

$ grep -e ^ExcludeArch -e ^ExclusiveArch */{devel,F-9,F-8}/*.spec | sort
k9copy/devel/k9copy.spec:ExcludeArch: ppc ppc64
k9copy/F-9/k9copy.spec:ExcludeArch: ppc ppc64
lxdream/devel/lxdream.spec:ExclusiveArch:  %{ix86} x86_64
lxdream/F-8/lxdream.spec:ExclusiveArch:  %{ix86} x86_64
lxdream/F-9/lxdream.spec:ExclusiveArch:  %{ix86} x86_64
mythtv/devel/mythtv.spec:ExcludeArch: ppc64
mythtv/F-8/mythtv.spec:ExcludeArch: ppc64
mythtv/F-9/mythtv.spec:ExcludeArch: ppc64        # ExcludeArchBug: 734
zsnes/devel/zsnes.spec:ExclusiveArch: i386
zsnes/F-8/zsnes.spec:ExclusiveArch: i386
zsnes/F-9/zsnes.spec:ExclusiveArch: i386

With a second call you are able to find all those without a bug:

$ grep -e ^ExcludeArch -e ^ExclusiveArch */{devel,F-9,F-8}/*.spec | \
   sort | grep -v ExcludeArchBug
k9copy/devel/k9copy.spec:ExcludeArch: ppc ppc64
k9copy/F-9/k9copy.spec:ExcludeArch: ppc ppc64
lxdream/devel/lxdream.spec:ExclusiveArch:  %{ix86} x86_64
lxdream/F-8/lxdream.spec:ExclusiveArch:  %{ix86} x86_64
lxdream/F-9/lxdream.spec:ExclusiveArch:  %{ix86} x86_64
mythtv/devel/mythtv.spec:ExcludeArch: ppc64
mythtv/F-8/mythtv.spec:ExcludeArch: ppc64
zsnes/devel/zsnes.spec:ExclusiveArch: i386
zsnes/F-8/zsnes.spec:ExclusiveArch: i386
zsnes/F-9/zsnes.spec:ExclusiveArch: i386

(Note that mythtv/F-9/mythtv.spec is missing here).

That of course should happen not only in RPM Fusion; instead, it should 
be in the Fedora Packaging Guidelines and then we will use it just the 
same way in RPM Fusion.

Comments?

CU
knurd