-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update synocli-devel #5882
update synocli-devel #5882
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few minor adjustments but otherwise looking good!
@th0ma7 the only thing that bothers me, is the build of This is an issue with github actions, since every package is built in its own environment (locally it builds native llvm only once and takes up to 3 h on my older build env). For ARMv5 archs the cause is the Probably you can remeber why you introduced I cannot imagine a solution (exept removing the |
set to WIP: |
Agreed, the nicest would be to either have a workaround to:
It is.
A work-around that would be to add a new "phase" within the build process to do the
That is mandatory for parallel building as otherwise every archs kicks-in at the same time and tries to build the same native dependencies one over the other causing it to fail miserably. Work-around could be:
Such a change would be relatively intrusive and benefit from being standalone.... Unless we install it into the docker image or find binaries to download from. EDIT: further looking at things, the real missing part is EDIT2: But it may well be available in a linux compatible binary form from the llvm download page https://releases.llvm.org/download.html |
IMHO this is not the case. Every build in the matrix runs in its own VM and does not share the native build with other arch-TCVERSION build. The only concurrency problem was, when all VMs wanted to download the same sources, but this we fixed with the prepare job and the caching of downloaded files. |
the only available prebuilt version for x64 (amd64) is for freebsd11 But this archive is missing What about self hosting the |
@th0ma7 I started a test to try selfhosted prebuilt native llvm-9.0 (local builds have a slightly different build path ( The prebuilt package is only 55MB tar.gz, so this will be a fast install of native/llvm-9.0. |
You are right in a github-action environment... But when building locally using So again, from a concurrency standpoint when building locally with |
Could we self-host it on synocommunity.com directly? Another option could be https://rpmfind.net/linux/rpm2html/search.php?query=clang-tblgen&submit=Search+...&system=&arch=x86_64 EDIT: I actually wonder are both tlbgen mandatory or just llvm (and clang could be discarded)? Testing currently... EDIT2: I've created a testing PR for EDIT3: |
- update strace to v6.5 - add and fix gdb for qoriq arch - add llvm-9.0 for qoric arch - add binutils-2.32 for OLD_PPC_ARCHS
- use self hosted prebuilt native/llvm-9.0 - add native/llvm-9.0-build to keep the build code
- choose gdb-latest based on gcc version with c++11 support only
- gdb-latest: disable sim for PPC
0356c11
to
3149bc5
Compare
with this in mind, the pre-build-native target should only be used in the context of BTW I never use Anyway, with the use of prebuilt native/llvm-9 this is not a priority issue anymore. |
Probably right indeed.
Well it the arch that breaks stops, remaining parallel jobs continues (i.e.
Even better! Still, I'll check later on if I can enforce building native only from EDIT: It is enforced for only
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Checklist
all-supported
completed successfullyType of change