Bug 199381

Summary: PATCH: sparc sun drivers
Product: [Fedora] Fedora Reporter: Dennis Gilmore <dennis>
Component: xorg-x11-driversAssignee: Adam Jackson <ajax>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideKeywords: EasyFix
Target Milestone: ---   
Target Release: ---   
Hardware: sparc   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-07-27 21:56:32 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
sun drivers
none
adds sparc drivers as requested none

Description Dennis Gilmore 2006-07-19 04:03:37 UTC 
Description of problem:
the attached patch adds the sun specific drivers to the xorg-x11-drivers 
package.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Dennis Gilmore 2006-07-19 04:03:38 UTC 
Created attachment 132658 [details]
sun drivers
Comment 2 Mike A. Harris 2006-07-25 05:42:37 UTC 
The Requires in this package are automatically generated by a one liner
shell script present in a comment in the spec file, as that is the only
sane way to ensure the package is in sync with what the actual driver
packages are in the OS.  When someone changes a driver, they are likely
to forget to update the drivers package, or to introduce inconsistencies,
so having it scriptable is a big advantage.

However, this approach unintentionally imposes a limit on which architectures
the package handles.  The solution I've got in mind for this is to change
the script from listing one ifarch block per driver, to having one ifarch
block per arch, and having a number of Requires: lines in the single
block for the given driver.

That will serve 3 useful purposes:

- Allows us to autogenerate the architectures we build for.

- Allows us to include statically generated content in the spec for 3rd
  party architectures such as sparc to supplement the autogenerated bits.

- Will make the spec file much smaller and neater looking.

Just need to write the script code that will spit out output of the form:

%ifarch %{ix86}
Requires: xorg-x11-drv-ati xorg-x11-drv-i810 ...
Requires: xorg....
%endif

%ifarch x86_64
Requires: ...
%endif

..., with each Requires line having a maximum of 4 drivers listed per line
for readability.  I suspect a small perl script is the quickest and easiest
way to do this.  Low priority right now, but I'll implement it at some point.

In the mean time, I can add a hack into the spec for sparc for now.  Can
you rewrite the spec in the form:

%ifarch sparc sparc64
Requires: <all of the new drivers to add>
%endif

Also, you generated your patch backwards (args reversed).

TIA
Comment 3 Dennis Gilmore 2006-09-11 22:15:52 UTC 
Created attachment 136038 [details]
adds sparc drivers as requested
Comment 4 Dennis Gilmore 2007-07-10 15:13:29 UTC 
With secondary archs to come online soon.  I will be submitting all the sun 
drivers for review and then they will live in fedora cvs space.  maybe that 
will help
Comment 5 Adam Jackson 2007-07-27 21:56:32 UTC 
Dennis, I've added you to the acl for xorg-x11-drivers.  When you've got the sun
drivers in CVS, go ahead and add them to the drivers metapackage behind %ifarch.