Bug 213121 - Review Request: fast-user-switch-applet - a panel applet for switching users
Summary: Review Request: fast-user-switch-applet - a panel applet for switching users
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Ray Strode [halfline]
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FC-ACCEPT
TreeView+ depends on / blocked
 
Reported: 2006-10-30 21:19 UTC by Matthias Clasen
Modified: 2007-11-30 22:11 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-01-08 21:25:16 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Matthias Clasen 2006-10-30 21:19:59 UTC
The fast-user-switch-applet builds on the gdmflexiserver framework to provide
convenient management of several X sessions at the same time. We want to get this
into Core now, since it will be part of our larger plans around user switching and
console handling for FC7.

Spec file and source and binary rpms can be found here:
http://people.redhat.com/mclasen/fusa/

Comment 1 Matthias Clasen 2006-12-06 15:37:44 UTC
I've put updated srpm and spec at the same location.

Comment 2 Ray Strode [halfline] 2007-01-08 19:27:06 UTC
spec file looks clean and shiny.

I always forget our policy of Epoch handling though.  It might be a good idea to
explicitly put an epoch of 0.

should we version the buildrequires?

What's --with-gdm-config=/etc/gdm/custom.conf do?  Remember for most people
custom.conf is just an empty stub file.  Will that cause a problem for the applet?

is rpmlint output clean?



Comment 3 Matthias Clasen 2007-01-08 19:50:47 UTC
From looking at the code, it uses gnome-vfs to monitor the config file, and
reloads the gdm configuration if it changes. It does not read the file though, 
it gets the configuration from gdm. I guess it would be better to get notification
on configuration changes from gdm, but thats just an upstream bug to file, not 
something we need to sort out prior to accepting the package.

I don't think versioning the BRs would add a lot. I'd much rather rely on
version checks in the configure script. The versioned BRs in the spec file get
outdated to easily.

rpmlint output was clean when I did the packages.

The packaging guidelines don't say anything about requiring an explicit epoch, 
and that would be just silly.

Comment 4 Ray Strode [halfline] 2007-01-08 19:53:01 UTC
okie dokie, looks good then.

build it!

Comment 5 Jesse Keating 2007-01-08 20:12:17 UTC
Buildsystem knows about it, import and build away.  Please close when built for
dist-fc7

Comment 6 Matthias Clasen 2007-01-08 21:25:16 UTC
built


Note You need to log in before you can comment on or make changes to this bug.