Re: [Github-comments] [geany/geany] geany-doxygen.m4:don't generate doxygen by default (#2190)

2019-06-13 Thread Andy Alt
andy5995 commented on this pull request. > @@ -7,7 +7,7 @@ AC_DEFUN([GEANY_CHECK_DOXYGEN], [AS_HELP_STRING([--enable-api-docs], [generate API documentation using Doxygen [default=no]])],

Re: [Github-comments] [geany/geany] geany-doxygen.m4:don't generate doxygen by default (#2190)

2019-06-13 Thread Andy Alt
@andy5995 pushed 1 commit. bd2fbc8ced93380747c45435949c909ca7a12791 show the default as "auto" (#2189) -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [Github-comments] [geany/geany] .gitignore:ignore the _build dir (#2191)

2019-06-13 Thread Matthew Brush
Closed #2191. -- 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/2191#event-2412210701

Re: [Github-comments] [geany/geany] .gitignore:ignore the _build dir (#2191)

2019-06-13 Thread Matthew Brush
@andy5995 if this is the directory you use to build in (good idea to build "out of tree"), you can use: ```bash $ echo "/_build/" >> .git/info/exclude ``` And it will ignore it for you locally. I do this sometimes with a `.misc` and `.build` for random files I don't want to check in and for

Re: [Github-comments] [geany/geany] geany-doxygen.m4:don't generate doxygen by default (#2190)

2019-06-13 Thread elextr
elextr commented on this pull request. > @@ -7,7 +7,7 @@ AC_DEFUN([GEANY_CHECK_DOXYGEN], [AS_HELP_STRING([--enable-api-docs], [generate API documentation using Doxygen [default=no]])],

Re: [Github-comments] [geany/geany] .gitignore:ignore the _build dir (#2191)

2019-06-13 Thread elextr
I think it might have meant an out-of-tree directory, I don't think you gain much if its in-tree :) I couldn't find any reference in the docs, but maybe @codebrainz will, he seems to be a better googler than me :) I don't know about the mingw builds (@eht16 ?), but since Travis will never

Re: [Github-comments] [geany/geany] geany-doxygen.m4:don't generate doxygen by default (#2190)

2019-06-13 Thread Matthew Brush
codebrainz commented on this pull request. > @@ -7,7 +7,7 @@ AC_DEFUN([GEANY_CHECK_DOXYGEN], [AS_HELP_STRING([--enable-api-docs], [generate API documentation using Doxygen [default=no]])],

Re: [Github-comments] [geany/geany] geany-doxygen.m4:don't generate doxygen by default (#2190)

2019-06-13 Thread Matthew Brush
I think this is intentional; if you have the needed dependencies, the docs will be built, otherwise they won't. If you use `--enable-*` and don't have the dependencies, it will fail with an error saying as much. -- You are receiving this because you are subscribed to this thread. Reply to this

Re: [Github-comments] [geany/geany] .gitignore:ignore the _build dir (#2191)

2019-06-13 Thread Andy Alt
> Where does "_build" come from? I thought I saw somewhere in the docs where _build is suggested as a separate build directory. That's what I use anyway. I guess it's only used for in travis.yml and maybe the mingw build? Ah, just close this if you don't think it's relevant. -- You are

Re: [Github-comments] [geany/geany] .gitignore:ignore the _build dir (#2191)

2019-06-13 Thread elextr
Where does "_build" come from? -- 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/2191#issuecomment-501908413

[Github-comments] [geany/geany] .gitignore:ignore the _build dir (#2191)

2019-06-13 Thread Andy Alt
You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/2191 -- Commit Summary -- * .gitignore:ignore the _build dir -- File Changes -- M .gitignore (1) -- Patch Links -- https://github.com/geany/geany/pull/2191.patch

Re: [Github-comments] [geany/geany] api-docs enabled by default (#2189)

2019-06-13 Thread Andy Alt
I figured it out (#2190) -- 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/issues/2189#issuecomment-501847956

[Github-comments] [geany/geany] geany-doxygen.m4:don't generate doxygen by default (#2190)

2019-06-13 Thread Andy Alt
This fixes the discrepancy where `configure --help` shows "default=no" but the api-docs are built anyway (fixes #2189) You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/2190 -- Commit Summary -- * geany-doxygen.m4:don't generate doxygen by

Re: [Github-comments] [geany/geany] [WIP] Add Groovy custom filetype (#2188)

2019-06-13 Thread Jonathan Bradley Whited
Looks good and works fine. Thanks. My suggestion for build-menu: ``` FT_00_LB=_Compile FT_00_CM=groovyc "%f" FT_00_WD= EX_00_LB=Execute _Script EX_00_CM=groovy "%f" EX_00_WD= ``` To execute a non-script made by using groovyc, you'd need to also add the groovy jar to the classpath, but too

Re: [Github-comments] [geany/geany] api-docs enabled by default (#2189)

2019-06-13 Thread Andy Alt
I meant.. I can disable it by using --disable-api-docs but don't know how to fix the discrepancy in the macro file. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[Github-comments] [geany/geany] api-docs enabled by default (#2189)

2019-06-13 Thread Andy Alt
`configure --help` shows that the doxygen api-docs aren't enabled by default. But at least for me, they are being generated by default. I'm afraid to admit that after looking at https://github.com/geany/geany/blob/master/m4/geany-doxygen.m4 I don't know how to fix that. -- You are receiving

Re: [Github-comments] [geany/geany] Kotlin Language Support (#1581)

2019-06-13 Thread Nick Treleaven
@esotericpig I've added a few things for Groovy and made a WIP pull: #2188. Please test it and advise on what the build commands should be. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[Github-comments] [geany/geany] [WIP] Add Groovy custom filetype (#2188)

2019-06-13 Thread Nick Treleaven
Uses the C lexer and C++ tag parsing. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/2188 -- Commit Summary -- * Add Groovy custom filetype * Don't highlight lines with unterminated string literal; C++ tags * filetype registration --

Re: [Github-comments] [geany/geany] Kotlin Language Support (#1581)

2019-06-13 Thread Nick Treleaven
> I tried to make one, but it's a bit more difficult Thanks, I've made a start using the C lexer with properties set for dollar identifiers and triple quoted `"""strings"""`: https://github.com/ntrel/geany/commits/groovy-ft The other strings aren't supported by Scintilla's LexCPP AFAIK. --

Re: [Github-comments] [geany/geany] Add Kotlin custom filetype from github.com/esotericpig (#2186)

2019-06-13 Thread Nick Treleaven
ntrel commented on this pull request. > +# https://kotlinlang.org/docs/reference/basic-types.html +secondary=Double Float Long Int Short Byte NaN Void +# documentation keywords for javadoc +doccomment=author deprecated exception param return see serial serialData serialField since throws todo

Re: [Github-comments] [geany/geany] Add Kotlin custom filetype from github.com/esotericpig (#2186)

2019-06-13 Thread Nick Treleaven
@ntrel pushed 1 commit. 72b7dc98023fe3458c45ac9863b358bd2559fe79 Use more usual build command labels -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [Github-comments] [geany/geany] Kotlin Language Support (#1581)

2019-06-13 Thread Jonathan Bradley Whited
> @esotericpig Thanks, I've made a pull to add this: #2186. > > > I'd like to also add Groovy. > > I'm happy to merge that if done as a custom filetype. Thank you. I tried to make one, but it's a bit more difficult. Specifically stuff like this: ```Groovy println "${string_interop}" println

Re: [Github-comments] [geany/geany] Add Kotlin custom filetype from github.com/esotericpig (#2186)

2019-06-13 Thread Jonathan Bradley Whited
esotericpig commented on this pull request. > +# https://kotlinlang.org/docs/reference/basic-types.html +secondary=Double Float Long Int Short Byte NaN Void +# documentation keywords for javadoc +doccomment=author deprecated exception param return see serial serialData serialField since throws

[Github-comments] [geany/geany] TypeScript syntax highlighting (#2187)

2019-06-13 Thread Nick Treleaven
Fixes #1449. Reboot of #1450 updating `data/Makefile.am`. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/2187 -- Commit Summary -- * TypeScript syntax highlighting -- File Changes -- M data/Makefile.am (1) A

Re: [Github-comments] [geany/geany] Kotlin Language Support (#1581)

2019-06-13 Thread Nick Treleaven
@esotericpig Thanks, I've made a pull to add this: #2186. > I'd like to also add Groovy. I'm happy to merge that if done as a custom filetype. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [Github-comments] [geany/geany] Add Kotlin custom filetype from github.com/esotericpig (#2186)

2019-06-13 Thread Nick Treleaven
ntrel commented on this pull request. > +# https://kotlinlang.org/docs/reference/basic-types.html +secondary=Double Float Long Int Short Byte NaN Void +# documentation keywords for javadoc +doccomment=author deprecated exception param return see serial serialData serialField since throws todo

[Github-comments] [geany/geany] Add Kotlin custom filetype from github.com/esotericpig (#2186)

2019-06-13 Thread Nick Treleaven
Fixes #1581. `filetypes.Kotlin.conf` written by https://github.com/esotericpig. You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/2186 -- Commit Summary -- * Add Kotlin custom filetype from github.com/esotericpig -- File Changes -- M