Bug 128341 - python3 support for cs_CZ/thesaurus/dictionary-to-thesaurus.py
Summary: python3 support for cs_CZ/thesaurus/dictionary-to-thesaurus.py
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Linguistic (show other bugs)
Version:
(earliest affected)
6.3.1.2 release
Hardware: All All
: high enhancement
Assignee: Mattia Rizzolo
URL:
Whiteboard: target:6.4.0
Keywords:
Depends on:
Blocks:
 
Reported: 2019-10-23 12:10 UTC by Mattia Rizzolo
Modified: 2019-10-24 15:58 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:

Attachments
python3.patch (2.10 KB, patch)
2019-10-23 12:10 UTC, Mattia Rizzolo 		Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mattia Rizzolo 2019-10-23 12:10:44 UTC 
Created attachment 155255 [details]
python3.patch

I'm not sure where this script is coming from, but looking at the header I think its upstream is directly in the libreoffice-dictionaries repo.

I'm attaching a very simple (tested) patch that adds python3 support to that script; if you prefer not to change the shebang for now that's also fine with me.
Comment 1 Xisco Faulí 2019-10-23 13:41:39 UTC 
Hello Mattia,
Thanks for reporting this issue.
Could you please submit the patch to gerrit as described here? < https://wiki.documentfoundation.org/Development/gerrit/SubmitPatch >
Comment 2 Mattia Rizzolo 2019-10-23 16:20:27 UTC 
I'm one of those that doesn't like such overly complicated procedures for such simple stuff but guess it doesn't harm to set me up.

that's https://gerrit.libreoffice.org/#/c/81410/ (+ 81411 for random style improvements to the script, while I was at it).

I also sent my license CLA to libreoffice@fdo, but it has yet to appear in the archive.
Please tell me if there are any step that I missed in the process.