On Thu, Jun 23, 2011 at 12:20 AM, Anas Nashif <[email protected]> wrote: > Why exactly is this link needed anyways
uxlaunch is now a generic xsession launcher. It will use the session files stored in /usr/share/xsessions (and a few other places including ~/.config/xsessions/) to start up the desktop session, more or less compatible with gdm, kdm etc. Also, the user themselves can both create sessions themselves and select which session to start at boot time completely independent of what the distro, or SA has set as global preference. The issue is that we need to figure out what the "system" default is at boot time. Also, the SA needs to be able to override that for all users, and last, the user needs to be able to override this himself. On top of that, in the current state of MeeGo it's impossible to have packages install "the default setting" since at any given time we will have multiple potential sessions available. Take a look for instance at netbook. As soon as a user installs the xbmc package, he now has 2 session files. It would be an issue if all of a sudden his desktop would no longer appear after a reboot, but something else instead. It also makes no sense to do a package %post, since, for instance, on netbook, it's quite likely that mcompositor is installed before mutter, and now your netbook image would start.... god knows what. So, I made the choice to switch from: `echo "session=/usr/bin/foo" >> /etc/sysconfig/uxlaunch" to `ln -sf gnome.desktop /usr/share/xsessions/default.desktop` which in itself is already better, since we're still allowing the config file to be used, and the user can override this choice, and we're not potentially messing up a config file. >, seems like a broken implementation already that we need one, and we are >breaking thing even more by doing this in mic. we're doing it in a %post, and changing one %post script line for another. the alternative is to wait for the mic2 guys to implement yet another way to support this in the already convoluted `xconfig` option, which is really not attractive at this time. oh BTW, the change below is horribly wrong, it needs to be: -ln -sf x-meego-ivi.desktop /usr/share/applications/default.desktop +ln -sf x-meego-ivi.desktop /usr/share/xsessions/default.desktop I hope I didn't mess up my original notes to RE, apologies if I did. The links all need to be made under /usr/share/xsessions/ instead. Auke > Can you give us some insight on why this link is needed please? > > > Anas > > > On 23 Jun 2011, at 00:01, Kok, Auke-jan H wrote: > >> impossible to do that, with the new method we can't do this in >> packages as they would always conflict. >> >> unless you want to introduce packages that conflict by default and >> install 1 symlink, which totally does not make any sense at all. >> >> This is actually a sane way of doing this that doesn't mean yet >> another mic2 change... >> >> Auke >> >> >> On Wed, Jun 22, 2011 at 1:51 PM, Anas Nashif <[email protected]> wrote: >>> Just in case you are submitting this again.... >>> On 22 Jun 2011, at 21:42, Tracy Graydon wrote: >>>> >>>> --- custom/scripts/uxlaunch-symlink-hs.post >>>> +++ custom/scripts/uxlaunch-symlink-hs.post >>>> +# Set symlink pointing to .desktop file >>>> +ln -sf x-meego-hs.desktop /usr/share/applications/default.desktop >>> >>> This is a hack, please provide a proper solution use packages. >>> >>> same for all variants below... >>> >>> Anas >>> >>> >>>> --- custom/scripts/uxlaunch-symlink-ivi.post >>>> +++ custom/scripts/uxlaunch-symlink-ivi.post >>>> +# Set symlink pointing to .desktop file >>>> +ln -sf x-meego-ivi.desktop /usr/share/applications/default.desktop >>>> --- custom/scripts/uxlaunch-symlink-nb.post >>>> +++ custom/scripts/uxlaunch-symlink-nb.post >>>> +# Set symlink pointing to .desktop file >>>> +ln -sf x-meego-nb.desktop /usr/share/applications/default.desktop >>>> --- custom/scripts/uxlaunch-symlink-tb.post >>>> +++ custom/scripts/uxlaunch-symlink-tb.post >>>> +# Set symlink pointing to .desktop file >>>> +ln -sf x-meego-tb.desktop /usr/share/applications/default.desktop >>>> --- custom/scripts/uxlaunch-symlink-tv.post >>>> +++ custom/scripts/uxlaunch-symlink-tv.post >>>> +# Set symlink pointing to .desktop file >>>> +ln -sf x-meego-tv.desktop /usr/share/applications/default.desktop >>>> --- custom/scripts/uxlaunch-trail.post >>>> +++ custom/scripts/uxlaunch-trail.post >>>> -echo "xopts=-nocursor" >> /etc/sysconfig/uxlaunch >>>> - >>>> --- handset/handset-armv7hl-n900.yaml >>>> +++ handset/handset-armv7hl-n900.yaml >>>> @@ -11,6 +11,7 @@ >>>> - Nokia N900 Proprietary Support >>>> - Handset Applications >>>> PostScripts: >>>> + - uxlaunch-symlink-hs >>>> - meegotouch-n900 >>>> - bootchart >>>> - fstab-n900 >>>> --- ivi/ivi-ia32-conx.yaml >>>> +++ ivi/ivi-ia32-conx.yaml >>>> @@ -20,4 +20,4 @@ >>>> - gpsd-clients >>>> - gd >>>> PostScripts: >>>> - - uxlaunch-ivi >>>> + - uxlaunch-symlink-ivi >>>> --- ivi/ivi-ia32.yaml >>>> +++ ivi/ivi-ia32.yaml >>>> @@ -13,4 +13,4 @@ >>>> - Korean Support >>>> - Samples and Demos >>>> PostScripts: >>>> -- uxlaunch-ivi >>>> + - uxlaunch-symlink-ivi >>>> --- netbook/netbook-ia32.yaml >>>> +++ netbook/netbook-ia32.yaml >>>> @@ -16,3 +16,5 @@ >>>> - flash-plugin >>>> Repos: >>>> - adobe >>>> +PostScripts: >>>> + - uxlaunch-symlink-nb >>>> --- repos.yaml >>>> +++ repos.yaml >>>> @@ -5,7 +5,13 @@ >>>> Url: >>>> http://download.meego.com/testing/repos/non-oss/@ARCH@/packages/ >>>> - Name: oss >>>> Url: >>>> http://repo.meego.com/MeeGo/builds/trunk/@BUILD_ID@/repos/oss/@ARCH@/packages/ >>>> + - Name: oss-source >>>> + Url: >>>> http://repo.meego.com/MeeGo/builds/trunk/@BUILD_ID@/repos/oss/source/ >>>> + Options: --source >>>> - Name: non-oss >>>> Url: >>>> http://repo.meego.com/MeeGo/builds/trunk/@BUILD_ID@/repos/non-oss/@ARCH@/packages/ >>>> + - Name: non-oss-source >>>> + Url: >>>> http://repo.meego.com/MeeGo/builds/trunk/@BUILD_ID@/repos/non-oss/source/ >>>> + Options: --source >>>> - Name: adobe >>>> Url: http://linuxdownload.adobe.com/linux/i386/ >>>> --- tablet/tablet-armv7hl-n900.yaml >>>> +++ tablet/tablet-armv7hl-n900.yaml >>>> +Name: MeeGo Tablet N900 (armv7hl) >>>> +Schedule: "*" >>>> +Active: True >>>> +Platform: TABLET >>>> +Mic2Options: -f raw --save-kernel --arch=armv7hl >>>> +FileName: tablet-armv7hl-n900 >>>> +Kernel: kernel-adaptation-n900 >>>> +Architecture: armv7hl >>>> +Part: n900-2G >>>> +BootloaderTimeout: 0 >>>> +Groups: >>>> + - Nokia N900 Support >>>> + - Nokia N900 Proprietary Support >>>> + - Samples and Demos >>>> + - Tablet Applications >>>> +PostScripts: >>>> + - uxlaunch-nocursor >>>> + - uxlaunch-symlink-tb >>>> + - gconf >>>> + - tablet-copy >>>> + - sensor-rm >>>> + - bootchart >>>> + - fstab-n900 >>>> + - arch-armv7hl >>>> + - meego-ux-daemon-n900-homekey >>>> --- tablet/tablet-ia32-mfld.yaml >>>> +++ tablet/tablet-ia32-mfld.yaml >>>> @@ -9,6 +9,8 @@ >>>> Kernel: kernel-adaptation-medfield >>>> Schedule: "*" >>>> Architecture: ia32 >>>> +Repos: >>>> + - adobe >>>> Groups: >>>> - Medfield Support >>>> - Tablet Applications >>>> --- tablet/tablet-ia32-mrst.yaml >>>> +++ tablet/tablet-ia32-mrst.yaml >>>> @@ -10,6 +10,8 @@ >>>> FileName: tablet-ia32-mrst >>>> Kernel: kernel-adaptation-mrst >>>> Architecture: ia32 >>>> +Repos: >>>> + - adobe >>>> Groups: >>>> - Moorestown Support >>>> - Samples and Demos >>>> --- tablet/tablet-ia32-oaktrail.yaml >>>> +++ tablet/tablet-ia32-oaktrail.yaml >>>> @@ -1,21 +1,24 @@ >>>> - Name: Meego Tablet Oaktrail >>>> - Schedule: "*" >>>> - Active: True >>>> - Platform: TABLET >>>> - Mic2Options: -f livecd >>>> - BootloaderAppend: "quiet" >>>> - BootloaderTimeout: 0 >>>> - FileName: tablet-ia32-oaktrail >>>> - Kernel: kernel-adaptation-oaktrail >>>> - Architecture: ia32 >>>> - Groups: >>>> - - Oaktrail Support >>>> - - Samples and Demos >>>> - - Tablet Applications >>>> - PostScripts: >>>> - - flash >>>> - - uxlaunch-trail >>>> - - gconf >>>> - - tablet-copy >>>> - - sensor-rm >>>> - - camera-rm >>>> +Name: Meego Tablet Oaktrail >>>> +Schedule: "*" >>>> +Active: True >>>> +Platform: TABLET >>>> +Mic2Options: -f livecd >>>> +BootloaderAppend: "quiet" >>>> +BootloaderTimeout: 0 >>>> +FileName: tablet-ia32-oaktrail >>>> +Kernel: kernel-adaptation-oaktrail >>>> +Architecture: ia32 >>>> +Repos: >>>> + - adobe >>>> +Groups: >>>> + - Oaktrail Support >>>> + - Samples and Demos >>>> + - Tablet Applications >>>> +PostScripts: >>>> + - flash >>>> + - uxlaunch-nocursor >>>> + - uxlaunch-symlink-tb >>>> + - gconf >>>> + - tablet-copy >>>> + - sensor-rm >>>> + - camera-rm >>>> --- tablet/tablet-ia32-pinetrail.yaml >>>> +++ tablet/tablet-ia32-pinetrail.yaml >>>> @@ -8,6 +8,8 @@ >>>> FileName: tablet-ia32-pinetrail >>>> Kernel: kernel-adaptation-pinetrail >>>> Architecture: ia32 >>>> +Repos: >>>> + - adobe >>>> Groups: >>>> - Pinetrail Support >>>> - Samples and Demos >>>> @@ -17,7 +19,8 @@ >>>> - sensorfw-pegatron >>>> PostScripts: >>>> - flash >>>> - - uxlaunch-trail >>>> + - uxlaunch-nocursor >>>> + - uxlaunch-symlink-tb >>>> - gconf >>>> - sensor-copy >>>> - egalax >>>> --- tablet/tablet-ia32-qemu.yaml >>>> +++ tablet/tablet-ia32-qemu.yaml >>>> @@ -8,6 +8,8 @@ >>>> Kernel: kernel >>>> Schedule: "1" >>>> Architecture: ia32 >>>> +Repos: >>>> + - adobe >>>> Groups: >>>> - Pinetrail Support >>>> - Base Development >>>> >>>> _______________________________________________ >>>> MeeGo-commits mailing list >>>> [email protected] >>>> http://lists.meego.com/listinfo/meego-commits >>> >>> _______________________________________________ >>> MeeGo-packaging mailing list >>> [email protected] >>> http://lists.meego.com/listinfo/meego-packaging >>> > > _______________________________________________ MeeGo-packaging mailing list [email protected] http://lists.meego.com/listinfo/meego-packaging
