Bug 484931
Summary: | Renaming Review Request: hosts3d - 3D real-time network visualiser | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Simon <cassmodiah> | ||||||
Component: | Package Review | Assignee: | Jochen Schmitt <jochen> | ||||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||||
Severity: | medium | Docs Contact: | |||||||
Priority: | medium | ||||||||
Version: | rawhide | CC: | fedora-package-review, jochen, notting, pahan, susi.lehtola | ||||||
Target Milestone: | --- | Flags: | jochen:
fedora-review+
kevin: fedora-cvs+ |
||||||
Target Release: | --- | ||||||||
Hardware: | All | ||||||||
OS: | Linux | ||||||||
Whiteboard: | |||||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||||
Doc Text: | Story Points: | --- | |||||||
Clone Of: | Environment: | ||||||||
Last Closed: | 2009-02-22 17:23:33 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
Simon
2009-02-10 18:48:07 UTC
Now the package Provides: homestead = 0.95-1 Obsoletes: homestead < 0.95-1 but the current version in F10 is homestead-0.95-1, which won't be obsoleted. Change this to Provides: homestead = %{version}-%{release} Obsoletes: homestead < %{version}-%{release} or just remove the versioning from Provides & Obsoletes? I think, you should wrote Provides: homestead = %{version}-%{release} Obsoletes: homestead < 0.95-1 - Provides: homestead = %{version}-%{release} Obsoletes: homestead < 0.95-1 - Perhaps THIS way?! This looks correct and make sense (to me)! Review will be done tomorrow. Good: + Base name of the SPEC file metches with package name. + Package name fullfill the naming guidelines + Package contains current release of the application + Could download sources from upstream + Packaged souces metches with upstream one. (md5sum: 260c55fe010520cbe894c97b224f996b) + Package contains valid license tag + License tag contains GPLv2+ as an valid OSS license + License on the license tag fit with copyright note of the included source files + Package contains verbatin copy of the license text + Proper use of the %{?dist] tag + Proper defintion of the Buildroot + Buildroot will been cleaned on the beginning of %clean and %install + Consistently rpm macro usage + Package use parallel build + Local build works fine. + Koji build works fine. + Rpmlint is quite on source rpm + Rpmlint is quite on binary rpm + Rpmlint is quite on debuginfo package + Files hat proper files permissions + %files stanza has no duplicated entries + All packaged files are owned by the package + %doc stanza is small, so we need no deparate subpackage + package contains desktop entry file. + Proper %changelog Bad: - Please change Provides of the main package and the subpackage as discussed in comment #3 - Package doesn't honor $RPM_OPT_FLAGS - Please remove the -p switch on make install - Debuginfo package contains no sources Question: * Does the sampler subpackage not requires the main subpackage? Created attachment 331728 [details]
Patch to get honor %{optflags}
Created attachment 331729 [details]
Suggested SPEC file with include proposal Patch
This SPEC file use a patch attached to this review request to patch the configure script. this is required, because the configure script overwrites the CXXFLAGS.
Thank you, i emailed this patch to upstream. I'm in a good contact with him :-) I will continue today and i hope we finished it soon. Thank you An additional change: It may be nice, if you can add the autoreconf command before the %configure statemant. The task of the autoreconf command is to make sure, that the configure.in file may be generated on the base of the configure.ac file, which will be patched by my suggested patch. Spec URL: http://cassmodiah.fedorapeople.org/hosts3d-0.97/hosts3d.spec SRPM URL: http://cassmodiah.fedorapeople.org/hosts3d-0.97/hosts3d-0.97-2.fc10.src.rpm done! Good: + autoconf was added as an BR. + autoreconf was called before %configure + Local build works fine + Local install works fine + Local uninstall works fine + Start of the application works without a crash Bad: - Build fails on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1128556 mh, replace autoreconf with autoconf for a better "koji-compatibility" Spec URL: http://cassmodiah.fedorapeople.org/hosts3d-0.97/hosts3d.spec SRPM URL: http://cassmodiah.fedorapeople.org/hosts3d-0.97/hosts3d-0.97-3.fc10.src.rpm Good: + Koji build works fine. *** APPROVED *** thank you Jochen. New Package CVS Request ======================= Package Name: hosts3d Short Description: 3D real-time network visualiser Owners: cassmodiah Branches: F-10 EL-5 cvs done. Included in F-10 devel and epel-5 i forgot to close the bug via bodhi |