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

bugzilla at redhat.com bugzilla at redhat.com
Tue Dec 13 09:46:55 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: smart -- Next generation package handling tool


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





------- Additional Comments From enrico.scholz at informatik.tu-chemnitz.de  2005-12-13 04:46 EST -------
* Tue Dec 13 2005 Enrico Scholz <enrico.scholz at informatik.tu-chemnitz.de> - 0.40-1.6 
- use %%python_sitelib instead of %%_libdir/... 
- added Encoding entry in desktop file 
- removed the ui(...) virtual provides for now 
- made 'ksmarttray' an real package (instead of a subpackage) and 
  added desktop entry 
- added -usermode subpackage which contains consolehelper wrappers for 
  smart and use it in the -gtk subpackage 
- enhanced 'smart-update' script 

http://ensc.de/fedora/smart.spec
http://ensc.de/fedora/smart-0.40-1.6.src.rpm

=====================


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

ok, I added xen0 + xenU

But the 'multiple-version' flag seems to be a match-whole-string
thing. So there must be done something upstream.


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

Should not matter. 'smart' itself requires at least rpm 4.4 and this
version understands '(missingok)'.


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

ok, removed for now


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

looks like a kdelibs-devel bug. I can not fix it now because I can not
express this BuildRequires: for pre-FC5 rpms.

When kdelibs-devel is not fixed till approval, I will add this
buildrequires for the FC5 branch in the CVS. For now, I will assume
builds on FC4.


> 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 added a ksmarttray.desktop but do not know how to test it


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

I think, sudo is the correct tool. 'ksmarttray' seems to execute
'smart-update' regularly and I would dislike periodically appearing
dialog boxes which are asking for a root password.


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

ok, added -usermode subpackage which provides the corresponding
wrappers.


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