[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: New Package: yumex



On Sat, 2005-06-18 at 18:31 -0700, Michael A. Peters wrote:

> Needs to be fixed:
> 1) Source0 needs to be full url path to the source tarball.
> 
> 2) Changelog should have versioned entries for Extras
> 
> IE instead of
> * Sat Jun 18 2005 Tim Lauridsen <tla rasmil dk>
> 
> use
> 
> * Sat Jun 18 2005 Tim Lauridsen <tla rasmil dk> - 0.40-2
> (don't worry about the %{?dist}
> 
> You don't need to go back and add those for past changelog entries, just
> start doing that with new changelog entries.
> 
> 3) rpmlint errors:
> rpmlint yumex-0.40-2.fc4.noarch.rpm :
> 
> E: yumex zero-length /usr/share/yumex/yumex.repos.conf
> 
> zero-length /usr/share/yumex/yumex.repos.conf
> zero-length files are a packaging error.
> 
> W: yumex no-dependency-on usermode-consoleonly
> no-dependency-on usermode-consoleonly
> 
> I really have no clue what usermode-consoleonly is - but since your
> package uses consolehelper, you should add
> Requires: usermode
> to the spec file
> 
> W: yumex
> symlink-should-be-relative /usr/bin/yumex /usr/bin/consolehelper
> 
> This probably needs to be patched in the Makefile

The %files section also needs to be fixed.
It should look something like this:

%files -f  %{name}.lang
%defattr(-, root, root, -)
%doc COPYING ChangeLog
%{_datadir}/pixmaps/yumex
%{_datadir}/yumex

%{_bindir}/yumex

%config(noreplace)  %{_sysconfdir}/yumex.conf
%config(noreplace) %{_sysconfdir}/pam.d/yumex
%config(noreplace) %{_sysconfdir}/security/console.apps/yumex

%{_datadir}/applications/fedora-%{name}.desktop



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]