[Bug 185951] Review Request: amsn : msn messenger clone
bugzilla at redhat.com
bugzilla at redhat.com
Sun Jun 18 22:29:25 UTC 2006
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: amsn : msn messenger clone
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=185951
paul at all-the-johnsons.co.uk changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |paul at all-the-johnsons.co.uk
------- Additional Comments From paul at all-the-johnsons.co.uk 2006-06-18 18:21 EST -------
The $RPM_BUILD_ROOTs need to be consistent
either ${RPM_BUILD_ROOT} or easier, %{buildroot}, but don't mix them.
You're also taking ownership of directories (by the looks of it)
Instead of
%{_datadir}/amsn/foo/
use
%{_datadir}/amsn/foo/*.zip (for example)
You cannot guarantee that later on someone won't create a plugin which needs to
be added to the directory "owned" by the package
Is there not other languages for this package?
If there is - under the %install
%find_lang {name}
Then on the files line
%files -f %{name}.lang
However, some apps install the language files as part of the make install, so
you may find this step fails.
--
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.
More information about the Fedora-package-review
mailing list