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 752292 - Add new syntax for The Meson Build System
Add new syntax for The Meson Build System
Status: RESOLVED FIXED
Product: gtksourceview
Classification: Platform
Component: Syntax files
unspecified
Other All
: Normal normal
: ---
Assigned To: GTK Sourceview maintainers
GTK Sourceview maintainers
Depends on:
Blocks:
 
 
Reported: 2015-07-12 17:12 UTC by Igor Gnatenko
Modified: 2015-07-23 11:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
language-specs: new highlight for meson (4.95 KB, patch)
2015-07-12 17:20 UTC, Igor Gnatenko
none Details | Review
language-specs: new highlight for meson (5.92 KB, patch)
2015-07-19 09:38 UTC, Igor Gnatenko
none Details | Review
language-specs: new highlight for meson (6.17 KB, patch)
2015-07-23 11:21 UTC, Igor Gnatenko
committed Details | Review

Description Igor Gnatenko 2015-07-12 17:12:18 UTC
Meson is an open source build system meant to be both extremely fast,
and, even more importantly, as user friendly as possible.

Currently meson highlight is a bit simplified, but in the future we are
planning to make it much better ;)

Reference: http://mesonbuild.com/
Language Reference: https://github.com/mesonbuild/meson/wiki/Reference-manual/cb3ef7a131afe067f56143e0c0f24817b3617315
Signed-off-by: Igor Gnatenko <ignatenko@src.gnome.org>
Comment 1 Igor Gnatenko 2015-07-12 17:20:23 UTC
Created attachment 307314 [details] [review]
language-specs: new highlight for meson

Meson is an open source build system meant to be both extremely fast,
and, even more importantly, as user friendly as possible.

Currently meson highlight is a bit simplified, but in the future we are
planning to make it much better ;)

The Meson Build System: http://mesonbuild.com/
Language Reference: https://github.com/mesonbuild/meson/wiki/Reference-manual/cb3ef7a131afe067f56143e0c0f24817b3617315
Reference: https://bugzilla.gnome.org/show_bug.cgi?id=752292
Signed-off-by: Igor Gnatenko <ignatenko@src.gnome.org>
Comment 2 Sébastien Wilmet 2015-07-17 09:19:46 UTC
Review of attachment 307314 [details] [review]:

Looks good, but you need to update POTFILES.in too.
Also, it's better to have an example in tests/syntax-highlighting/.
Comment 3 Igor Gnatenko 2015-07-19 09:38:35 UTC
Created attachment 307668 [details] [review]
language-specs: new highlight for meson

Meson is an open source build system meant to be both extremely fast,
and, even more importantly, as user friendly as possible.

Currently meson highlight is a bit simplified, but in the future we are
planning to make it much better ;)

v2: add meson.lang to POTFILES.in and add example file for testing

The Meson Build System: http://mesonbuild.com/
Language Reference: https://github.com/mesonbuild/meson/wiki/Reference-manual/cb3ef7a131afe067f56143e0c0f24817b3617315
Reference: https://bugzilla.gnome.org/show_bug.cgi?id=752292
Signed-off-by: Igor Gnatenko <ignatenko@src.gnome.org>
Comment 4 Sébastien Wilmet 2015-07-19 12:34:35 UTC
Review of attachment 307668 [details] [review]:

A language definition file shipped by GtkSourceView needs to have a mimetype defined in the shared-mime-info database. Sorry to not have spotted it at the first review.
Comment 5 Igor Gnatenko 2015-07-19 13:07:10 UTC
(In reply to Sébastien Wilmet from comment #4)
> Review of attachment 307668 [details] [review] [review]:
> 
> A language definition file shipped by GtkSourceView needs to have a mimetype
> defined in the shared-mime-info database. Sorry to not have spotted it at
> the first review.
okay, I will take it ;)
Comment 6 Jussi Pakkanen 2015-07-19 21:52:48 UTC
The patch to add this has been submitted to fdo bugzilla here: https://bugs.freedesktop.org/show_bug.cgi?id=91397
Comment 7 Igor Gnatenko 2015-07-23 10:23:27 UTC
http://cgit.freedesktop.org/xdg/shared-mime-info/commit/?id=962056d5f178cd0b2fc3fc3788058b7dfe878928

Done in upstream shared-mime-info database :)
Comment 8 Sébastien Wilmet 2015-07-23 10:58:16 UTC
Cool! Can you add the mimetype to the lang file?
Comment 9 Igor Gnatenko 2015-07-23 11:21:21 UTC
Created attachment 307982 [details] [review]
language-specs: new highlight for meson

Meson is an open source build system meant to be both extremely fast,
and, even more importantly, as user friendly as possible.

Currently meson highlight is a bit simplified, but in the future we are
planning to make it much better ;)

v2: add meson.lang to POTFILES.in and add example file for testing

v3: add text/x-meson to mimetypes and add meson_options.txt to matching files

The Meson Build System: http://mesonbuild.com/
Language Reference: https://github.com/mesonbuild/meson/wiki/Reference-manual/cb3ef7a131afe067f56143e0c0f24817b3617315
Related: http://cgit.freedesktop.org/xdg/shared-mime-info/commit/?id=962056d5f178cd0b2fc3fc3788058b7dfe878928
Reference: https://bugzilla.gnome.org/show_bug.cgi?id=752292
Signed-off-by: Igor Gnatenko <ignatenko@src.gnome.org>
Comment 10 Sébastien Wilmet 2015-07-23 11:26:48 UTC
Review of attachment 307982 [details] [review]:

Looks good.
Comment 11 Igor Gnatenko 2015-07-23 11:35:44 UTC
Attachment 307982 [details] pushed as f8b78d5 - language-specs: new highlight for meson