There is something wrong with Python3-Caja in upcoming Ubuntu MATE 21.10

[mate-desktop]

There is something wrong with Python3-Caja in upcoming Ubuntu MATE 21.10

[rbuj]

@flexiondotorg I will take a look. I have to create a new VM to debug and test the code on Ubuntu MATE 21.10. I haven't seen these log lines on Fedora.

[rbuj]

#67 may fix the issue, it didn't show any bug on Fedora, I will try it on Ubuntu ASAP...

[N0rbert]

In the live session (Ubuntu-MATE 21.10 "Impish Indri" - Beta amd64 (20210925)) running caja-file-management-properties from terminal and navigating to Extensions tab leads to the following terminal output:

$ caja-file-management-properties 
Initializing caja-open-terminal extension
Initializing caja-sendto extension
Initializing caja-wallpaper extension

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:49.682: Failed to set text from markup due to error parsing markup: Error on line 1 char 35: Invalid UTF-8 encoded text in name — not valid “Python: \xb0\xd5\xfb\u000b\xe7\u007f”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:49.682: Failed to set text from markup due to error parsing markup: Error on line 1 char 35: Invalid UTF-8 encoded text in name — not valid “Python: \xb0\xd5\xfb\u000b\xe7\u007f”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:49.686: Failed to set text from markup due to error parsing markup: Error on line 1 char 31: Invalid UTF-8 encoded text in name — not valid “Python: `\u00192~\xdcU”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:50.885: Failed to set text from markup due to error parsing markup: Error on line 1 char 35: Invalid UTF-8 encoded text in name — not valid “Python: \xb0\xd5\xfb\u000b\xe7\u007f”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:50.885: Failed to set text from markup due to error parsing markup: Error on line 1 char 35: Invalid UTF-8 encoded text in name — not valid “Python: \xb0\xd5\xfb\u000b\xe7\u007f”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:50.885: Failed to set text from markup due to error parsing markup: Error on line 1 char 35: Invalid UTF-8 encoded text in name — not valid “Python: \xb0\xd5\xfb\u000b\xe7\u007f”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:50.886: Failed to set text from markup due to error parsing markup: Error on line 1 char 35: Invalid UTF-8 encoded text in name — not valid “Python: \xb0\xd5\xfb\u000b\xe7\u007f”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:50.887: Failed to set text from markup due to error parsing markup: Error on line 1 char 31: Invalid UTF-8 encoded text in name — not valid “Python: `\u00192~\xdcU”

(caja-file-management-properties:5153): Gtk-WARNING **: 18:54:50.888: Failed to set text from markup due to error parsing markup: Error on line 1 char 31: Invalid UTF-8 encoded text in name — not valid “Python: `\u00192~\xdcU”

The above output is not expected.

[flexiondotorg]

@rbuj Reverting this commit (from #58) fixes this issue:

• 3ad9a5d

[flexiondotorg]

@rbuj Patched versions (that commit reverted) are published in this PPA for Focal, Hirsute and Impish

• https://launchpad.net/~ubuntu-mate-dev/+archive/ubuntu/qa-team

My testing confirms the issue is resolved and also the extensions list is correctly displayed now.

[cwendling]

@rbuj Reverting this commit (from #58) fixes this issue:

* [3ad9a5d](https://github.com/mate-desktop/python-caja/commit/3ad9a5d5e71736632ad979557c6f687596d24992)

Yeah this commit is buggy, because caja_python_load_file() while actually having a const char* argument keeps a reference in all_pyfiles without making a copy, so it must not be freed. It also discards the const qualifier in the PyImport_ImportModuleEx() call, not sure what the implications are there.

[rbuj]

I may be wrong, but at first glance a g_strdup (filename) seems to be missing at the line

python-caja/src/caja-python.c

Line 133 in 94538f8

all_pyfiles = g_list_append(all_pyfiles, (gchar*)filename);

[rbuj]

@flexiondotorg python extensions seem to work properly after applying #67 on Ubuntu MATE 21.10.

[flexiondotorg]

Thanks for working on this. I'll test it later 👍