[Bug 520322] Review Request: mm-common - common files for GNOME C++ bindings

[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.


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





--- Comment #10 from Krzesimir Nowak <qdlacz@xxxxxxxxx>  2009-09-13 14:43:03 EDT ---
(In reply to comment #9)
> Well, for 0.7.3-1
> 
> * Timestamps
>   - Now this package installs more files, keeping timestamps
>     on installed files as much as possible is preferable.
>     Please consider to use
> ------------------------------------------------------
> make install DESTDIR=%{buildroot} INSTALL="insatll -p" CPPROG="cp -p"
> ------------------------------------------------------
>     to keep timestamps on installed files.
>     - Usually adding INSTALL="insatll -p" works for makefiles based
>       on Makefiles generated from recent autotools
>     - However this package also uses "install-sh" to install files.
>       For this, adding CPPROG="cp -p" will (usually) keep timestamps
>       on installed files.
> 

Done.

> * redundant %doc
>   - Files under %{_mandir} are automatically marked as %doc.
>   - Also files under %{_docdir} are automatically marked as %doc.
> 

Done.

> * Directory ownership issue
>   - This is actually under discussion, however this package installs
>     some files under %{_datadir}/aclocal/ and currently owned by
>     "automake" rpm. So adding "Requires: automake" to this package is 
>     preferable to satisfy directory ownership issue.
> 

Done.

>   - -docs package installs some files under %{_docdir}/%{name}/,
>     however the directory %{_docdir}/%{name} is not owned by any packages,
>     which should be owned by "this" (i.e. mm-common-docs) package:
> 
>    
> https://fedoraproject.org/wiki/Packaging/UnownedDirectories#Forgetting_to_Include_a_Toplevel_Directory

I hope I got it right now. I added ownerships to %{_datadir}/%{name}/ and to
%{_docdir}/%{name}/. 

> 
> ! %changelog
>   - I recommend to add entries into %changelog even this is still
>     under review.  

Ok.

spec: http://wips.pl/~kudi/rpm/mm-common/mm-common.spec
srpm: http://wips.pl/~kudi/rpm/mm-common/mm-common-0.7.3-2.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
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]