Skip to content

Set 'path' and add fallback for 'includeexpr' - #431

Merged
whonore merged 10 commits into
whonore:mainfrom
mateoxh:path
Aug 1, 2026
Merged

Set 'path' and add fallback for 'includeexpr'#431
whonore merged 10 commits into
whonore:mainfrom
mateoxh:path

Conversation

@mateoxh

@mateoxh mateoxh commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Until the server is started the command gf and insert-mode completion for
included files don't work, so I changed &path to point to Rocq's libraries
directories and added a fallback to &includeexpr when the query to Rocq
fails.

One small caveat is that for gf the filename is first searched in &path and
if it can't be found it is modified with &includeexpr and another search is
done, so (in theory) it may be possible that the file opened by gf is not the
same one found by Rocq (&includeexpr is always used on the files found with
&include).
I don't think it's a big issue but remapping gf is a simple solution.

Comment thread autoload/coqtail/util.vim Outdated
@mateoxh

mateoxh commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

Is that the correct find_coq function? It's not coqidetop but it accepts the same flag and it works.
Also, here is the only place you check for the user defined variables, not sure if that call can be replaced with the new function in coqtail/util.vim.

@whonore

whonore commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Thanks, I think this looks good. I'll double check that everything works ok tomorrow.

@whonore
whonore merged commit 6bfe28f into whonore:main Aug 1, 2026
9 checks passed
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

Successfully merging this pull request may close these issues.

2 participants