Bug 111008 - gnome-python2 needs spec cleanup
Summary: gnome-python2 needs spec cleanup
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: gnome-python2
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Warren Togami
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: FC3Target
TreeView+ depends on / blocked
 
Reported: 2003-11-26 14:20 UTC by Maxim Dziumanenko
Modified: 2007-11-30 22:10 UTC (History)
11 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2004-10-12 00:08:15 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Fix installed file permissions, own more dirs (3.37 KB, patch)
2004-09-11 18:11 UTC, Ville Skyttä
no flags Details | Diff
Patch to make BuildRequires complete. (472 bytes, patch)
2004-09-29 18:57 UTC, Toshio Kuratomi
no flags Details | Diff
Patch to complete BuildRequires. (1.68 KB, patch)
2004-09-29 19:34 UTC, Toshio Kuratomi
no flags Details | Diff

Description Maxim Dziumanenko 2003-11-26 14:20:50 UTC
Description of problem:
libgnomeprint22 and libgnomeprintui headers and development libs
needed to build gnome-python2, thus it should BuildRequires:
libgnomeprintui-devel >= 2.2.0 (according to configure.in of
gnome-python2 package).

Version-Release number of selected component (if applicable):
gnome-python2-2.0.0-3

How reproducible:
Always

Steps to Reproduce:
1.rpm -e libgnomeprintui-devel
2.rpmbuild --rebuild gnome-python2-2.0.0-3.src.rpm
3.Wait for build errors
  
Actual results:


Expected results:


Additional info:

Comment 1 Miloslav Trmac 2003-11-26 19:26:26 UTC
Triage->Easyfix

Comment 2 Warren Togami 2004-04-13 01:56:58 UTC
Removing EasyFix.

Missing BuildRequires are (some may be redundant):
libgnomeui-devel gnome-panel-devel gtkhtml2-devel libbonoboui-devel
nautilus libgnomeprint22-devel libgnomeprintui22-devel

This spec file needs to be largely redone because it depends on
existence of stuff in the buildroot in order to conditionally include
BuildRequires, which is wrong.  This is because pkg-config and the .pc
files that it looks for are not present during parsing of the
specfile, which is then used to tell the build system which packages
to install within the buildroot.  I suspect this kind of requirement
is missing from fedora.us documents.  Perhaps it could be worded
something like:

"Do not use conditional BuildRequires unless they are only dependent
upon packages within the defined minimal buildroot.  Also do not
depend on features or version of the running kernel."

Any comments?

Comment 3 Michael Schwendt 2004-04-13 10:09:19 UTC
Without a comment in the spec file, it doesn't become clear _why_ it
is done like that.

Even if pkgconfig were added as a buildrequires, it would not find any
of the .pc templates in a clean build environment and hence would not
enable any of the sub-packages. --with/--without switches would be
cleaner nowadays.

The notion of "conditional BuildRequires" may need a good definition
before it is discussed. There's a difference between "%define buildfoo
1" and "%define buildfoo $(do something to find out)".

At fedora.us, most auto-conditional buildrequires are used to adapt a
single spec file to the specific features of the different build
target platforms (e.g. different chain of -devel dependencies,
different vfolder structure of KDE 3.2, Python checks for versioned
dependencies,...).

[As a side-note, current "mach" is broken easily by anything in a spec
file which is executed to modify buildrequires:
https://bugzilla.fedora.us/show_bug.cgi?id=1426 ]

Comment 4 Ville Skyttä 2004-09-11 18:11:15 UTC
Created attachment 103725 [details]
Fix installed file permissions, own more dirs

Here's one suggested patch, fixes the following: there are a lot of bad file
permissions in the current package, some unowned directories, and cosmetic
issues with the %doc's in all subpackages.

Comment 5 Toshio Kuratomi 2004-09-29 18:57:20 UTC
Created attachment 104539 [details]
Patch to make BuildRequires complete.

Looks like someone fixed the rawhide spec file to take care of the conditional
BuildRequires problem in comments #2 and #3.  This patch should add the
necessary BuildRequires reported in the original report and Comment #2.  It
also removes a great many BuildRequires dragged in via dependencies.  Tested
with mach configured for FC2.

Comment 6 Toshio Kuratomi 2004-09-29 19:34:22 UTC
Created attachment 104541 [details]
Patch to complete BuildRequires.

Sorry, older version.  This is the one that also removes BuildRequires dragged
in via dependencies.

Comment 7 Ville Skyttä 2004-09-29 20:52:33 UTC
Please note that the patch in attachment 104541 [details] is not cumulative, and
attachment 103725 [details] is still needed in addition to it.

Comment 8 Toshio Kuratomi 2004-09-30 00:24:55 UTC
I've tested attachment 103725 [details] and it is a definite improvement to the
%files section.  The only thing I noticed is that it needs to include
the COPYING file, probably in the main package's %doc line.

Comment 9 Warren Togami 2004-10-12 00:08:15 UTC
Should be fixed in tomorrow's rawhide.


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