search

eclipse-cdt-cloud / cdt-cloud-blueprint

CDT Cloud Blueprint is a template for building custom web-based C/C++ tools. It is made up of existing open source components and can be easily downloaded and installed on all major operating system platforms.
MIT License
22 stars 11 forks source link

Integrate latest Theia Blueprint changes and update C++ dependencies #15

Closed sdirix closed 2 years ago

sdirix commented 2 years ago

What it does

Updating CDT.cloud Blueprint to the latest state of Theia Blueprint (706916e). Also updates the traceviewer extension and CMake plugins and integrates the clangd contexts example VS Code extension.

Contributed on behalf of STMicroelectronics

How to test

Run CDT.cloud Blueprint

sdirix commented 2 years ago

At the moment we don't bundle any "dependencies" with the Electron app, e.g. you also need a C++ toolchain and CMake on your system. I always start the TraceViewer server manually too.

The clangd extension is able to download clangd itself. So maybe we should suggest that for the trace viewer extension too, that's probably the cleanest solution.

planger commented 2 years ago

The clangd extension is able to download clangd itself. So maybe we should suggest that for the trace viewer extension too, that's probably the cleanest solution.

Yeah, I agree. Anyway, we should definitely do something about that soon. I think we can't expect a user to figure out what to do and adding it to the welcome page would more and more glutter it.

sdirix commented 2 years ago

Rebased on master and replicated the changes for the Browser application

sdirix commented 2 years ago

The clangd extension is able to download clangd itself. So maybe we should suggest that for the trace viewer extension too, that's probably the cleanest solution.

Yeah, I agree. Anyway, we should definitely do something about that soon. I think we can't expect a user to figure out what to do and adding it to the welcome page would more and more glutter it.

I forgot that the traceviewer is a Theia extension so we should bundle it ourselves. This is how the example TraceViewer application does it. I'll create an issue.

Edit: https://github.com/eclipse-cdt-cloud/cdt-cloud-blueprint/issues/17

sdirix commented 2 years ago

@planger I switched to the development build of the browser application because the Windows CI build always ran out of memory during the theia build. Is that fine for you?

planger commented 2 years ago

@planger I switched to the development build of the browser application because the Windows CI build always ran out of memory during the theia build. Is that fine for you?

Sure, thanks a lot!