Bug 325741 - pm-utils requires vbetool which is x86 only
Summary: pm-utils requires vbetool which is x86 only
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: pm-utils
Version: rawhide
Hardware: ia64
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Phil Knirsch
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-10-10 00:19 UTC by Doug Chapman
Modified: 2015-03-05 01:19 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-10-10 05:51:37 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Doug Chapman 2007-10-10 00:19:17 UTC
Description of problem:
There was a recent change to pm-utils to make it depend on vbetool.  Since
vbetool is x86 specific this prevents pm-utils and hence the majority of the
gnome rpms since they depend on pm-utils.

I am unclear as to the "right" thing being pm-utils not depending on vbetool for
non-x86 OR gnome not depending on pm-utils but one or the other will be needed
in order for Fedora to work on non-x86.

I am running into this while trying to bring up Fedora on ia64 so I imagine
those working on bringing up sparc and arm on Fedora are seeing this as well.  I
also imagine this is an issue for ppc.

I am open to suggestions on how we can make this portable.


Version-Release number of selected component (if applicable):
 pm-utils-0.99.4-3.fc8

worked OK in pm-utils-0.99.3-11.fc8

Comment 1 Doug Chapman 2007-10-10 01:33:49 UTC
After looking at the spec file I see that ppc and ppc64 are already accounted for:

%ifnarch ppc ppc64 
Requires: vbetool
%endif


Can we update this to include ia64 (and for that matter sparc and arm since I
know others are in the process of bringing up fedora on those arches as well). 
Or, perhaps it would be better to change this to something along the lines of:

%ifarch i386 x86_64  
However I am not sure if this is the correct syntax.



Comment 2 Till Maas 2007-10-10 05:51:37 UTC
fixed in cvs


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