[Bug 226572] Merge Review: xorg-x11-docs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: xorg-x11-docs


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226572


fedora@xxxxxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|fedora@xxxxxxxxxxxxx        |sandmann@xxxxxxxxxx
                 CC|                            |fedora@xxxxxxxxxxxxx
               Flag|fedora-review?              |fedora-review-




------- Additional Comments From fedora@xxxxxxxxxxxxx  2007-02-03 13:02 EST -------
* package name

 Quoting the guidelines: "Large documentation files should go in a -doc
subpackage." doc, not docs. Please rename and place proper obsoletes and
provides with versions.

* install location

 All doc files should go to %doc (e.g. /usr/share/docs) -- that holds true for
files in seperate doc packges, too. Please fix. Having a symlink from
%{_datadir}/X11/doc/ to the target directory in %doc might be acceptable if
there is a good reasons for it (e.g. in case upstream is installing the packages
there) -- I leave that discussion up to you

* version

 There is a 1.3 version online already. Is that specific to X.org 7.2 (which is
not in rawhide as of now)? If not it might be wise to update to it.

* rplint
rpmlint on ./xorg-x11-docs-1.2-4.fc7.src.rpm
W: xorg-x11-docs invalid-license MIT/X11
-> probably MIT (see below)

W: xorg-x11-docs unversioned-explicit-obsoletes XFree86-doc
W: xorg-x11-docs unversioned-explicit-obsoletes xorg-x11-doc
W: xorg-x11-docs unversioned-explicit-provides XFree86-doc
W: xorg-x11-docs unversioned-explicit-provides xorg-x11-doc
-> Please place a version on those (well, not for xorg-x11-doc, as that will be
the new name for the package)


rpmlint on ./xorg-x11-docs-1.2-4.fc7.noarch.rpm
W: xorg-x11-docs incoherent-version-in-changelog 1.2-4.fc6 1.2-4.fc7
W: xorg-x11-docs invalid-license MIT/X11

* MISC:

 * I don't like that tarname macro, as it should be no problem to hardcode that
in the two places where it is getting used.

 * From %install:
rm -rf $RPM_BUILD_ROOT%{_datadir}/X11/doc/hardcopy/XPRINT
   Would be nice to know why that is done -- a small comment would be nice Ohh,
actually there is a comment in the files section. Why no simply exclude the
deleted files there below the comment?

  Well, maybe even better: It IMHO would make the whole specfile a lot more
readable if the files section would just contain the topdir of the doc target
dir (and the man files). rpm will then make sure that all files and dirs get
included properly. But that's a matter of style.

 * the source package ships a README -- should we ship that?

 * there is no COPYING file in the package -- how to make sure it's MIT? Might
be wise to contact upstream about it.

* Besides that:
 package meets naming and packaging guidelines.
 specfile is properly named, is cleanly written and uses macros consistently.
 build root is correct.
 source files match upstream:
  f817c5df43817846c1b27bac83da74d7  xorg-docs-1.2.tar.bz2
 final provides and requires are sane:
 no shared libraries are present.
 package is not relocatable.
 owns the directories it creates.
 doesn't own any directories it shouldn't.
 no duplicates in %files.
 file permissions are appropriate.
 %clean is present.
 no scriptlets present.
 no headers.
 no pkgconfig files.
 no libtool .la droppings.
 not a GUI app.
 not a web app.
 no open bugs

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]