Bug 226650

Summary: Merge Review: xorg-x11-xdm
Product: [Fedora] Fedora Reporter: Nobody's working on this, feel free to take it <nobody>
Component: Package ReviewAssignee: Parag AN(पराग) <panemade>
Status: CLOSED NEXTRELEASE QA Contact: Fedora Package Reviews List <fedora-package-review>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: panemade, pertusus, sandmann
Target Milestone: ---Flags: panemade: fedora-review+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-10-01 16:38:00 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: 237621    
Bug Blocks:    
Attachments:
Description Flags
spec file patch with cleanups none

Description Nobody's working on this, feel free to take it 2007-01-31 21:34:14 UTC
Fedora Merge Review: xorg-x11-xdm

http://cvs.fedora.redhat.com/viewcvs/devel/xorg-x11-xdm/
Initial Owner: sandmann

Comment 1 Patrice Dumas 2008-01-11 18:34:35 UTC
I think that almost many of the files in /etc/X11/xdm/ should be
%config and even %config(noreplace).

Having the pam files %config and not %config(noreplace) seems a
bit dubious to me.

It seems to me that xdm lacks proper integration in fedora for 2 
things, it doesn't use consolekit or /usr/share/xsessions. To 
help with that, I created little package that add:
    * Support for a session chooser using xsessions/*.desktop files
    * ConsoleKit session
at 
http://www.environnement.ens.fr/perso/dumas/xdm-ck-xsessions.html

I think that it should be nice to use it in some way in fedora.

Comment 2 Patrice Dumas 2008-01-11 18:39:19 UTC
Created attachment 291409 [details]
spec file patch with cleanups

remove old cruft, keep timestamps, version obsoletes and provides.

%{_datadir}/X11/app-defaults/Chooser is not to be edited by the 
user,and hopefully  there is a place where user customization
should go.

Comment 3 Patrice Dumas 2008-03-23 16:14:09 UTC
xdm-1.1.7 is out, please update and also please consider my
patch above.

Comment 4 Patrice Dumas 2008-04-06 10:35:37 UTC
It would be nice to have consolekit support in F-9. I am using what I describe
above for some moonths now, and I think it would be nice to have it in fedora. I
can submit it for inclusion or you can add it in the xdm package, but please
move on.

Comment 5 Parag AN(पराग) 2010-10-01 16:38:00 UTC
Its simple cleanup so directly done in git. Fixed in xorg-x11-xdm-1.1.6-21.fc15

Remember Ajax don't want to remove Provides: so I have kept them and he will be updating this package to latest upstream soon.


APPROVED.

Comment 6 Parag AN(पराग) 2012-06-15 16:33:53 UTC
hah! howcome i missed this package to set the flag fedora-review+ ?