Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upload map.vim and map_fold.vim to vim runtime #990

Open
Freed-Wu opened this issue Dec 28, 2024 · 4 comments
Open

Upload map.vim and map_fold.vim to vim runtime #990

Freed-Wu opened this issue Dec 28, 2024 · 4 comments
Milestone

Comments

@Freed-Wu
Copy link

Freed-Wu commented Dec 28, 2024

Upload

to vim runtime to make it out of box. and complete syntax for mapserver config https://mapserver.gis.umn.edu/de/mapfile/config.html

TIA!

@jmckenna
Copy link
Member

@Freed-Wu I'm not quite understanding. I assume that you want to change the VIM Syntax steps in the document at https://mapserver.org/development/editing/vim.html

Updates on those steps would indeed be warmly accepted here. Are you able to create a Pull Request for your changes? The source of that file lives here.

Or, please explain more clearly what changes you need.

@jmckenna jmckenna added this to the 8.6.0 Release milestone Jan 16, 2025
@Freed-Wu
Copy link
Author

Ust upload these files to vim runtime. Not modification.

@ejn
Copy link
Contributor

ejn commented Jan 17, 2025

I assume @Freed-Wu means that the Mapserver VIM-Syntaxfiles should be added (via Pull-Request) to the official VIM repo at https://github.com/vim/vim/tree/master/runtime/syntax so that they are included in the standard VIM-Runtime bundle and don't require additional installation.

Sounds in principle like a good idea, but would mean that they would need to be kept up-to-date (or possibly even multiple versions?), which obviously isn't a problem if they are distributed via MapServer as everyone is then responsible for ensuring they have the correct version for their own MapServer version..

@ejn
Copy link
Contributor

ejn commented Jan 17, 2025

Edit: "complete syntax for mapserver config" means that the syntax files aren't yet present or complete for the new global configuration file? That would be a separate ticket to wanting them included in the standard VIM runtime bundle.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants