search

kergoth / vim-bitbake

Vim files for the BitBake tool
MIT License
61 stars 36 forks source link

Parse function names with nested vars #4

Closed staticshock closed 11 years ago

staticshock commented 11 years ago

This takes care of parsing ${PN} in shell function names, but not in python function names. It should be extended to python if that's allowed, but I grepped through meta-oe and found no python functions that actually make use of it.

This fixes #3

kergoth commented 11 years ago

It is allowed in python function names (python foo () {} syntax, anyway, not 'def ...' syntax) as well, so we should alter that pattern too. See https://github.com/openembedded/bitbake/blob/master/lib/bb/parse/parse_py/BBHandler.py#L41 -- the same regex is used for both python and non-python functions. Note that ${} isn't allowed in python functions defined with 'def ...', however. Please also alter the python pattern and resubmit, if you would, Thanks for the contribution and issue reports!

staticshock commented 11 years ago

This does it.

kergoth commented 11 years ago

Thanks, sorry for the delay!