[Sugar-devel] Debian Advocacy for Sugar - Update

Chihurumnaya Ibiam ibiamchihurumnaya at gmail.com
Wed Jun 10 13:44:05 EDT 2020


The commit #930e1c
<https://salsa.debian.org/pkg-sugar-team/sugar-browse-activity/-/commit/930e1c139635b1d520d1f5464c676236690b3539>
applies patch
<https://github.com/sugarlabs/browse-activity/commit/ffdacaa572272c718c54f344a87dae9f0fb3223a>
to the package, the commit message said

Embedding the GSettings schema complicates generating during build.
This patch replicates the embedded schema as standalone file (partly
reverting git commit ffdacaa). . Additionally, this patch sprinkles
some indentation changes to the embedded schema, to ensure that if
upstream changes the schema then applying the patch will fail hard

The commit message says the commit was reverting ffdacaa
<https://salsa.debian.org/pkg-sugar-team/sugar-browse-activity/-/commit/ffdacaa572272c718c54f344a87dae9f0fb3223a>
while having the schema
embedded with some indentation changes "to ensure that if the upstream
changes the schema
then applying the patch will fail hard".

The actual issue isn't the commit changing the indentation as that's
not what's been addressed.
I've not been able to see any commit that change the lines, the error
says L192 of webactivity.py calls
TabbedView in browser.py and this is where the lines get mixed up.

Yes you'll have to investigate further why the schema can't be loaded
but also make sure the files in all 3 places correspond
- sugar, salsa, github -.



-- 
Ibiam Chihurumnaya
ibiamchihurumnaya at gmail.com



On Wed, Jun 10, 2020 at 5:26 PM shaansubbaiah.cs18 at bmsce.ac.in <
shaansubbaiah.cs18 at bmsce.ac.in> wrote:

> Hello James and Ibiam,
>
> I should have been more clearer in my mail, the diff posted in the
> previous mail was generated by using Python with the difflib library.
>
> I have now gone through the commits in the Salsa repository and have found
> the commit that has caused the line changes. I’m not completely sure what
> the commit message means, it appears the difference in the indentation of
> the embedded schema was used to ensure that the patch in the commit would
> fail if upstream changed the schema. Linked here, #930e1c
> <https://salsa.debian.org/pkg-sugar-team/sugar-browse-activity/-/commit/930e1c139635b1d520d1f5464c676236690b3539>
>
> Also, libglib2.0-bin had been added as a dependency in commit #c5717
> <https://salsa.debian.org/pkg-sugar-team/sugar-browse-activity/-/commit/c5717fc304d1e85707144ed9b18efe6f9d578c22>
> and had been later replaced by a GSettings schema
> (org.laptop.WebActivity.gschema.xml) located at
> /usr/share/glib-2.0/schemas/ . Linked here, #0977a
> <https://salsa.debian.org/pkg-sugar-team/sugar-browse-activity/-/commit/0977ac6923a63dc843aeb75f7caeaead20a257e5>,
> making Browse activity independent of libglib2.0-bin.
>
> I’ll have to investigate further as to why the Browse activity cannot
> detect the GSettings schema.
>
> Best,
> Shaan
>> On Wednesday, June 10, 2020, 12:16:21 PM GMT+5:30, James Cameron <
> quozl at laptop.org> wrote:
>
>
> Shaan,
>
> * the sources are different, and this is not unusual, and not a problem
> worth reporting unless an explanation cannot be found,
>
> * the commits in salsa explain the reason for every difference.
>
> Now that you have detected a difference, please make sure you read the
> commits, in all aspects;
>
> * the commit message,
>
> * the commit diff,
>
> * the context or ordering in which the diff occurs.
>
> The diff format must be learned to the point you can be comfortable to
> make a change by hand.  Not that you have to; tools like patch(1) can do it
> for you, but sometimes a diff is malformed when it is sent by other means
> than git.
>
> On Tue, Jun 09, 2020 at 06:29:10PM +0100, Chihurumnaya Ibiam wrote:
> > Thanks for the update.
> >
> > Both files in the Debian salsa repo and the github repo have matching
> lines and
> > I'm also guessing the file at
> > the installed /usr/share/activities/browseactivity will also have
> matching -
> > can you confirm if it does? - lines
> > but the traceback prints the wrong lines.
> >
> > --
> >
> > Ibiam Chihurumnaya
> > [1]ibiamchihurumnaya at gmail.com
> >
> > On Tue, Jun 9, 2020 at 6:02 PM [2]shaansubbaiah.cs18 at bmsce.ac.in <[3]
> > shaansubbaiah.cs18 at bmsce.ac.in> wrote:
> >
> >    Hello Ibiam,
> >
> >    Comparing browse.py in:
> >
> >      1. Github repository [4]
> https://github.com/sugarlabs/browse-activity/
> >      2. Debian Salsa repository [5]
> https://salsa.debian.org/pkg-sugar-team/
> >        sugar-browse-activity/-/blob/master/browser.py
> >      3. The installed activity at /usr/share/activities/browseactivity
> >
> >    The Github and Salsa versions have matching line numbers. Difference
> >    between the Github and installed versions:
> >
> >    --- bgit # browse.py in Github
> >    +++ bdeb # installed browse.py
> >    @@ -110,15 +110,13 @@
> >                      [
> >                          '<?xml version="1.0" encoding="UTF-8"?>',
> >                          '<schemalist>',
> >    -                    '<schema id="org.laptop.WebActivity" '
> >    -                    'path="/org/laptop/WebActivity/">',
> >    -                    '<key name="home-page" type="s">',
> >    -                    "<default>''</default>",
> >    -                    '<summary>Home page URL</summary>',
> >    -                    '<description>URL to show as default or when
> home button '
> >    -                    'is pressed.</description>',
> >    -                    '</key>',
> >    -                    '</schema>',
> >    +                    '    <schema id="org.laptop.WebActivity"
> path="/org/laptop/WebActivity/">',
> >    +                    '        <key name="home-page" type="s">',
> >    +                    "            <default>''</default>",
> >    +                    '            <summary>Home page URL</summary>',
> >    +                    '            <description>URL to show as default
> or when home button is pressed.</description>',
> >    +                    '        </key>',
> >    +                    '    </schema>',
> >                          '</schemalist>',
> >                      ]
> >                  open(os.path.join(path, src), 'w').writelines(lines)
> >
> >    Seems the only difference was the lines that was split into new lines
> have
> >    been concatenated to a single line.
> >
> >    Best,
> >    Shaan
> >    ​
> >    On Monday, 8 June, 2020, 10:19:48 pm IST, Chihurumnaya Ibiam <[6]
> >    ibiamchihurumnaya at gmail.com> wrote:
> >
> >    You might have to include it as a dependency for your next release as
> >    that's the latest version
> >    that Browse currently has, what doesn't sit well with me is the line
> >    difference,  I can't do any
> >    investigation at the moment but it'll be great if you took a look at
> the
> >    source code to confirm the line
> >    differences. Thanks.
> >
> >    --
> >
> >    Ibiam Chihurumnaya
> >    [7]ibiamchihurumnaya at gmail.com
> >
> >
> >
> >    On Mon, Jun 8, 2020 at 7:46 AM [8]shaansubbaiah.cs18 at bmsce.ac.in <[9]
> >    shaansubbaiah.cs18 at bmsce.ac.in> wrote:
> >
> >        Hello Ibiam,
> >        The version of the browse-activity installed was 205-2.
> >
> >        Sorry for the late reply, temporarily away from home, I'm on a
> >        different system testing Sugar and internet bandwidth isn't very
> high.
> >        Should be back in a day or two.
> >
> >        Best,
> >        Shaan
> >
> >        On Sunday, 7 June, 2020, 3:50:23 am IST, Chihurumnaya Ibiam <[10]
> >        ibiamchihurumnaya at gmail.com> wrote:
> >
> >        Thanks Shaan, what version of Browse are you running as the lines
> in
> >        the traceback are off
> >        from the latest version?
> >
> >        --
> >
> >        Ibiam Chihurumnaya
> >        [11]ibiamchihurumnaya at gmail.com
> >
> >
> >
> >        On Sat, Jun 6, 2020 at 6:58 PM [12]shaansubbaiah.cs18 at bmsce.ac.in
> <[13]
> >        shaansubbaiah.cs18 at bmsce.ac.in> wrote:
> >
> >            Hello Ibiam,
> >            Installing the libglib2.0-dev package fixes the issue. I have
> also
> >            just found that this issue is mentioned in the browse-activity
> >            readme '' Unusually, Browse also depends on
> glib-compile-schemas to
> >            compile a Gio.Settings schema. ''.
> >
> >            Online searches also return similar results, eg. :
> >            [14]https://askubuntu.com/questions/204643/
> >
> getting-missing-dependency-for-compilation-glib-compile-schemas
> >             and
> >            [15]https://askubuntu.com/questions/161739/
> >
> why-does-launchpad-fail-to-find-glib-compile-schemas-while-building-for-ppa
> >
> >            I'm fairly certain that this is the correct fix, however, I
> have
> >            not found libglib2.0-dev as a dependency of Browse Activity
> for the
> >            earlier releases of Debian at [16]
> https://packages.debian.org/
> >            search?keywords=sugar-browse-activity . So I do not know how
> this
> >            was handled in older releases or if it was handled at all.
> >
> >            Best,
> >            Shaan
> >            On Saturday, June 6, 2020, 10:47:34 PM GMT+5:30, Chihurumnaya
> Ibiam
> >            <[17]ibiamchihurumnaya at gmail.com> wrote:
> >
> >            Can you test to confirm if it's the problem?
> >
> >            --
> >
> >            Ibiam Chihurumnaya
> >            [18]ibiamchihurumnaya at gmail.com
> >
> >
> >
> >            On Sat, Jun 6, 2020 at 9:11 AM Shaan Subbaiah B C <[19]
> >            shaansubbaiah.cs18 at bmsce.ac.in> wrote:
> >
> >                Hello Ibiam,
> >                Great to know that the markdown formatted as html is more
> >                convenient.
> >
> >                I'm not entirely sure that the web activity issue is due
> to
> >                libglib2.0-dev no being present as a depenency but the
> error
> >                seems to be caused due to some header file missing that is
> >                fixed by installing that package.
> >
> >                I'll open the issues as you specified and test further.
> >
> >                Best,
> >                Shaan
> >
> >                On Sat, 6 Jun, 2020, 4:40 am Chihurumnaya Ibiam, <[20]
> >                ibiamchihurumnaya at gmail.com> wrote:
> >
> >                    Thanks for the update Shaan,
> >
> >                    The error you shared in 2) happens in #840 like you
> said
> >                    but the traceback in the logs is seen for the first
> time
> >                    and looks like it's a Python 3 port regression,
> kindly open
> >                    an issue in sugar so it can be tracked.
> >
> >                    Can you confirm that 3) is caused by libglib2.0-dev
> not
> >                    listed as a dependency in the control file?
> >
> >                    Could you open an issue for 4) in the write repo?
> >
> >                    I find the markdown rendered as HTML convenient and
> most of
> >                    it appears as [21]plaintext in the devel archive,
> >                    except the bullet points that start with a `-` above
> them.
> >
> >                    --
> >
> >                    Ibiam Chihurumnaya
> >                    [22]ibiamchihurumnaya at gmail.com
> >
> >
> >                    On Fri, Jun 5, 2020 at 10:08 AM [23]
> >                    shaansubbaiah.cs18 at bmsce.ac.in <[24]
> >                    shaansubbaiah.cs18 at bmsce.ac.in> wrote:
> >
> >                        I have tried to install sucrose on the standard
> Debian
> >                        Live ISO (it has no Desktop Environment, only
> CLI) as
> >                        there were some issues while trying to do so in
> the
> >                        Debian Live (KDE Plasma) ISO:
> >
> >                          ◎ Attempting to install sucrose v117-x after
> adding
> >                            the unstable repositories to sources.list and
> >                            setting the default release as stable by sudo
> apt
> >                            install -t unstable sucrose causes 100’s of
> >                            packages to be removed (Including all KDE
> >                            applications, Firefox, Konsole, etc) and
> initiates
> >                            the install of several 100 language packs for
> >                            Firefox, LibreOffice, etc.
> >
> >                          ◎ Installing by setting the default release as
> >                            unstable and sudo apt install sucrose causes
> the
> >                            same issue above.
> >
> >                          ◎ Setting default release as ‘stable’ and
> attempting
> >                            to install using sudo apt install
> sucrose/unstable
> >                            does not cause this issue but requires
> including
> >                            all the dependencies with /unstable postfix
> >                            (otherwise dependencies are fetched from the
> stable
> >                            repo itself). This is very messy, the Desktop
> >                            Environment and almost all useful preinstalled
> >                            applications get wiped, some dependencies
> aren’t
> >                            satisfied while testing Sugar after install.
> >
> >
> ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
> >
> >                        Install Debian Live Standard ISO
> >
> >                        [25]
> https://cdimage.debian.org/debian-cd/current-live/amd64/bt-hybrid/debian-live-10.4.0-amd64-standard.iso.torrent
> >
> >                        Set default package repository to unstable
> >
> >                        # /etc/apt/apt.conf.d/my-default-release
> >                        APT::Default-Release "unstable";
> >
> >                        Update, Install gcc-8-base, sucrose
> >
> >                        sudo apt update
> >                        sudo apt install gcc-8-base # select 'yes' on the
> prompt
> >                        sudo apt install sucrose
> >
> >                        Debian Live Standard doesn’t come with a Display
> >                        Manager by default, install lightdm
> >
> >                        sudo apt install lightdm
> >
> >                        Reboot, select Sugar and log in.
> >
> >
> ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
> >
> >                        On testing:
> >
> >                        1) Sugar launches. Again most activities work
> without
> >                        issues. Listed issues below.
> >
> >                        2) Error in shell.log
> >
> >                        1591278095.943871 ERROR dbus.proxies: Introspect
> error on
> :1.28:/org/laptop/Activity/376730b6f93a8aa9b2929c28e423b45b73f2b515:
> dbus.exceptions.DBusException: org.freedesktop.DBus.Error.NoReply: Message
> recipient disconnected from message bus without replying
> >                        1591278095.944801 ERROR root: set_active()
> failed: org.freedesktop.DBus.Error.ServiceUnknown: The name :1.28 was not
> provided by any .service files
> >                        1591278100.365706 ERROR dbus.proxies: Introspect
> error on
> :1.30:/org/laptop/Activity/376730b6f93a8aa9b2929c28e423b45b73f2b515:
> dbus.exceptions.DBusException: org.freedesktop.DBus.Error.NoReply: Message
> recipient disconnected from message bus without replying
> >                        1591278100.366105 ERROR root: set_active()
> failed: org.freedesktop.DBus.Error.ServiceUnknown: The name :1.30 was not
> provided by any .service files
> >                        Traceback (most recent call last):
> >                          File
> "/usr/lib/python3/dist-packages/jarabe/desktop/viewcontainer.py", line 69,
> in do_size_allocate
> >                            self._layout.allocate_children(allocation,
> self._children)
> >                          File
> "/usr/lib/python3/dist-packages/jarabe/desktop/favoriteslayout.py", line
> 250, in allocate_children
> >                            name_hash =
> hashlib.md5(child.get_bundle_id().decode())
> >                        AttributeError: 'str' object has no attribute
> 'decode'
> >
> >                        ^ Seems to be similar to Sugar issues #840 and
> #843
> >
> >                        3) Web Activity doesn’t launch, error:
> >
> >                        sh: 1: glib-compile-schemas: not found
> >                        Traceback (most recent call last):
> >                          File "/usr/bin/sugar-activity3", line 5, in
> <module>
> >                            activityinstance.main()
> >                          File
> "/usr/lib/python3/dist-packages/sugar3/activity/activityinstance.py", line
> 230, in main
> >                            instance =
> create_activity_instance(activity_constructor, activity_handle)
> >                          File
> "/usr/lib/python3/dist-packages/sugar3/activity/activityinstance.py", line
> 59, in create_activity_instance
> >                            activity = constructor(handle)
> >                          File
> "/usr/share/sugar/activities/Browse.activity/webactivity.py", line 192, in
> __init__
> >                            self._tabbed_view = TabbedView(self)
> >                          File
> "/usr/share/sugar/activities/Browse.activity/browser.py", line 145, in
> __init__
> >                            self.settings = _get_local_settings(activity)
> >                          File
> "/usr/share/sugar/activities/Browse.activity/browser.py", line 127, in
> _get_local_settings
> >                            source =
> Gio.SettingsSchemaSource.new_from_directory(path, None, True)
> >                        gi.repository.GLib.Error: g-file-error-quark:
> Failed to open file
> “/home/ssbc/.sugar/default/org.laptop.WebActivity/data/schemas/gschemas.compiled”:
> open() failed: No such file or directory (4)
> >                        Terminated by signal 11, pid 866 activity_id
> 376730b6f93a8aa9b2929c28e423b45b73f2b515
> >
> >                        ^ This occurred while using the other installation
> >                        methods as well. It was fixed by installing
> >                        libglib2.0-dev. This may be a packaging issue
> where
> >                        ‘libglib2.0-dev’ is not specified as a dependency
> for
> >                        the Web Activity?
> >
> >                        4) Write Activity crashes when trying to move a
> table.
> >                        ^ Reproduced by: Opening Words Activity -> Insert
> a
> >                        table -> Try dragging/resizing one of the inner
> row/
> >                        horizontal lines (Not the ones forming the outer
> >                        boundary) -> Crash
> >
> >                        Terminated by signal 11, pid 4715 activity_id
> 6305d02f3035c4afc03030e685fe338e6e394807
> >
> >
> ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
> >
> >                        I’m trying to keep a log/executed snippets in
> markdown
> >                        on a Github repository. Some files may not contain
> >                        anything useful, just snippets for future
> reference.
> >                        I’ll make it public after I clean it up a bit.
> >
> >                        I’ll post anything important through mail. I have
> used
> >                        the ‘Markdown Here’ extension to render markdown
> as
> >                        HTML. Let me know if you find this more
> convenient and
> >                        I’ll have to check how this appears in the Sugar
> Devel
> >                        Archive. If not suitable, I’ll revert back to
> markdown/
> >                        plaintext.
> >
> >                        ​
> >                        _______________________________________________
> >                        Sugar-devel mailing list
> >                        [26]Sugar-devel at lists.sugarlabs.org
> >                        [27]
> http://lists.sugarlabs.org/listinfo/sugar-devel
> >
> > References:
> >
> > [1] mailto:ibiamchihurumnaya at gmail.com
> > [2] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [3] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [4] https://github.com/sugarlabs/browse-activity/
> > [5]
> https://salsa.debian.org/pkg-sugar-team/sugar-browse-activity/-/blob/master/browser.py
> > [6] mailto:ibiamchihurumnaya at gmail.com
> > [7] mailto:ibiamchihurumnaya at gmail.com
> > [8] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [9] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [10] mailto:ibiamchihurumnaya at gmail.com
> > [11] mailto:ibiamchihurumnaya at gmail.com
> > [12] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [13] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [14]
> https://askubuntu.com/questions/204643/getting-missing-dependency-for-compilation-glib-compile-schemas
> > [15]
> https://askubuntu.com/questions/161739/why-does-launchpad-fail-to-find-glib-compile-schemas-while-building-for-ppa
> > [16] https://packages.debian.org/search?keywords=sugar-browse-activity
> > [17] mailto:ibiamchihurumnaya at gmail.com
> > [18] mailto:ibiamchihurumnaya at gmail.com
> > [19] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [20] mailto:ibiamchihurumnaya at gmail.com
> > [21]
> http://lists.sugarlabs.org/archive/sugar-devel/2020-June/058403.html
> > [22] mailto:ibiamchihurumnaya at gmail.com
> > [23] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [24] mailto:shaansubbaiah.cs18 at bmsce.ac.in
> > [25]
> https://cdimage.debian.org/debian-cd/current-live/amd64/bt-hybrid/debian-live-10.4.0-amd64-standard.iso.torrent
> > [26] mailto:Sugar-devel at lists.sugarlabs.org
>
> > [27] http://lists.sugarlabs.org/listinfo/sugar-devel
>
>
> --
> James Cameron
> http://quozl.netrek.org/
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sugarlabs.org/archive/sugar-devel/attachments/20200610/d1a69d95/attachment-0001.htm>


More information about the Sugar-devel mailing list