Bug 238971 - koji honours ExcludeArch even without bug numbers.
Summary: koji honours ExcludeArch even without bug numbers.
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: koji
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Cantrell
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-05-04 09:17 UTC by David Woodhouse
Modified: 2013-01-10 01:37 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-06-12 19:25:58 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description David Woodhouse 2007-05-04 09:17:25 UTC
We have rules which say you _MUST_ have a bug filed and on a given ExcludeArch
tracker in order to use ExcludeArch:. The build system should enforce this, to
help people remember.

Simply keeping track of excluded builds isn't sufficient -- when the bugs are
filed, they also give some _explanation_ of what the problem is and why the fix
is beyond the capacity of the package maintainer and needs support from an
expert for the given architecture.

Something simple like requiring bug numbers on the line before the ExcludeArch:
line in the specfile would probably be sufficient.

Comment 1 Mike McLean 2007-05-04 16:01:28 UTC
This is the only reference I can find to the policy:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines?highlight=%28ExcludeArch%29

Is this the extent of it, or is there something more specific?

Currently Koji does not not parse the spec directly, it lets rpm handle that.
Since this policy involves a comment, we could have to rip the spec out and
parse it. This is complicated by the fact that the policy does not set a strict
format for the bug reference(s).

All this should be doable in principle, but I would feel better about things if
there was a more rigid rule for the bug reference.

Did plague enforce this?

Comment 2 Jesse Keating 2007-05-04 16:13:12 UTC
No, plague did not enforce this.  I wonder if it would make more sense at the
SCM level instead, parse the spec at check in time or what not and if no proper
comment is found...

Comment 3 Mike McLean 2007-05-04 16:32:49 UTC
Enforcing at SCM occurred to me as well. This may make more sense.
 - spec is more easily available at that point
 - developer gets the needed feedback sooner


Comment 4 David Woodhouse 2007-05-04 17:56:19 UTC
Yeah, doing it at the SCM level seems sensible.

Comment 5 Jesse Keating 2007-06-12 19:25:58 UTC
I'm closing this WONTFIX as we want this at the SCM level.


Note You need to log in before you can comment on or make changes to this bug.