Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 182415
Summary: | Review Request: man-pages-uk - Ukrainian man pages from Linux Documentation Project | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Andy Shevchenko <andy> | ||||
Component: | Package Review | Assignee: | Dmitry Butskoy <dmitry> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | dmitry | ||||
Target Milestone: | --- | ||||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2006-06-08 09:36:16 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: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 163779 | ||||||
Attachments: |
|
Description
Andy Shevchenko
2006-02-22 13:44:05 UTC
*** Bug 182411 has been marked as a duplicate of this bug. *** *** Bug 182413 has been marked as a duplicate of this bug. *** This is my first package and I need a sponsor. P.S. Sorry for dups. While I am looking for license of previous files I found new page with newest tarballs. The tarball includes license now and project not dead. Spec Name or Url: ftp://andriy.asplinux.com.ua/pub/people/andy/extras/man-pages- uk.spec SRPM Name or Url: ftp://andriy.asplinux.com.ua/pub/people/andy/extras/man-pages- uk-0.1-0.1.20051210wiki.src.rpm Just a couple of quick comments 1. Why the need to change the name to remove the -utf8? I can't recall seeing anything in the FE rules over that 2. {buildroot} should really be ${RPM_BUILD_ROOT} 3. INSTALLPATH - should that not be DESTDIR? 4. Do you need to include the epoch? It can be more of a pain than anything (well, from what I've seen of it). Other than that, it looks okay. I'm not happy with the naming scheme - Release: 0.1%{?date:.%{date}wiki}%{?dist} seems somewhat long (to me). Can you not just have Release:0.1%{?dist}. I would normally reserve the longer release if the source was dragged from cvs/svn > 2. {buildroot} should really be ${RPM_BUILD_ROOT}
According to the packaging guidelines, either is fine as long as it is used
consistently throughout the spec file (which appears to be the case here). The
use of %{buildroot} is fine here.
1. Naming for this package derived from man-pages-$LANG. I think the more convenient to use man-pages-uk instead of smth. else. I see also "Addon packages" from package naming guidelines. And I consider proposed name building similar to "Addon package (locales)". I should like to hear RH people about this. 3. Do I need to patch original Makefile? INSTALLPATH here is a author's variable. 4. Sure, epoch is my local requirement (I'll droped it in spec). Some words about release. This project does not have svn/cvs, but wiki page with last changes. Author of package builds tarball with %date. As package guidelines describes I need to put this date in release. May be I am wrong. I found addon info about this package in its Makefile. Also, Gentoo people package it as real tarball name. I've fixed name, release, version in specfile. ftp://andriy.asplinux.com.ua/pub/people/andy/extras/manpages-uk-utf8-0.0.0.2-0. 1.20060228.src.rpm ftp://andriy.asplinux.com.ua/pub/people/andy/extras/manpages-uk-utf8.spec Please, review it again. Thank you. Certainly, it is a useful package, but IMHO it is a bad choice for the first FE experience. To sponsor you, a sponsor should review your stuff more carefully (rather than in normal case) -- i.e., for this package the sponsor must know Ukranian language... Maybe you can submit something also? After that (when you will be sponsored) it would be more easy to review this one. Set Component to "Package Review" (was aalib for some reason...) * Tue Apr 04 2006 Andy Shevchenko <andriy.ua> - update to 20060328 snapshot - change version according to Makefile Spec Name or Url: ftp://andriy.asplinux.com.ua/pub/people/andy/extras/manpages- uk-utf8.spec SRPM Name or Url: ftp://andriy.asplinux.com.ua/pub/people/andy/extras/manpages- uk-utf8-0.1-0.1.20060328.src.rpm For comment #5: > 1. Why the need to change the name to remove the -utf8? I can't recall seeing > anything in the FE rules over that The similar Core packages have names in the form "man-pages-LL", where LL is "fr", "de", etc. Therefore this package must have the same naming scheme as the Core uses. It is an obvious exception from the "FE rules" -- the Core behaviour always takes precedence over the Extras, as Extras is just the additional stuff for the Core... Paul, with the current naming there are: "man-pages-cs", "man-pages-de", "man-pages-es", ... "manpages-uk-utf8" ?!?! Agree, it looks ugly! Andy, I suggest you to rename it back. According to http://www.redhat.com/archives/fedora-extras-list/2006-May/ msg00927.html I'll rename package to man-pages-uk. It's more sense here. Rename version here: Spec Name or Url: ftp://andriy.asplinux.com.ua/pub/people/andy/extras/man-pages- uk.spec SRPM Name or Url: ftp://andriy.asplinux.com.ua/pub/people/andy/extras/man-pages- uk-0.1-0.1.20060328.src.rpm Since comment #4 all your urls appear wrapped, I cannot "just click" them. IMHO bugzilla input forms never auto-wraps such url lines... ?.. The built package still contains non-utf manuals. Consult, for example, man-pages-ru to get idea how to fix this (using iconv(1) etc.) It seems that there is no any official version yet, perhaps it is better in such situation to use versioning scheme similar to "man-pages-ja" package (:)), i.e. the cvs date as the version. But it is not so good from the point of view of FE Package Guidelines. Maybe ask maillists about this issue (i.e., the versioning of a cvs stuff which has no any versions at all). Or perhaps it was discussed somewhere earlier? What the manuals are non-utf8 exactly? I review all manuals in tarball and find nothing non-utf mans. I've catched the package version from a VERSION Makefile's variable. Is it not correct? > and find nothing non-utf mans. Oops, sorry. I was confused a little because of mc uses nroff to show man pages, and nroff converts utf8 to my ancient locale... :) > I've catched the package version from a VERSION Makefile's variable. OK, surely it is correct. Next days I'll finish the review. Remarks and nitpicks: - Summary and description could sounds like in the other man-pages-LL packages (It also can help (a little) to include this package into Core :)) - INSTALL is useless in the %doc - License must be "FDL" - *.UK %doc files should be marked %lang(uk) - IMHO it would be nice to add "%description -l uk". Created attachment 130610 [details]
Suggested changes for the spec file
Updated version here: SRPM: ftp://andriy.asplinux.com.ua/pub/people/andy/extras/man-pages-uk-0.1-0.2. 20060328.src.rpm rpmlint OK Must/Should items OK Source matches upstream Works fine Approved! Already built in devel. Thanks for review. FC-3 branch requires FESCO approval. Ask on fedora-extras-list. |