theia-ide / theia-apps

Theia applications examples - docker images, desktop apps, packagings
Apache License 2.0
1.04k stars 346 forks source link

llvm: bump llvm version to 12 #393

Closed vince-fugnitto closed 4 years ago

vince-fugnitto commented 4 years ago

What it does

Fixes: #386

The following pull-request bumps the LLVM version to 12 (latest dev branch), and fixes the build issue regarding packages not successfully found.

How to test

Verify that CI passes successfully.

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com

marcdumais-work commented 4 years ago

I think https://github.com/theia-ide/theia-apps/pull/392 will need to go in first, before the next versions of theia-cpp and theia-full can compile correctly?

vince-fugnitto commented 4 years ago

LGTM - thanks @vince-fugnitto . I expect the CI failures for next images should be fixed once #392 is merged.

@marcdumais-work I merged the @theia/languages PR and rebased this one so hopefully the build passes 🤞

vince-fugnitto commented 4 years ago

The build for theia-full:next is failing due to @theia/rust which pulls @theia/languages. Merging to continue resolving the builds...