Bug 435227
Summary: | Review Request: xpsk31 - GTK+ graphical version of lpsk31 | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Robert 'Bob' Jensen <bob> |
Component: | Package Review | Assignee: | Lucian Langa <lucilanga> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, kevin, lucilanga, notting, vanmeeuwen+fedora |
Target Milestone: | --- | Flags: | vanmeeuwen+fedora:
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: | 2008-08-03 15:09:04 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: |
Description
Robert 'Bob' Jensen
2008-02-28 03:02:56 UTC
Source0 is a 404: http://5b4az.chronos.org.uk/pkg/psk31/xpsk31/xpsk31-0.7.tar.gz Upstream apparently deletes old source, the new source will require a version bump http://5b4az.chronos.org.uk/pkg/psk31/xpsk31/xpsk31-0.8.tar.gz Some pre-review pointers:- - Use of make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p" is recommended as it keeps the timestamps on installed files. This is useful for recent autotools based make file. - INSTALL file is not required in %doc section as it is used to install from the source. RPM installation does not require INSTALL file. I am going to stall this review for a couple days, Upstream has other related packages I would like to also get in to the review process, they all will need similar fixes. I tried to update this package and ran in to problems. I will try again and collect the build log if someone does not beat me to is. Spec URL: http://bjensen.fedorapeople.org/pkgs/hams/SPECS/xpsk31.spec SRPM URL: http://bjensen.fedorapeople.org/pkgs/hams/SRPMS/xpsk31-0.8-1.fc8.src.rpm Koji build on dist-f9: http://koji.fedoraproject.org/koji/taskinfo?taskID=484167 Fails on missing po/Makefile.in.in: == configure: creating ./config.status config.status: creating Makefile config.status: creating src/Makefile config.status: error: cannot find input file: po/Makefile.in.in error: Bad exit status from /var/tmp/rpm-tmp.91780 (%build) == Changing status to NEEDINFO from Reporter Hello, I'm taking over this review request because bob has some problems I've corrected spec file and bumped version new files: http://lucilanga.fedorapeople.org/xpsk31.spec http://lucilanga.fedorapeople.org/xpsk31-0.8-2.fc9.src.rpm The source tarball is a 404, again, and it seems upstream has a new version 1.2 that builds with the same spec and patches RPM Lint: OK, silent Package name: OK Spec file: OK License: OK, GPLv2+ Actual License: OK, no license file but code says GPLv2+ %doc License: No, license is in the code Spec file language: OK Spec file readable: OK Upstream source vs. used tarball: Upstream tarball disappears when new version is released. Newer version 1.2 builds with the same spec and the same patches, so no problem. Might wanna contact upstream to make them have the tarballs around a little longer though. Compile and Build: - F-7: OK - F-8: OK - rawhide: OK - EL-5: OK Applicable Package Guidelines: Locales: N/A Shared libs: N/A Relocatable: N/A Directory and file ownership: OK No duplicate files in %files: OK File Permissions: OK Macro usage: OK Code vs. Content: OK (Large) Documentation: OK %doc affecting runtime: OK Header files in -devel package: N/A Static Libraries in -static package: N/A pkgconfig Requires: N/A Library files: N/A Devel requires base package: N/A .la libtool archives: N/A Duplicate ownership of files/directories: OK Remove BuildRoot: OK UTF-8 filenames: OK package approved in principle, but you might want to bump the version and contact upstream about the disappearing tarballs. > The source tarball is a 404, again, and it seems upstream has a new version 1.2
> that builds with the same spec and patches
Verified building with 1.2, I will initial import the newer version and contact
upstream.
New Package CVS Request ======================= Package Name: xpsk31 Short Description: GTK+ graphical version of lpsk31 for Ham Radio Owners: lucilanga,bjensen Branches: F-8 F-9 InitialCC: Cvsextras Commits: yes New Package CVS Request ======================= Package Name: xpsk31 Short Description: GTK+ graphical version of lpsk31 for Ham Radio Owners: lucilanga,bjensen Branches: F-8 F-9 EL-5 InitialCC: Cvsextras Commits: yes cvs done. xpsk31-1.2-1.fc9 has been submitted as an update for Fedora 9 xpsk31-1.2-1.fc8 has been submitted as an update for Fedora 8 xpsk31-1.2-1.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report. xpsk31-1.2-1.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. |