[Bug 175438] Review Request: smart -- Next generation package handling tool

[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: smart -- Next generation package handling tool


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=175438





------- Additional Comments From ville.skytta at iki.fi  2005-12-12 14:10 EST -------
All other packagers I'm aware of ship ksmarttray as "ksmarttray", not
"smart-ksmarttray", I'd suggest following that.  It just looks and feels better
that way IMO (bundling in the tarball aside, kyum vs yum-kyum, yumex vs
yum-yumex, synaptic vs apt-synaptic etc).

I don't know of a way to enumerate kernel variants but I don't think listing a
bunch of known ones would be much of a maintenance burden.  It wouldn't work for
custom kernels though.  If it helps, kernel module packages can also be
recognized by a name prefix or something in the future once the proposal is
finalized.

What rpm versions does Requires(missingok) work with?  (At least so that the
specfile parser groks it.)

Specifying "tui" or "gtk" as the version to ui(%name) sounds strange, and now
that the text UI is bundled in the main package, ui(%name) doesn't seem to serve
a purpose any more.

desktop-file-install warns:
/home/scop/rpmbuild/SOURCES/smart.desktop: missing encoding  (guessed UTF-8)

ksmarttray build fails on FC5t1+rawhide, configure tries to use -lXt but it
isn't found.  BuildRequires: libXt-devel fixes it.

How is ksmarttray supposed to be found by end users?  Some kind of menu entry
(an usual one, or one that is listed in KDE's "add applet" choices) should be
included.

I don't think sudo is the correct tool to handle updating with ksmarttray,
something with the ability to prompt for the root password in a GUI would sound
better (eg.  consolehelper and friends).

IMO smart-gtk should be changed to use consolehelper too.

If you want to have a Provides: smartpm, adding Provides: smartpm-gtk to the gtk
subpackage too wouldn't hurt.

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