[Bug 168608] Review Request: emelFM2

[bugzilla at redhat.com]

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.