Bug 226648
| Summary: | Merge Review: xorg-x11-xauth | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Nobody's working on this, feel free to take it <nobody> | ||||
| Component: | Package Review | Assignee: | Parag AN(पराग) <panemade> | ||||
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | medium | ||||||
| Version: | rawhide | CC: | ajax, notting, panemade, sandmann | ||||
| Target Milestone: | --- | Keywords: | Reopened | ||||
| Target Release: | --- | Flags: | panemade:
fedora-review+
|
||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2010-09-03 05:18:31 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
Nobody's working on this, feel free to take it
2007-01-31 21:33:59 UTC
1) I will suggest this pacakge to follow current packaging guidelines and remove buildroot, %clean section and cleaning of build root in %install 2) See http://fedoraproject.org/wiki/Upgrade_paths_%E2%80%94_renaming_or_splitting_packages#Do_I_need_to_Provide_my_old_package_names.3F I think we should remove provides and Obsoletes now. I can't find old packages built on koji. 3) also good to preserve timestamps when using install commands. 4) rpmlint on SRPM and RPM reports xorg-x11-xauth.src: W: strange-permission mkxauth 0775 xorg-x11-xauth.src:24: W: unversioned-explicit-provides xauth xorg-x11-xauth.src:25: W: unversioned-explicit-provides mkxauth xorg-x11-xauth.src:27: W: unversioned-explicit-obsoletes XFree86-xauth xorg-x11-xauth.src:27: W: unversioned-explicit-obsoletes mkxauth xorg-x11-xauth.i686: W: obsolete-not-provided XFree86-xauth xorg-x11-xauth.i686: W: obsolete-not-provided XFree86 xorg-x11-xauth.i686: W: self-obsoletion mkxauth obsoletes mkxauth xorg-x11-xauth.i686: E: zero-length /usr/share/doc/xorg-x11-xauth-1.0.2/AUTHORS xorg-x11-xauth.i686: E: zero-length /usr/share/doc/xorg-x11-xauth-1.0.2/README xorg-x11-xauth.i686: E: zero-length /usr/share/doc/xorg-x11-xauth-1.0.2/NEWS Created attachment 432892 [details]
xorg-x11-xauth-spec-cleanup.patch
I have tried to cleanup spec and here is patch which will fix above issues. Just need to add Changelog entry. and Fix permission for file mkxauth
Looks koji build is successful with new changes. http://koji.fedoraproject.org/koji/taskinfo?taskID=2328928 sandmann ping sandmann ping ajax ping After repetitive pings on bz and IRC to sandmann and ajax, looking into discussion happened in yesterday's FESCo meeting where X/Desktop team members almost shown un-willingness to fix this review, I am pushing back this to NEW queue. Mostly addressed in F15. The explicit provides are not removed though, all the X collection packages provide their binaries explicitly so people don't need to rely on a particular package providing them. Looks no one yet officially reviewed this package. Not sure if recent approvals from FESCo allows to close the reviews without any reviewer reviewing changes. So, I am re-opening this. I think generally maintainer should provide koji build link for final review. Thanks Ajax for the updated build. This package is reviewed now as requested changes are partially committed. I see that rpmlint still showing xorg-x11-xauth.x86_64: E: zero-length /usr/share/doc/xorg-x11-xauth-1.0.2/AUTHORS xorg-x11-xauth.x86_64: E: zero-length /usr/share/doc/xorg-x11-xauth-1.0.2/README xorg-x11-xauth.x86_64: E: zero-length /usr/share/doc/xorg-x11-xauth-1.0.2/NEWS but zero-length files are harmless. Closing this now. |