Ticket #1692 (new defect)

Opened 3 years ago

Last modified 3 years ago

Add sugar-settings-manager

Reported by: sayamindu Owned by: tomeu
Priority: Unspecified by Maintainer Milestone: 0.90
Component: sugar Version: Unspecified
Severity: Unspecified Keywords: r!
Cc: sdz, erikos Distribution/OS: Unspecified
Bug Status: New

Description

The code for sugar-settings-manager lives in
 http://git.sugarlabs.org/projects/sugar-settings-manager

Patch for starting up the manager:

diff --git a/bin/sugar.in b/bin/sugar.in
index 498b573..813b189 100644
--- a/bin/sugar.in
+++ b/bin/sugar.in
@@ -36,6 +36,8 @@ while [ $# -ne 0 ] ; do
     shift
 done
 
+sugar-settings-manager &
+
 if test -z "$SUGAR_SCALING"; then
     export SUGAR_SCALING=72
 fi

Attachments

sugar-jhbuild-settings-manager.patch Download (0.9 KB) - added by sayamindu 3 years ago.
Patch for sugar-jhbuild
sugar-toolkit-dont-configure-fonts.patch.gz Download (450 bytes) - added by sayamindu 3 years ago.
Patch for sugar-toolkit

Change History

Changed 3 years ago by sayamindu

Patch for sugar-jhbuild

  Changed 3 years ago by sayamindu

  • keywords r? added

  Changed 3 years ago by sayamindu

Patch against sugar-toolkit is at  http://pastebin.be/23210 (Trac won't let me attach it since it thinks it is spam)

  Changed 3 years ago by tomeu

  • keywords r! added; r? removed

I think that we should have a release of settings-manager before sugar can depend on it. Also, are you going to maintain that module in sugar? Or as an external dependency?

  Changed 3 years ago by sayamindu

I can do a release immediately. Which makes more sense - as a submodule inside sugar, or as an external project.

Changed 3 years ago by sayamindu

Patch for sugar-toolkit

follow-up: ↓ 6   Changed 3 years ago by sdz

  • cc sdz, erikos added

What's the best way to proceed here? We got it reviewed and built in Fedora so far. So should we temporarily apply these two patches?

in reply to: ↑ 5   Changed 3 years ago by tomeu

Replying to sdz:

What's the best way to proceed here? We got it reviewed and built in Fedora so far. So should we temporarily apply these two patches?

If you really want, sure. But the reason why upstream Sugar is not depending on the settings-manager yet is because we don't have any UI yet for changing the font size, so it wouldn't bring us any benefit. But it may land in a later 0.88 minor release if a deployment asks strongly for it.

  Changed 3 years ago by erikos

  • status_field changed from Unconfirmed to New
  • milestone changed from Unspecified by Release Team to 0.90

I move this to 0.90. If a deployment cares strongly for this to be present, we can move it to 0.88.x. Please comment in this ticket, if.

Note: See TracTickets for help on using tickets.