[Bug 529375] Review Request: emerillon - A map viewer for GNOME

[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=529375





--- Comment #9 from Simon Wesp <cassmodiah@xxxxxxxxxxxxxxxxx>  2009-12-22 05:25:03 EDT ---
(In reply to comment #8)
> $RPM_BUILD_DIR is not the same as %{buildroot}. 
this is correct

> I'm not sure of the newer style macro for BUILD_DIR 
%{builddir}
Please take a look in /usr/lib/rpm/macros (LINE 836)


(In reply to comment #7)
> Simon, is this package okay?
just a few notes...
Peter, perhaps it is better to use the name-macro instead of the name in the
file list!

It would be nice if you could add a  / to the directories in the filelist.
%{_bindir}/%{name}
%{_libdir}/%{name}
both looks like a file
%{_bindir}/%{name}
%{_libdir}/%{name}/
differs directory and file

in the includedir you should use a "*" instead of the versionnumber. it's
easier for an update.

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