Dextrose/3/Patches State: Difference between revisions

(11 intermediate revisions by 4 users not shown)

Line 1:

~~In eduJam Rafael, Reuben~~ and ~~Gonzalo did a review of pending patches~~:

{{Obsolete|Moved and died at https://sugardextrose.org/projects/dextrose/wiki/Patches_state}}

Resources:

Line 6:

http://dev.activitycentral.com/issues/18

Activity Central Priority Patches list & Status: [[Dextrose/3/ACPriorityPatches]]

= Ready to upstream =

Line 34:

Line 36:

http://git.sugarlabs.org/dextrose/mainline/blobs/bleeding-edge/rpms/sugar/0040-Globalkey-for-touchpad-device-icon.patch

~~[anish~~, ~~ready, need to publish on sugar-devel]~~

* Assigned to: Anish (m_anish)

* Blockers: None

* Current status: Patch coming soon (tested, ported)

= To do while GTK3 port =

Line 45:

Line 50:

http://git.sugarlabs.org/dextrose/mainline/blobs/bleeding-edge/rpms/sugar/0045-Clipboard-menu-off-screen-2201.patch

~~[ajay will work~~ in ~~gtk3 port]~~

* Assigned to: Anish (m_anish)

* Current status: Resend patch with typo in description fixed

* Link: https://bugs.sugarlabs.org/ticket/2201

* Blockers: None

== Set default scaling to 100 (unknown author) ==

in GTK3 is going to be possible to set scale with other units than pixels. like CSS. We must see Gnome bugs..and ask Carlos Garnacho.

~~[anish?]~~

* Assigned to: Anish (m_anish)

* Blockers/Current status: Downstream patch is a hack, upstream needs to be a proper fix, for which discussions with gnome need to happen. As for the current patch, I'll post it upstream, but it would be understandable if its rejected. Further research and follow up with gnome is outside the scope of upstreaming this particular patch.

== Add font dpi schema (Jorge Saldivar) ==

Needs to be ~~analized~~ for post - GTK3 and propose a features page.

Needs to be analyzed for post - GTK3 and propose a features page.

Old feature page http://wiki.sugarlabs.org/go/Features/Font_configuration

~~[anish]~~

* Assigned to: Anish (m_anish)

* Current status: Need to review whether to include in future dextrose. (Needs some testing, after which it may be dropped)

* Quoting Sascha <silbe@activitycentral.com> verbatim:

Used in combination with sugar-settings-manager. Should probably have never been in sugar, but rather in sugar-settings-manager. sugar-settings-manager apparently didn't work reliably and was dropped from Dextrose 2, never shipped in Dextrose 3. We should check whether we still need this (for configuring global default font settings) in GTK 3 world and try to use upstream gnome-settings-daemon rather than duplicating its functionality.

= Need design reviews =

Line 66:

Line 81:

http://git.sugarlabs.org/dextrose/mainline/blobs/bleeding-edge/rpms/sugar/0042-Database-support-for-3G-control-panel.patch

~~[anish. Ready for upstream after design review]~~

* Assigned to: Anish (m_ainsh)

* Current status/Blockers: This probably requires porting some of the code in model.py to be NM 0.9 compliant. A simple port of this patch to mainline didnt work out of the box (preliminary attempt)

== Notification enhancements (Martin Abente) ==

Line 106:

Line 122:

Does this needs CPU cycles, suspend resume behavior?, Using it only with the fram is open.

~~[anish~~: ~~checkout design discussions, can we upstream?]~~

* Assigned to: Anish (m_anish)

* CUrrent status: Not ported

* Blockers:

** Design issues. Fixing the sad face

** Heuristic issues. The heuristic needs to be ARM compliant.

** Disagreement on whether such numbers should be made visible in the fashion they are with this implementation.

* Action: Check and update feature page.

== Operations over multiple files in the journal ==

Line 157:

Line 178:

Bug reference: https://dev.laptop.org.au/issues/1028

[Ajay?]

Assigned to : Ajay

[DISCUSS how to implement this]

= Bugs to solve =

@@ Line 1: / Line 1: @@
-In eduJam Rafael, Reuben and Gonzalo did a review of pending patches:
+{{Obsolete|Moved and died at https://sugardextrose.org/projects/dextrose/wiki/Patches_state}}
 Resources:
@@ Line 6: / Line 6: @@
 http://dev.activitycentral.com/issues/18
+Activity Central Priority Patches list & Status: [[Dextrose/3/ACPriorityPatches]]
 = Ready to upstream =
@@ Line 34: / Line 36: @@
 http://git.sugarlabs.org/dextrose/mainline/blobs/bleeding-edge/rpms/sugar/0040-Globalkey-for-touchpad-device-icon.patch
-[anish, ready, need to publish on sugar-devel]
+* Assigned to: Anish (m_anish)
+* Blockers: None
+* Current status: Patch coming soon (tested, ported)
 = To do while GTK3 port =
@@ Line 45: / Line 50: @@
 http://git.sugarlabs.org/dextrose/mainline/blobs/bleeding-edge/rpms/sugar/0045-Clipboard-menu-off-screen-2201.patch
-[ajay will work in gtk3 port]
+* Assigned to: Anish (m_anish)
+* Current status: Resend patch with typo in description fixed
+* Link: https://bugs.sugarlabs.org/ticket/2201
+* Blockers: None
 ==  Set default scaling to 100 (unknown author) ==
 in GTK3 is going to be possible to set scale with other units than pixels. like CSS. We must see Gnome bugs..and ask Carlos Garnacho.
-[anish?]
+* Assigned to: Anish (m_anish)
+* Blockers/Current status: Downstream patch is a hack, upstream needs to be a proper fix, for which discussions with gnome need to happen. As for the current patch, I'll post it upstream, but it would be understandable if its rejected. Further research and follow up with gnome is outside the scope of upstreaming this particular patch.
 == Add font dpi schema (Jorge Saldivar) ==
-Needs to be analized for post - GTK3 and propose a features page.
+Needs to be analyzed for post - GTK3 and propose a features page.
 Old feature page http://wiki.sugarlabs.org/go/Features/Font_configuration
-[anish]
+* Assigned to: Anish (m_anish)
+* Current status: Need to review whether to include in future dextrose. (Needs some testing, after which it may be dropped)
+* Quoting Sascha <silbe@activitycentral.com> verbatim:
+Used in combination with sugar-settings-manager. Should probably have never been in sugar, but rather in sugar-settings-manager. sugar-settings-manager apparently didn't work reliably and was dropped from Dextrose 2, never shipped in Dextrose 3. We should check whether we still need this (for configuring global default font settings) in GTK 3 world and try to use upstream gnome-settings-daemon rather than duplicating its functionality.
 = Need design reviews =
@@ Line 66: / Line 81: @@
 http://git.sugarlabs.org/dextrose/mainline/blobs/bleeding-edge/rpms/sugar/0042-Database-support-for-3G-control-panel.patch
-[anish. Ready for upstream after design review]
+* Assigned to: Anish (m_ainsh)
+* Current status/Blockers: This probably requires porting some of the code in model.py to be NM 0.9 compliant. A simple port of this patch to mainline didnt work out of the box (preliminary attempt)
 == Notification enhancements (Martin Abente) ==
@@ Line 106: / Line 122: @@
 Does this needs CPU cycles, suspend resume behavior?, Using it only with the fram is open.
-[anish: checkout design discussions, can we upstream?]
+* Assigned to: Anish (m_anish)
+* CUrrent status: Not ported
+* Blockers:
+** Design issues. Fixing the sad face
+** Heuristic issues. The heuristic needs to be ARM compliant.
+** Disagreement on whether such numbers should be made visible in the fashion they are with this implementation.
+* Action: Check and update feature page.
 == Operations over multiple files in the journal ==
@@ Line 157: / Line 178: @@
 Bug reference: https://dev.laptop.org.au/issues/1028
-[Ajay?]
+Assigned to : Ajay
 [DISCUSS how to implement this]
 = Bugs to solve =