[Github-comments] [geany] Gtkdoc hdr (#890)

[Colomban Wendling]

As pointed out, it's only optional so long as you don't want GI.  And as you want it, it's not really optional.

This said, I didn't see you were using an existing library to do some of the work.  But unfortunately, this library isn't available in Debian apparently, so it'd mean we also either bundle it, or depend on the user fetching it somehow (without having searched for it, I myself have no clue how or where).  Meh.
Maybe I'm underestimating it, but like @codebrainz I wonder if it really can be that hard to parse some XML, which is supposed to be relatively easy to traverse.

But well.  Yeah I understand you invested time in writing this, and it might not be nice to hear it's getting refused because it's written in a particular language.  But without re-stating the arguments above, I don't think it's unfair, it's just not something I'm (we're) ready to commit to, nor find reasonable.
Also, I doubt it'd throw all your efforts away.  It's likely possible to translate most of the script to e.g. Python without having to start fro scratch, even if it will be some work.

> Also, you forgot to mention PHP, we have a PHP script also (at least some core devs know PHP though).

AFAIK the only PHP script we have is to generate the PHP tags file, which is currently checked in.  And well, I wouldn't have recommended using PHP, but I would also probably accept more easily a Ruby script if it was here only to generate some Ruby-specific stuff (e.g. I could understand that it maybe could access some internal AST or whatnot and that being interesting and very hard to reproduce in some other language).
And yeah, I also happen to understand PHP quite well, so it's also one less barrier ^^

---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/890#issuecomment-175633167
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20160127/073cf96d/attachment.html>