D5338: Add syntax highlighting file for Doxyfile
This revision was automatically updated to reflect the committed changes. Closed by commit R216:916efa110fff: Doxygen: Add Doxyfile highlighting (authored by dhaumann). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D5338?vs=13205=17112#toc REPOSITORY R216 Syntax Highlighting CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D5338?vs=13205=17112 REVISION DETAIL https://phabricator.kde.org/D5338 AFFECTED FILES autotests/folding/Doxyfile.example.fold autotests/html/Doxyfile.example.html autotests/input/Doxyfile.example autotests/reference/Doxyfile.example.ref data/syntax/doxyfile.xml To: kfunk, dhaumann, kwrite-devel, vkrause Cc: turbov, emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
emaurer added a comment. In https://phabricator.kde.org/D5338#128106, @dhaumann wrote: > @emaurer We just agreed that your version is better and would like to use this. Are you also OK with using the MIT license? Currently, we are trying to unify the licensing to MIT for as many of our xml highlighting files as possible. yes, confirmed, you can use the code with MIT license. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, dhaumann, kwrite-devel, vkrause Cc: turbov, emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
kfunk abandoned this revision. kfunk added a comment. Note: Abandoning this Diff in favor of the one from @emaurer REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, dhaumann, kwrite-devel, vkrause Cc: turbov, emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
dhaumann added a comment. @emaurer We just agreed that your version is better and would like to use this. Are you also OK with using the MIT license? Currently, we are trying to unify the licensing to MIT for as many of our xml highlighting files as possible. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, dhaumann, kwrite-devel, vkrause Cc: turbov, emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
dhaumann requested changes to this revision. dhaumann added a comment. This revision now requires changes to proceed. Any news on this? Also, do we have a unit test on this? Please decide on one version and make a review request again to get this done. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, dhaumann, kwrite-devel, vkrause Cc: turbov, emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
turbov added a comment. my 2 cents: I've used the syntax made by Ernst quite a long time and pretty happy w/ it :) REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, kwrite-devel, dhaumann Cc: turbov, emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
emaurer added a comment. > Heya, > > indeed, I saw that you had a set of valid keywords. If you're okay with it I'll just pull them in myself & then push this patch. We can work together on adding more stuff to it later on, no problem. sure, feel free to use from mine for a pushing seems the key diferences are 1. highlight a auto-generated comments between secions to see these during quick scrolling (comments like fullline #- 2. highlight varibles from build systems : with prefix @ 3. valid for doxy config keywords REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, kwrite-devel, dhaumann Cc: emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
kfunk added a comment. In https://phabricator.kde.org/D5338#101466, @emaurer wrote: > In https://phabricator.kde.org/D5338#100575, @kfunk wrote: > > > In https://phabricator.kde.org/D5338#100509, @dhaumann wrote: > > > > > Oh, and see this: https://github.com/KDE/syntax-highlighting/pull/5 > > > > > > What the... Created at almost the exact same time? :) > > > Hello guys, I was using my version for a long time and just Apr7 decided to commit, > and raised a pull request on Github. > and what I see - a couple hours earlier the same config was commited here :) > > you see my version has a few minor enhancements, primary of these is a defining valid keywords. > I'm newbie here, need a some time to understand how to use the instruments. > after that, I'll suggest my additions to current version , if you do not mind Heya, indeed, I saw that you had a set of valid keywords. If you're okay with it I'll just pull them in myself & then push this patch. We can work together on adding more stuff to it later on, no problem. REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, kwrite-devel, dhaumann Cc: emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
emaurer added a comment. In https://phabricator.kde.org/D5338#100575, @kfunk wrote: > In https://phabricator.kde.org/D5338#100509, @dhaumann wrote: > > > Oh, and see this: https://github.com/KDE/syntax-highlighting/pull/5 > > > What the... Created at almost the exact same time? :) guys, I was using my version for a long time and just Apr7 decided to commit, and raised a pull request on Github. and what I see - a couple hours earlier the same config was commited here :) you see my version has a few minor enhancements, primary of these is a defining valid keywords. I'm newbie here, need a some time to understand how to use the instruments. after that, I'll suggest my additions to current version , if you do not mind REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, kwrite-devel, dhaumann Cc: emaurer, #frameworks
D5338: Add syntax highlighting file for Doxyfile
kfunk added a comment. In https://phabricator.kde.org/D5338#100509, @dhaumann wrote: > Oh, and see this: https://github.com/KDE/syntax-highlighting/pull/5 What the... Created at almost the exact same time? :) REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, kwrite-devel, dhaumann Cc: #frameworks
D5338: Add syntax highlighting file for Doxyfile
dhaumann added a comment. Oh, and see this: https://github.com/KDE/syntax-highlighting/pull/5 REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, kwrite-devel, dhaumann Cc: #frameworks
D5338: Add syntax highlighting file for Doxyfile
dhaumann added a comment. I would prevent MIT license for the highlighting - and also for the test. REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, kwrite-devel, dhaumann Cc: #frameworks
D5338: Add syntax highlighting file for Doxyfile
vkrause accepted this revision. vkrause added a comment. This revision is now accepted and ready to land. Thank you, was missing this as well :) Just one nitpick: please make the license attribute more precise, e.g. "LGPLv2+". And if you happen to have a file around you can share, please add that to the unit test folder too. REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, dhaumann, kwrite-devel Cc: #frameworks
D5338: Add syntax highlighting file for Doxyfile
kfunk edited the summary of this revision. kfunk added reviewers: vkrause, dhaumann, kwrite-devel. REPOSITORY R216 Syntax Highlighting REVISION DETAIL https://phabricator.kde.org/D5338 To: kfunk, vkrause, dhaumann, kwrite-devel Cc: #frameworks
D5338: Add syntax highlighting file for Doxyfile
kfunk created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REPOSITORY R216 Syntax Highlighting BRANCH master REVISION DETAIL https://phabricator.kde.org/D5338 AFFECTED FILES data/syntax/doxyfile.xml To: kfunk Cc: #frameworks