[Github-comments] [geany/geany] Status of ctags parsers in Geany (Issue #3170)

elextr notifications at github.com
Wed Apr 20 23:44:41 UTC 2022


> (Also, if you somehow merged all the PR into a single branch, there will probably be conflicts that have to be resolved because each of the new parsers is added to the end of the parser list separately).

No each was in its own branch, and anyway there were no conflicts applying them, when they didn't work I actually went back and checked the pull and it was fine.  Are there any other PRs (not labelled ctags) needed for them to work?  The tests pass, but `simple.clj` shows no symbols, so maybe the tests are also faulty?

I built all previously with meson, so I made two new clones compiled one with autotools, one with meson, applied just #3162 to both and built and both work, so its not meson.  Maybe some interaction, git did clojure as a fast forward not a merge as the other ones were?  I just copy and paste step 1 of the "command line instructions" on the PR to the terminal, nothing special.

Also checked with a random selection from [here](https://github.com/clojure/clojure/tree/master/test/clojure/test_clojure) and seemed to work, so I would say #3162 is fine by itself, I guess we see how it goes when they are all merged together.

Having had to wait for an autotools build I am now out of time :stuck_out_tongue_winking_eye: , will test others tomorrow.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/3170#issuecomment-1104557197
You are receiving this because you are subscribed to this thread.

Message ID: <geany/geany/issues/3170/1104557197 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20220420/ba7cfb22/attachment.htm>


More information about the Github-comments mailing list