[Bug 168608] Review Request: emelFM2
bugzilla at redhat.com
bugzilla at redhat.com
Mon Sep 26 22:21:11 UTC 2005
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: emelFM2
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=168608
------- Additional Comments From wickerpms at arcor.de 2005-09-26 18:20 EST -------
(In reply to comment #1)
> - Both patches contain hardcoded paths (/usr/share in patch0 and /usr in
> patch1). It has to be %{_datadir} and %{_prefix} respectively. You could for
> example use %DATADIR% and %PREFIX% in your patches and do sed substitution in
> %prep to set the right path.
Thanks. Did that with prefix in Makefile.config.
Removed broken-icon path using only sed instead.
> - In the specfile, you defined the patches names with %{version} included. It is
> going to cause trouble on the next package update. You'd better replace it with
> 0.1.2.
fixed
> - In %install, you use %{prefix}. It should be %{_prefix} (with underscore)
fixed
> - The package sould own %{_libdir}/%{name} and %{_datadir}/pixmaps/%{name}
fixed
New SRPM:
http://home.arcor.de/christoph.wickert/fedora/extras-review/SRPMS/emelfm2-0.1.2-2.fc4.src.rpm
Updated SPEC:
http://home.arcor.de/christoph.wickert/fedora/extras-review/SPECS/emelfm2.spec
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
More information about the fedora-extras-list
mailing list