Slingshot

slingshot depends on gala gsettings schema

Bug #1248484 reported by Pim Vullers on 2013-11-06

This bug affects 8 people

Affects		Status	Importance	Assigned to	Milestone
	Slingshot	Fix Released	Low	Unassigned	Slingshot juno-beta1

Bug Description

Slingshot builds fine from trunk but fails to run:

pim@chaos ~ $ slingshot-launcher
[_LOG_LEVEL_INFO 11:53:58.533434] Application.vala:77: Slingshot version: 0.7.6
[_LOG_LEVEL_INFO 11:53:58.533577] Application.vala:79: Kernel version: 3.12.0-gentoo
[_LOG_LEVEL_WARN 11:53:58.541322] [GLib-GIO] Settings schema 'org.pantheon.desktop.gala.behavior' is not installed
Trace/breakpoint trap

Apparently due to not having gala installed. This creates a hard dependency on gala being available, (perhaps even running), that was not present in earlier versions. It would be nice if we could use slingshot also when gala is not present.

Tags:

Revision history for this message

Mazen Mardini (mazen-mardini) wrote on 2013-12-01:

I can reproduce this error with the latest version (0.7.6):

mazen@vaio:~$ slingshot-launcher
[_LOG_LEVEL_INFO 19:04:32.305988] Application.vala:77: Slingshot version: 0.7.6
[_LOG_LEVEL_INFO 19:04:32.306056] Application.vala:79: Kernel version: 3.11.0-13-generic
[_LOG_LEVEL_WARN 19:04:32.308037] [GLib-GIO] Settings schema 'org.pantheon.desktop.gala.behavior' is not installed

Revision history for this message

quequotion (quequotion) wrote on 2013-12-12:

I came across this in slingshot-launcher 0.7.6.1 after removing gala.

Revision history for this message

quequotion (quequotion) wrote on 2013-12-12:

There are a couple of these bugs at the moment. They wouldn't be noticed by users of Elementary OS: only users who have installed some /parts/ of the elementary suite will encounter them.

In this case, installing slingshot-launcher but not gala.

Revision history for this message

quequotion (quequotion) wrote on 2013-12-22:

Slingshot can run without gala; each of the Pantheon desktop components are quite modular!

Because of this bug, it is only necessary to /install/ gala.

Revision history for this message

Pim Vullers (pimvullers) wrote on 2013-12-22: Re: [Bug 1248484] Re: slingshot depends on gala gsettings schema

On 12/22/2013 05:49 PM, quequotion wrote:
> Slingshot can run without gala; each of the Pantheon desktop components
> are quite modular!
>
> Because of this bug, it is only necessary to /install/ gala.

That's true, but it is an undesired runtime dependency.

Revision history for this message

quequotion (quequotion) wrote on 2013-12-22:

This bug comes from code in src/Settings.vala:

http://bazaar.launchpad.net/~elementary-pantheon/slingshot/trunk/view/head:/src/Settings.vala

Is there a reason why this needs to be based on org.pantheon.desktop.gala.behavior specifically?

The two keys have very little in common aside from being gsettings and related to Pantheon.

quequotion (quequotion) on 2013-12-22

Changed in slingshot:
status:	New → Confirmed

Danielle Foré (danrabbit) on 2015-09-23

Changed in slingshot:
milestone:	none → loki-beta1

Cody Garver (codygarver) on 2015-11-30

Changed in slingshot:
importance:	Undecided → Medium

Cody Garver (codygarver) on 2016-02-12

Changed in slingshot:
importance:	Medium → Low

Danielle Foré (danrabbit) on 2016-02-14

tags:

added: technical-debt

Cody Garver (codygarver) on 2016-03-14

Changed in slingshot:
milestone:	loki-beta1 → loki+1-beta1

Revision history for this message

quequotion (quequotion) wrote on 2016-07-21:

Any progress on this/these?

There were quite a few strewn about the pantheon components' code last I checked.

Maybe the most simple strategy would be to check every "base" to make sure it uses a key that isn't related to elementary OS and will always be there, like one of the gnome keys.

Revision history for this message

quequotion (quequotion) wrote on 2016-10-22:

I've gotten a little more familiar with the slingshot code now. The reason it's checking gala's gsettings is to find the setting for the hotcorner. Could we implement a way for this to failsafe if gala's gsettings are not there?

Revision history for this message

David Hewitt (davidmhewitt) wrote on 2017-05-07:

Fixed in: https://github.com/elementary/applications-menu/pull/10