Bug 5100 - merge gnome profiles
Summary: merge gnome profiles
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Misc (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.2.0
Assignee: Pierre Ossman
URL:
Keywords: relnotes, samuel_tester
Depends on:
Blocks:
 
Reported: 2014-04-16 13:14 CEST by Pierre Ossman
Modified: 2014-05-06 16:16 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Pierre Ossman cendio 2014-04-16 13:14:14 CEST 
With bug 5077 sorted out, we might want to consider merging the Gnome 2 and Gnome 3 profiles. The reason we have two are:

 - Gnome 3 didn't start (fixed when we upgraded Xorg)

 - Gnome 3 is started differently on Ubuntu (fixed by bug 5077)

 - Gnome 2 and 3 look different so different screen shots are needed

Only the last point is still valid, and the question is if it's important enough to have an extra profile definition. For KDE we only have the one profile, even though it can result in three different desktops (KDE 3, KDE 4 and KDE Plasma).
Comment 1 Pierre Ossman cendio 2014-04-22 11:38:44 CEST 
Fixed in r28907.

Tester should verify that that:

 - Gnome 3 works
 - Gnome Classic works
 - Gnome 2 works on RHEL
 - Gnome 2 works on Solaris

Remember to check the images and the translations.
Comment 2 Samuel Mannehed cendio 2014-04-24 17:10:52 CEST 
(In reply to comment #1)
> Fixed in r28907.
> 
> Tester should verify that that:
> 
>  - Gnome 3 works
>  - Gnome Classic works
>  - Gnome 2 works on RHEL
>  - Gnome 2 works on Solaris
> 
> Remember to check the images and the translations.

Looks good, verified using build 4332.
Comment 3 Peter Åstrand cendio 2014-05-05 16:17:25 CEST 
In profiles.hconf:

default=gnome-2
Comment 4 Pierre Ossman cendio 2014-05-06 15:33:47 CEST 
(In reply to comment #3)
> In profiles.hconf:
> 
> default=gnome-2

Fixed in r28943.
Comment 5 Samuel Mannehed cendio 2014-05-06 16:16:29 CEST 
Looks good.
Note You need to log in before you can comment on or make changes to this bug.