Bug 182320
Summary: | Review Request: gnome-build | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Paul F. Johnson <paul> | ||||
Component: | Package Review | Assignee: | John Mahowald <jpmahowald> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | ||||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2006-08-14 21:14:42 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: | |||||||
Bug Depends On: | 182319 | ||||||
Bug Blocks: | 163779, 189685 | ||||||
Attachments: |
|
Description
Paul F. Johnson
2006-02-21 20:27:16 UTC
Build failed on devel: No Package Found for anjuta-gdl Upstream source download URL is 404'd. fedora-qa script found: * Missing SMP flags. If it doesn't build with it, please add a comment (wiki: PackagingGuidelines#parallelmake) Minor: * Duplicate BuildRequires: gtk+-devel (by gnome-libs-devel), gnome-libs-devel (by libglade-devel), libxml-devel (by libglade-devel), libglade-devel (by gal-devel), gal-devel (by gtkhtml-devel) Should've done bug 182319 first. Making this depend on it. Can we move forward on this and #182319? Spec Name or Url: http://www.smmp.salford.ac.uk/packages/gnome-build.spec SRPM Name or Url: http://www.smmp.salford.ac.uk/packages/gnome-build-0.1.2-2.src.rpm Missing some things when building against developement: No Package Found for gnome-libs-devel No Package Found for gtkhtml-devel No Package Found for libglade-devel >= 2.0.1 No Package Found for gal-devel No Package Found for oaf-devel I'm not too familiar with Gnome devel packaging, but you'll probably want to change to the lastest gtkhtml3-devel, and you'll have to change libglade-devel to libglade2-devel to meet the version requirement. Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec SRPM Name or Url: http://www.knox.net.nz/~nodoid/gnome-build-0.1.2-2.src.rpm Change of URL The BuildRequires mentioned in comment 5 still need to be changed. Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec SRPM Name or Url: http://www.knox.net.nz/~nodoid/gnome-build-0.1.3-2.src.rpm Updated spec and src.rpm Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec SRPM Name or Url: http://www.knox.net.nz/~nodoid/gnome-build-0.1.3-3.src.rpm Various rpmlint fixes Builds in i386 mock In what build environment did you try this? No Match for argument: oaf-devel No Match for argument: gnome-libs-devel No Match for argument: gal-devel [...] checking for pkg-config... no checking for GBF... configure: error: The pkg-config script could not be found o r is too old. Make sure it is in your PATH or set the PKG_CONFIG environment variable to the full path to pkg-config. Alternatively, you may set the environment variables GBF_CFLAGS and GBF_LIBS to avoid the need to call pkg-config. See the pkg-config man page for more details. To get pkg-config, see <http://www.freedesktop.org/software/pkgconfig>. See `config.log' for more details. error: Bad exit status from /home/qa/tmp/rpm/tmp/rpm-tmp.74787 (%build) Building in FC-5. I'll fix the problems you've highlighted Is there a reason you are using the gnome 1 libraries to build gnome-build? It requires gnome2. The configure script says is needs gtk 2.3.0 and gnome 2.3.3. The gnome-build.spec.in in the tarball is either really old or for another distribution that packages gnome differently. The nrpms.net gnome-build for FC4 uses the gnome2 dependencies. Its requirements are: BuildRequires: gtk2-devel BuildRequires: gnome-vfs2-devel BuildRequires: libbonobo-devel BuildRequires: libglade2-devel BuildRequires: libgal2-devel BuildRequires: gdl-devel BuildRequires: gtkhtml2-devel BuildRequires: libxml2-devel Requires: gnome-vfs2 Requires: libbonobo Requires: libglade2 Requires: gtkhtml2 Requires: libgal2 Requires: gdl Requires: libxml2 (In reply to comment #12) > Is there a reason you are using the gnome 1 libraries to build gnome-build? Yes; I read the spec too literally and it was a mistake. I'll remove them again. I've had a look at the buildreqs that are actually needed for this package and will attach a specfile patch shortly. The patch will also address the directory ownership issues. I'm doing a test build for mock/rawhide/i386 at the moment. Created attachment 132189 [details] Fix myriad of issues with this spec AFter applying this spec patch, the package builds in mock for rawhide. No need for any Gnome-1 packages. PFJ should chmod 644 his spec and tarball before building a new SRPM, and then the only rpmlint issue left should be: W: gnome-build-devel no-documentation Given that the perl modules in /usr/share/gnome-build/GBF are not going to be found by any random perl package that looks for them in standard places, I would advocate stripping out the perl requires and provides from the main package altogether. See: http://fedoraproject.org/wiki/Packaging/Perl Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec SRPM Name or Url: http://www.knox.net.nz/~nodoid/gnome-build-0.1.3-5.src.rpm Fixed all of the above problems as well as stripping out the perl stuff. (In reply to comment #15) > Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec > SRPM Name or Url: http://www.knox.net.nz/~nodoid/gnome-build-0.1.3-5.src.rpm > > Fixed all of the above problems as well as stripping out the perl stuff. You've just copied the perl dep-stripping stuff verbatim from the wiki page, including the literal text "unwanted_provide" and "unwanted_require". You're supposed to replace that text with a sed regex matching the perl deps you're trying to get rid of (it's unusual to want to get rid of all of them as is the case here). You should check that you've got this right by doing: $ rpm -qp --provides your-binary-rpm-filename $ rpm -qp --requires your-binary-rpm-filename after building them to make sure that the provides and requires are as you want them. If the perl deps are still there, the filter isn't working. make LIBTOOL=/usr/bin/libtool contains a hardcoded /usr/bin and lacks SMP flags. And yup, the perl requires are still there. Source0 is not downloading for some reason. Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec SRPM Name or Url: http://www.knox.net.nz/~nodoid/gnome-build-0.1.3-5.src.rpm It still lacks the smp flags as my system doesn't always like smp flags. No idea why and have never been able to trace the fault (it won't, for example compile anjuta2 with smp flags on) libtool now fixed as is the perl issue. W: gnome-build mixed-use-of-spaces-and-tabs E: gnome-build shlib-with-non-pic-code /usr/lib64/libgbf-1.so.0.0.1 E: gnome-build shlib-with-non-pic-code /usr/lib64/libgbf-widgets-1.so.0.0.1 For some reason -fPIC isn't working on these libraries. Strange. I can't replicate this problem on either x86 or x86_64. I've fixed the space and tabs problem though. Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec Updated spec file (the SRPM is the same) Contains fPIC fix, spaces/tabs fix and Source0 fix ping! Given that -fPIC is being passed let's ignore shlib-with-non-pic-code. A couple things yet to address: Needs work: - Fix the cp line in %install to a %doc macro or explain why it's needed. - missing smp flags or a note on why not needed (was in 0.1.2-2 ?) Recommend: * Changing URL to http://www.gnome.org/projects/devtools/gnomebuild.shtml Good: + license (GPL) + sources good + builds on devel x86_64 + %find_lang macro + version-release good, is latest + macros used consistently + proper -devel subpackage + %clean good + ldconfig good + proper file ownership + proper permissions + removed .la files Spec Name or Url: http://www.knox.net.nz/~nodoid/gnome-build.spec SRPM Name or Url: http://www.knox.net.nz/~nodoid/gnome-build-0.1.3-8.src.rpm Fixes #23 Added pkgconfig as a R for the -devel package rpmlint just says W: gnome-build-devel no-documentation Dependencies OK. APPROVED as per comment 23. Thanks for the review. Much appreciated. There is still a dependency on perl(GBF::Make) in the built package, which makes it uninstallable. Dammit! The package itself creates perl(GBF::Make). It looks like I'll need to explicitly include that it provides perl(GBF::Make). (In reply to comment #28) > The package itself creates perl(GBF::Make). It looks like I'll need to > explicitly include that it provides perl(GBF::Make). I think you should filter out the perl(GBF::Make) dependency, as you've done with the others. This is because the perl modules provided by and used by this package are not in the standard perl module install directories, which means that no other packages will be able to use these perl modules unless they know exactly where to look for them. I'd check to ensure that no perl Provides: have slipped through as well. |