Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 372223 - app-editors/vim-core: Please update bindzone.vim
Summary: app-editors/vim-core: Please update bindzone.vim
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Jim Ramsay (lack) (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-19 01:58 UTC by Dan Wallis
Modified: 2011-08-03 01:12 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dan Wallis 2011-06-19 01:58:04 UTC
This may be better sent upstream, and if so, please let me know so that I might be able to keep searching for whom to send this request.

Please can line 39 of /usr/share/vim/vim73/syntax/bindzone.vim of app-editors/vim-core be updated? According to the list on wikipedia[0], HINFO is deprecated, and I'd like to see SPF & DNAME added to the list.

[0] http://en.wikipedia.org/wiki/List_of_DNS_record_types

Something like this might be sufficient, or should this list be the full list as per wikipedia link above?



fredden@parakeet /usr/share/vim/vim73/syntax $ diff -u bindzone.vim ~ 
--- bindzone.vim	2011-06-19 12:40:44.000000000 +1200
+++ /home/fredden/bindzone.vim	2011-06-19 13:56:47.000000000 +1200
@@ -36,7 +36,7 @@
 syn match       zoneSpecial     contained /^[@*.]\s/
 syn match       zoneTTL         contained /\<\d[0-9HhWwDd]*\>/  nextgroup=zoneClass,zoneRRType skipwhite
 syn keyword     zoneClass       contained IN CHAOS              nextgroup=zoneRRType,zoneTTL   skipwhite
-syn keyword     zoneRRType      contained A AAAA CNAME HINFO MX NS PTR SOA SRV TXT nextgroup=zoneRData skipwhite
+syn keyword     zoneRRType      contained A AAAA CNAME DNAME MX NS PTR SOA SPF SRV TXT nextgroup=zoneRData skipwhite
 syn match       zoneRData       contained /[^;]*/ contains=zoneDomain,zoneIPAddr,zoneIP6Addr,zoneText,zoneNumber,zoneParen,zoneUnknown
 
 syn match       zoneIPAddr      contained /\<[0-9]\{1,3}\(\.[0-9]\{1,3}\)\{,3}\>/
Comment 1 Pacho Ramos gentoo-dev 2011-06-28 09:16:22 UTC
Do you need the same for vim-core-7.3.219?
Comment 2 Dan Wallis 2011-06-29 03:04:27 UTC
Same as? Yes, it would be nice if the current version(s) in the tree can be revision-bumped to include this, but I'm happy to wait for the next version to come through.
Comment 3 Jim Ramsay (lack) (RETIRED) gentoo-dev 2011-07-15 00:50:24 UTC
I'd suggest first contacting the maintainer of the script:
  Maintainer:   Julian Mehnle <julian@mehnle.net>
  URL:          http://www.mehnle.net/source/odds+ends/vim/syntax/

As of vim-core-7.3.244 I've finally started incorporating updates to the runtime files for vim, but bindzone.vim is still at the 2006 version you can find at the URL above.  If you can get the change in there and are patient enough to wait for it to cycle from there to vim.org and then into a gentoo release, it will eventually appear.

I'll also try to put this on my list of things-to-add to the next release.
Comment 4 Dan Wallis 2011-07-18 00:32:10 UTC
Patch send upstream and accepted there.
Comment 5 Dan Wallis 2011-07-18 21:27:35 UTC
Bram (the vim.org maintainer) says he'll include the update.
Comment 6 Jim Ramsay (lack) (RETIRED) gentoo-dev 2011-08-03 01:12:00 UTC
I'm happy to report that Bram did indeed include the update, and my recent bump to [g]vim-7.3.266 includes this updated runtime.

Enjoy!