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 172183 - Review Request: drgeo-doc
Summary: Review Request: drgeo-doc
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Aurelien Bompard
QA Contact: David Lawrence
URL:
Whiteboard:
Depends On:
Blocks: FE-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2005-11-01 08:31 UTC by Eric Tanguy
Modified: 2007-11-30 22:11 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2005-11-09 21:22:51 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)

Description Eric Tanguy 2005-11-01 08:31:52 UTC 
Spec Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc.spec
SRPM Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc-1.6-1.src.rpm
Description: Html documentation for drgeo
Comment 1 Aurelien Bompard 2005-11-01 11:48:16 UTC 
* Everything below %setup should be in %install. %prep is just for unpacking the
archive and applying the patches. The install of the files should be in the
%install section.
* If you use "install -D" you don't need to use mkdir before
* Use the -p switch of "install" to preserve timestamps
* HTML files and pictures should be installed with mode 644, not 755.
* Add the AUTHORS, README, TODO, ChangeLog, and COPYING files
Comment 2 Eric Tanguy 2005-11-01 17:17:11 UTC 
(In reply to comment #1)
> * Everything below %setup should be in %install. %prep is just for unpacking the
> archive and applying the patches. The install of the files should be in the
> %install section.
Ok
> * If you use "install -D" you don't need to use mkdir before
> * Use the -p switch of "install" to preserve timestamps
> * HTML files and pictures should be installed with mode 644, not 755.
Ok
> * Add the AUTHORS, README, TODO, ChangeLog, and COPYING files
Ok but without TODO which does not exist

Spec Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc.spec
SRPM Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc-1.6-2.src.rpm
Comment 3 Aurelien Bompard 2005-11-01 17:22:13 UTC 
About the AUTHORS, README, ChangeLog, and COPYING files, it's better to package
them as %doc in the %files section.
Comment 4 Ralf Corsepius 2005-11-01 17:27:39 UTC 
What the rationale for not packaging these files as part of the drgeo rsp.
drgeo-devel packages?

To me, shipping them separately doesn't make much sense.
Comment 5 Eric Tanguy 2005-11-01 17:53:36 UTC 
(In reply to comment #4)
> What the rationale for not packaging these files as part of the drgeo rsp.
> drgeo-devel packages?
> 
> To me, shipping them separately doesn't make much sense. 
It makes sense to package them separately because the soft and the doc does not
move at the same rate. That's why the version number for the soft is  
1.1.0 and 1.6 for doc. And that's why also the are shipped in 2 separate archives.
New version : 
Spec Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc.spec
SRPM Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc-1.6-3.src.rpm
Comment 6 Rex Dieter 2005-11-01 18:01:20 UTC 
Also, a separate -doc rpm can be packaged as .noarch.rpm
Comment 7 Aurelien Bompard 2005-11-01 23:35:33 UTC 
* The BuildRoot must be cleaned at the beginning of %install
* %{_datadir}/drgeo/help should be owned by the package, that was right.
* Don't copy the AUTHORS...COPYING files in %install, just put this in %files:
  %doc AUTHORS README ChangeLog COPYING
  It will copy the files to the right place (%_docdir) and tag them
Comment 8 Eric Tanguy 2005-11-02 05:26:07 UTC 
Done.
Spec Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc.spec
SRPM Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc-1.6-4.src.rpm
Comment 9 Aurelien Bompard 2005-11-02 09:33:55 UTC 
You have to own %{_datadir}/drgeo/help in this package, because it is not in
drgeo. Everything else looks OK.
Comment 10 Eric Tanguy 2005-11-02 18:37:44 UTC 
Done :
Spec Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc.spec
SRPM Name or Url: http://perso.wanadoo.fr/eric.tanguy/drgeo-doc-1.6-5.src.rpm
Comment 11 Aurelien Bompard 2005-11-02 19:44:10 UTC 
Review for release 5:
* RPM name is OK
* Source drgeo-doc-1.6.tar.gz is the same as upstream
* Builds fine in mock
* rpmlint of drgeo-doc looks OK
* File list of drgeo-doc looks OK
* Will be accessible from drgeo when bug 172321 is fixed
APPROVED
Note You need to log in before you can comment on or make changes to this bug.