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 684200 - YAML syntax file
YAML syntax file
Status: RESOLVED FIXED
Product: gtksourceview
Classification: Platform
Component: Syntax files
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GTK Sourceview maintainers
GTK Sourceview maintainers
Depends on:
Blocks:
 
 
Reported: 2012-09-17 11:07 UTC by Fabian Deutsch
Modified: 2014-02-05 15:12 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Language file for YAML (8.98 KB, patch)
2012-09-18 14:20 UTC, Fabian Deutsch 		none Details | Review
Updated patch (9.08 KB, patch)
2012-09-18 14:33 UTC, Fabian Deutsch 		none Details | Review

Description Fabian Deutsch 2012-09-17 11:07:36 UTC 
There is a yaml syntax file floating in the net, would be nice if this could be kept upstream

https://code.google.com/p/masood/source/browse/#svn%2Ftrunk%2Fyaml-language-spec
Comment 1 Fabian Deutsch 2012-09-17 11:34:37 UTC 
The upstream syntax file is GPLv3+, gtksourceview is lgplv2+, thus the author needs to re or dual license the syntax file.
Comment 2 Fabian Deutsch 2012-09-18 14:20:31 UTC 
Created attachment 224635 [details] [review]
Language file for YAML

Masood, the author of the syntax file, agreed to change the license. Thanks again Masood.

The following patch adds the language file.
Comment 3 Paolo Borelli 2012-09-18 14:30:35 UTC 
As I was telling Fabian on irc, we are now string frozen so this will have to wait a bit.

Thanks for relicensing though
Comment 4 Fabian Deutsch 2012-09-18 14:33:25 UTC 
Created attachment 224636 [details] [review]
Updated patch
Comment 5 Cristian Ciupitu 2013-04-08 16:23:40 UTC 
Any news on this?
Comment 6 Fabian Deutsch 2013-04-10 16:04:32 UTC 
note from nacho:
classes for spelling are missing.

see i.e https://git.gnome.org/browse/gtksourceview/tree/data/language-specs/c.lang#n303
Comment 7 Fabian Deutsch 2013-08-12 12:59:14 UTC 
Hey, news on this?
Comment 8 Adam Dingle 2013-11-09 12:02:32 UTC 
I'd love to see this land.  Can I help in any way?
Comment 9 Paolo Borelli 2013-11-10 15:58:21 UTC 
The comment pointed out by Nacho about addding the classes to denote whether a context should be spell checked or not are still valid.

If that gets fixed feel free to push to master
Comment 10 Masood Behabadi 2013-11-12 11:20:28 UTC 
I have added classes for spelling. Updated source is available from https://github.com/dentcat/gtksourceview-yaml
Comment 11 Adam Dingle 2013-11-16 13:08:46 UTC 
Thanks, Masood!  I updated Fabian's patch with Masood's latest version and pushed to master.
Comment 12 Adam Dingle 2013-11-16 13:15:09 UTC 
It would be nice to push this to the 3.10 branch too, but I assume that's not possible since this new syntax file contains translatable strings (it's in POTFILES.in).  Is that correct?
Comment 13 Paolo Borelli 2013-11-16 13:26:19 UTC 
yup, that's correct