Bug 1113541
| Summary: | Rename Request: x11trace - Utility for tracing X11 protocol for debugging | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | David Howells <dhowells> |
| Component: | Package Review | Assignee: | Parag AN(पराग) <panemade> |
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | medium | ||
| Version: | rawhide | CC: | package-review |
| Target Milestone: | --- | Flags: | panemade:
fedora-review+
gwync: 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: | 2014-07-24 14:16:30 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
David Howells
2014-06-26 11:38:15 UTC
The package and its components have been renamed from xtrace (which clashes with a glibc utility) to x11trace.
Review:
+ mock build is successful for rawhide
+ rpmlint output on rpms gave
x11trace.src: W: spelling-error %description -l en_US strace -> trace, traces, strafe
x11trace.src: W: spelling-error %description -l en_US xtrace -> trace, x trace, extract
x11trace.src: W: strange-permission xtrace_1.3.1.orig.tar.gz 0444L
x11trace.x86_64: W: spelling-error %description -l en_US strace -> trace, traces, strafe
x11trace.x86_64: W: spelling-error %description -l en_US xtrace -> trace, x trace, extract
3 packages and 0 specfiles checked; 0 errors, 5 warnings.
+ Source verified with upstream as (sha256sum)
srpm tarball : b22ca970efe24fcedff44f38075c5f34e0893f66abb419f3180ff92adf9e2a22
upstream tarball : b22ca970efe24fcedff44f38075c5f34e0893f66abb419f3180ff92adf9e2a22
+ License "GPLv2" is valid and included in source files.
suggestions:
1) Package must own all directories that it creates. Note: Directories without known owners: /usr/share/x11trace
==> fix this by changing
%{_datadir}/x11trace/*.proto
to
%{_datadir}/x11trace
2) you can change source tarball permission to 0644 before packaging to srpm
3) I see you have following which can be removed from spec
echo hello2
4) all your patche prefix names should start with package name i.e.
change xtrace to x11trace
5) Now the most important thing for a Rename package review is to follow http://fedoraproject.org/wiki/Packaging:Guidelines#Renaming.2FReplacing_Existing_Packages
you will then need in your spec
Provides: xtrace = %{version}-%{release}
Obsoletes: xtrace < 1.3.1-7
also use the master branch spec only while renaming package. You should use release number 7 and not 6
Fix above issues and submit new package for review.
Spec URL: http://people.redhat.com/dhowells/xtrace/x11trace.spec SRPM URL: http://people.redhat.com/dhowells/xtrace/x11trace-1.3.1-7.fc20.src.rpm now rpmlint only complains x11trace.x86_64: W: obsolete-not-provided xtrace which is okay as glibc-utils provides /usr/bin/xtrace APPROVED. New Package SCM Request ======================= Package Name: x11trace Short Description: A program for X11 protocol tracing Owners: dhowells Branches: f20 InitialCC: I have reset the fedora-cvs+ to fedora-cvs? flag as its what you need. Git done (by process-git-requests). x11trace is now built for rawhide. Should I build it for F20 also? yes you should :) Okay, build for F20. Built for F20, that is Thanks. We can close this review then. |