FS#69780 - [nginx] Vim syntax file uses `hi link` instead of `hi def link`

Attached to Project: Arch Linux
Opened by Chris Morgan (ChrisMorgan) - Thursday, 25 February 2021, 12:21 GMT
Last edited by Giancarlo Razzolini (grazzolini) - Tuesday, 20 April 2021, 16:45 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Giancarlo Razzolini (grazzolini)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Package: nginx 1.18.0-2
File: /usr/share/vim/vimfiles/syntax/nginx.vim

Lines 2316–2336 start with `hi link`, but should start with `hi def link`. Otherwise I can’t set the highlighting in my colorscheme file, because the syntax file which is loaded after it clobbers it.

See ':highlight-default' in the Vim help for more information and explanation.

Attached is a patch that just adds `def` on each line that needs it.
This task depends upon

Closed by  Giancarlo Razzolini (grazzolini)
Tuesday, 20 April 2021, 16:45 GMT
Reason for closing:  Upstream
Additional comments about closing:  Upstream has fixed this bug
Comment by Giancarlo Razzolini (grazzolini) - Thursday, 25 February 2021, 16:21 GMT
That file comes from upstream. I think this would be better filled as a bug against upstream.
Comment by Chris Morgan (ChrisMorgan) - Thursday, 25 February 2021, 16:25 GMT
Ah hah, found it at http://hg.nginx.org/nginx/file/tip/contrib/vim/syntax/nginx.vim. Sorry, for some bizarre reason I just assumed this would be Arch-specific rather than having come from nginx. I’ll figure out how to file it upstream.
Comment by Giancarlo Razzolini (grazzolini) - Thursday, 25 February 2021, 16:57 GMT
No problem. You can add the upstream bug here too, and keep this open.
Comment by Chris Morgan (ChrisMorgan) - Thursday, 25 February 2021, 16:58 GMT

Loading...