Update ctags to version p5.9.20210411.0 #2830
Conversation
As much as we love to complain about Github, in this case I think its Linus' fault because Git doesn't have the capability of making a patch that depends on another until the first is merged. |
|
@dolik-rce just to let you know this isn't forgotten, but we are trying to update the windows build architecture for GTK3 and then get a team together to make a 1.38 release. This will be merged as soon after that so it has time to be tested before the next release, but its better not to do it just before. |
|
Ok, I'll wait. I very much appreciate the update, thank you. |
|
Not sure how we are going to handle ctags updates in Geany - I'd suggest always updating to the latest ctags version after making a release and then, if possible, not touching it until the next release so it gets sufficient testing. If we do it this way, this pull request might be replaced by a newer ctags release (should be no problem, I can do it myself). |
|
@techee yeah, that was the plan [Edit: just to be clear, I don't mean that you should do it, though of course you are welcome, I mean that it should be done soon after each release] |
|
@elextr: Now that 1.38 was released, is it the right time to update ctags? Should I create new PR to update to newest version? |
|
@dolik-rce not forgotten, this was the next thing I planned to test after Scintilla 5 updates if nobody beat me to it. Due to Scintilla being central to Geany and this update being a jump of two major versions of Scintilla it is important to do first to allow maximum testing. Unfortunately there have been a couple of bugs in Scintilla which are addressed in 5.1.4 due out "in a couple of days" and should allow that to be merged. Of course that doesn't stop anyone else from looking at this and/or updating to the latest ctags as discussed above. |
Yeah, I was actually thinking about doing it myself (against the latest version of ctags) in case some problems pop up as I probably have most experience with this stuff. Will try to prepare a pull request today. |
|
This has been superseded by #2984, closing. |
Update the ctags to minimal version required for the Kotlin parser to work. No changes to the geany sources were necessary.
This a prerequisite to dolik-rce#1, as suggested by @elextr in #2778 (comment). Unfortunately, that PR can't be created here directly, since it compares two branches in my fork, but I will move it to this repository as soon as this PR is accepted or when someone with commit rights to geany repository pulls in this branch.