After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 790147 - build: Drop data-to-c.pl in favour of data-to-c.py
build: Drop data-to-c.pl in favour of data-to-c.py
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: build
unspecified
Other All
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2017-11-10 00:55 UTC by Philip Withnall
Modified: 2017-11-10 11:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
build: Drop data-to-c.pl in favour of data-to-c.py (3.27 KB, patch)
2017-11-10 00:55 UTC, Philip Withnall
committed Details | Review

Description Philip Withnall 2017-11-10 00:55:44 UTC
Simple patch attached.
Comment 1 Philip Withnall 2017-11-10 00:55:48 UTC
Created attachment 363318 [details] [review]
build: Drop data-to-c.pl in favour of data-to-c.py

The Python version was added for the Meson build, but we might as well
use it from autotools too, since it does exactly the same thing as the
Perl version (modulo not including a trailing linebreak, but that
doesn’t matter).

Works fine with Python 2.7 or Python 3.

Signed-off-by: Philip Withnall <withnall@endlessm.com>
Comment 2 Emmanuele Bassi (:ebassi) 2017-11-10 11:05:26 UTC
Review of attachment 363318 [details] [review]:

Looks good to me
Comment 3 Philip Withnall 2017-11-10 11:50:04 UTC
Cheers! Just gen-unicode-tables.pl, tests/gen-casefold-txt.pl and tests/gen-casemap-txt.pl to go now.

Attachment 363318 [details] pushed as 1897e66 - build: Drop data-to-c.pl in favour of data-to-c.py