[Github-comments] [geany/geany] Ctags sync rebased (#2666)

[Jiří Techet]

> No, I don't. I just ran the checks now with this PR as I thought it might help.
The tests fail in the same way for Geany master, so you are absolutely right: it's not new and not related to this PR.

Ctags uses appveyor for CI on Windows so it might be worth considering to use it as well so we can run tests for Windows builds too.

>> 1. Would you try opening the file in binary mode and check what tests fail when you do?

> No change.

Weird, I don't get it then. This is the place where tags are written and newlines are written using `\n` - so it's a matter of figuring out why `\n` produces CRLF.

>> 2. Would you try changing CRLF line ends in the SQL tests that fail to LF to see if this is the reason for the problem? I checked uctags implementation of file reading and CRLF should be handled there correctly so it could be something parser-specific.

> This actually helps. I converted the test files of the six failures to LF and the tests passed fine.

Good, at least we know it's the CRLFs. Now only to figure out why. I'm afraid I'm out of ideas so it's your turn with debugging :-(

> So I think we should address this separately, maybe after this PR got merged.

Sounds good to me.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2666#issuecomment-735276366
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20201128/344133b1/attachment.htm>