-
Notifications
You must be signed in to change notification settings - Fork 630
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
path and directiries for optlib and xcmd #92
Comments
Sorry for the late reply. Before responding to your main question I have a few comments about the optlib mechanism in general:
Okay, now for the actual optlib configurations. I am actually currently working on extracting the lower-level ctags parts from Tagbar into its own plugin in order to make it usable by any Vim plugin that wants to make use of ctags functionality, and part of that is that I want to convert the wiki data into configurations files directly in the plugin. Originally I thought that it might be a good idea to have a common repository that both fishman-ctags and my new plugin can use, but after thinking about it some more I don't think this is really a good idea: one, it would mean that people who want to contribute a new filetype have to add it to two repositories (the ctags configuration one and the Vim-plugin one for the Vim-specific configuration), and two, since I would also have to support the standard exuberant-ctags it would mean that the configuration wouldn't be able to make use of the new fishman-ctags features. So maybe for now it would be better to have the configurations separate and maybe sync them occasionally. As for the license of the wiki contents, I don't think there should be any issue – it's just some simple configurations. I would certainly consider it public domain. I'll hopefully have some time next to do check things out with fishman-ctags and do some test conversions. |
Thank you very much for great comment. |
Have you thought about my comments? I think it would be a good idea to reach a consensus here before the first release to avoid having to change things later. |
I read again and again your comments.
I think default.ctags causes the complexity. In current To avoid the meaningless complexity. I found both two type of directories, optlib and preload. Background and requirementI assumed there are two types of optlib file; one should be turned on Informal specificationpreload directory family is for storing optlib files which should be optlib directory family is for storing optlib files which should be As far as conflicting the name of files, files under preload directory Files under optlib directory family are not loaded automatically. --options option can be used in files under preload directory, this Following directories are involved. A. /usr/lib/ctags/preload/.ctags (or /usr/local...) Ctags behavior Files under optlib can be used with specifying Role of directories For the personal purpose, use E or F. For the site-private purpose, use C or D. If you are a ctags developer, use A or B. |
Thanks for the explanation, I now understand your idea better. I think it would be good to clarify this main difference more in the documentation. I'm still not entirely sure though why you don't want to load all configurations by default -- what would be the downside to that? Performance? Potential incompatibilities? Considering that your mechanism allows for the disabling of configurations if they create problems I'm not sure the extra complexity of two mechanisms is worth it. I have two other questions:
In case that the separation of these two mechanism will remain I would suggest renaming them to make their intent more clear, for example rename By the way, I noticed a small issue in the news file: In the optlib section you load |
Here is my humble opinion; let me know what you think. I have the same opinion as @majutsushi, I would prefer to simplify and merge both optlib and preload dirs into only one. I would do a A. Users are not expected to touch A since it is owned by root. If names are not in conflict, all files are loaded automatically: If necessary, we could have a This flag makes How do we enable and disable certain *.ctags so they don't get loaded?(Which is the same question as: How we provide By not loading files which suffix isn't one we recognize (eg: *.ctags). Eg: This is consistent to what happens to other conf files in other projects (eg: .vimrc vs .vimrc.bkp or .vimrc_bkp), and it simplifies things. Why there isn't a
|
@viccuad, thank you for comments. I have been thinking again and again this issue.
It is just an experimental code but when I successfully integrate the translator to the build script, we can make this issue much simpler; we can focus on /etc and home directory. The critial issue is other members allow me to use bash in building process:-P |
Sorry for the noise. Just switching the "Release blocker" tag for the "Initial release" milestone so that everything is tracked in one place. |
Mach simlified mechanisms are introduced in #1519. |
Fixes universal-ctags#92. This fix was ported from oniguruma: kkos/oniguruma@257082d
@majutsushi, forgive me if I'm impolite.
Are you have interested in improving ctags?
We, fishman-ctags developers, are interested in your wiki page:
https://github.com/majutsushi/tagbar/wiki
fishman-ctags has ability to use the contents of .ctags and command line as a first class library with a mechanism called optlib. In my opinion it is better for users that your wiki contents are part of ctags and
made runnable.
Technically I, who designed and implmented optlib mechanisum, can import code from your wiki page.
However, it is ... not so ... good.
First of all I want feedback for what I implemented from other person than myself.
In addition I'm not sure the copyright and license term about the contents of wiki.
So I wonder whether you have interests writing an optlib file.
If your answer was yes, could you look at
https://github.com/fishman/ctags/blob/master/docs/f-news.rst
https://github.com/fishman/ctags/blob/master/docs/f-optlib.rst
and
https://github.com/fishman/ctags/blob/master/docs/f-units.rst
?
Questions are welcome.
Thank you very much for reading this.
The text was updated successfully, but these errors were encountered: