Bug 844162

Summary: Review Request: mate-session-manager - session manager for the MATE desktop
Product: [Fedora] Fedora Reporter: Wolfgang Ulbrich <fedora>
Component: Package ReviewAssignee: Thomas Spura <tomspur>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: dan.mashal, notting, package-review, tomspur
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: 2012-08-21 11:52:04 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:

Description Wolfgang Ulbrich 2012-07-29 13:02:58 UTC
Spec URL: http://raveit65.fedorapeople.org/Mate-Desktop/fc16/SPECS/mate-session-manager.spec
SRPM URL: http://raveit65.fedorapeople.org/Mate-Desktop/fc16/SRPM/mate-session-manager-1.4.0-5.fc16.src.rpm
Description: mate-session manages a MATE desktop or GDM / MDM login session. It starts up
the other core MATE components and handles logout and saving the session.
Fedora Account System Username: raveit65

Comment 1 Dan Mashal 2012-08-06 09:03:40 UTC
Due to your mental instability and immaturity with mate-conf you are no longer welcome as a co-maintainer of the mate-desktop group. Please discontinue any further work on this package.

Comment 2 Dan Mashal 2012-08-13 11:34:49 UTC
I take back my previous comments, please continue the review. We have gotten past our differences and working together now, for the record.

Comment 3 Thomas Spura 2012-08-13 11:55:34 UTC
What's this:
"""
# Needed for mate-settings-daemon
Requires: mate-control-center
"""

When mate-settings-daemon needs this, it should require it, so the session-manager doesn't need to require it.
(But I didn't find a mate-settings-daemon nor mate-control-center package...)

Comment 4 Thomas Spura 2012-08-13 11:56:48 UTC
Sorry, just the mate-control-center dependency seems missing.

mate-settings-daemon is now in the depends on list.

Comment 5 Wolfgang Ulbrich 2012-08-21 11:52:04 UTC
I don't work for the Mate-Desktop project anymore, because i see no basic for working together with Dan Mashal.