Bug 213121

Summary:	Review Request: fast-user-switch-applet - a panel applet for switching users
Product:	[Fedora] Fedora	Reporter:	Matthias Clasen <mclasen>
Component:	Package Review	Assignee:	Ray Strode [halfline] <rstrode>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	davidz, fedora-package-review, gnomeuser, havardw, notting, pcfe, tjb
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2007-01-08 21:25:16 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	188268

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