RFR: 8223678: Add Visual Studio Code workspace generation support (for native code)
Robin Westberg
robin.westberg at oracle.com
Mon Jun 3 08:36:40 UTC 2019
Hi Erik,
> On 31 May 2019, at 18:15, Erik Joelsson <erik.joelsson at oracle.com> wrote:
>
> Hello Robin,
>
> On 2019-05-31 05:26, Robin Westberg wrote:
>> Hi Erik,
>>
>>> On 29 May 2019, at 17:22, Erik Joelsson <erik.joelsson at oracle.com> wrote:
>>>
>>> Thanks, looks good!
>> Thanks for reviewing! Unfortunately I had to do a few more general changes to get the “ccls" indexer working properly as well, hope you don’t mind taking another look:
>>
>> Webrevs:
>> - Full: http://cr.openjdk.java.net/~rwestberg/8223678/webrev.03/
>> - Inc: http://cr.openjdk.java.net/~rwestberg/8223678/webrev.02-03/
>
> Looks good!
>
> I sure appreciate adding tests when modifying the hairier utility macros.
Thanks again for reviewing!
I did a small tweak to the settings of one of the optional indexers (ccls), perhaps not something that needs a detailed review, but here’s the final version that I’m thinking of pushing:
Webrev:
- Full: https://cr.openjdk.java.net/~rwestberg/8223678/webrev.04/
- Inc: http://cr.openjdk.java.net/~rwestberg/8223678/webrev.03-04/
Best regards,
Robin
>
> /Erik
>
>> Best regards,
>> Robin
>>
>>> /Erik
>>>
>>> On 2019-05-29 06:52, Robin Westberg wrote:
>>>> Hi Erik,
>>>>
>>>> Thanks for taking a look!
>>>>
>>>>> On 28 May 2019, at 15:52, Erik Joelsson <erik.joelsson at oracle.com> wrote:
>>>>>
>>>>> Hello Robin,
>>>>>
>>>>> Looks good.
>>>>>
>>>>> Adding of ide.md is very nice, but could you try to format it like testing.md and building.md in regards to line lengths? I believe we are trying for 80 chars in those to make them read reasonably in a standard editor.
>>>> Sure, did a bit of reflowing and updated some formatting in order to look more like the other .md files.
>>>>
>>>> Webrevs:
>>>> - Full: https://cr.openjdk.java.net/~rwestberg/8223678/webrev.02/
>>>> - Inc: https://cr.openjdk.java.net/~rwestberg/8223678/webrev.01-02/
>>>>
>>>> Best regards,
>>>> Robin
>>>>
>>>>> /Erik
>>>>>
>>>>> On 2019-05-27 09:03, Robin Westberg wrote:
>>>>>> Hi all,
>>>>>>
>>>>>> Please review this change that adds build system support for generating a Visual Studio Code workspace configured for working with the JDK native code. It configures the default C/C++ IntelliSense Engine to allow code completion/navigation and similar features. It also configures two executable targets (gtestLauncher and java) that can be built and debugged from the IDE.
>>>>>>
>>>>>> The main target is "make vscode-project”, additional information can be found in doc/ide.[md|html].
>>>>>>
>>>>>> Issue:
>>>>>> https://bugs.openjdk.java.net/browse/JDK-8223678
>>>>>>
>>>>>> Webrev:
>>>>>> https://cr.openjdk.java.net/~rwestberg/8223678/webrev.01/
>>>>>>
>>>>>> Testing:
>>>>>> Manual testing on Linux, MacOS and Windows
>>>>>>
>>>>>> Thanks Erik Joelsson for taking a look at an earlier version of it!
>>>>>>
>>>>>> Best regards,
>>>>>> Robin
More information about the build-dev
mailing list