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

Missing configure flag for dir naming (/etc/ctags, /usr/share/ctags, /usr/lib/x86_64-linux-gnu/ctags) #580

Closed
viccuad opened this issue Sep 27, 2015 · 9 comments · Fixed by #587

Comments

@viccuad
Copy link
Contributor

viccuad commented Sep 27, 2015

I have looked at configure --help but I can't seem to find a flag for this.

Right now, if I install universal-ctags I get (among other things):

/etc/ctags/* (optlib, preload and corpora)
/usr/lib/x86_64-linux-gnu/ctags/* (optlib, preload and corpora)
/usr/share/ctags/* (optlib, preload and corpora)

It would be useful to have a configure flag for changing those ctags dirs into other string (and maybe defaulting to universal-ctags?).

That would enable universal-ctags to be installed in parallel with other ctags projects, without the possibility of collisions in those directories.

@p-montanus
Copy link
Contributor

For discussion, quick-hacked branch created.
p-montanus/ctags@cb2ab83 (includes #579 changes)
p-montanus/ctags@2608055
p-montanus/ctags@b7dd6bb
p-montanus/ctags@7f26184

Add options to configure:

--with-ctags-datadir=[DATADIR/ctags]
--with-ctags-sysconfdir=[SYSCONFDIR/ctags]
--with-ctags-libexecdir=[LIBEXECDIR/ctags]

--without-ctags-datadir is equiv. to --with-ctags-datadir=DATADIR,
--without-ctags-sysconfdir is equiv. to --with-ctags-sysconfdir=SYSCONFDIR, and
--without-ctags-libexecdir is equiv. to --with-ctags-libexecdir=LIBEXECDIR.

@masatake
Copy link
Member

@p-montanus, the changes of #579 was merged.

@p-montanus
Copy link
Contributor

Thank you for merging #579.
rebased.

@masatake
Copy link
Member

@viccuad, the names of directories are not sutable.
See #92. Is it o.k.? I want to stablize it but I cannot find time to do it because there are so many critical issues.

@masatake
Copy link
Member

@p-montanus, why do you say "quick-hacked" about your branch?

@p-montanus
Copy link
Contributor

why do you say "quick-hacked" about your branch?

Due to not well testing at that moment, and not thinking carefully about option names.

@masatake
Copy link
Member

Oh, I see. You can utilize misc/tinst script for install testing.

@p-montanus
Copy link
Contributor

You can utilize misc/tinst script for install testing.

Thank you for letting me know.
make tinst passed.

@p-montanus
Copy link
Contributor

revised: p-montanus/ctags@7f26184.
configure options are renamed from --with-pkg*dir to --with-ctags-*dir, and handle --without-* cases.

If no exception to this specification in a half day, I will create a pull request.

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

Successfully merging a pull request may close this issue.

4 participants