Conversation
|
Oh, we've got another #194634... |
|
Oh I only saw the closed PR. Should I close it ? The other PR seems to have bitrotted. |
That's not for me to decide. To be honest, I also have PRs unclosed while better PRs are coming in. 😅 (Looks like this PR does not build on aarch64-linux, while the other one does.)
I feel the same way you do. I'd rather see that PR merged NOW, and additional patches added gradually, so that other important PRs (GHC, Zig 0.10.0, etc) can come in. One out of 197 committers takes the step, the problem is solved, while the reality is not. |
pkgs/top-level/all-packages.nix
Outdated
There was a problem hiding this comment.
Why does it need stdenv from llvm 13?
There was a problem hiding this comment.
They dropped support for clang12 and I tested on macos so stdenv is llvm)
Let me find the line with this info
There was a problem hiding this comment.
It probably should be under isDarwin condition since gcc is used to build llvm on Linux.
There was a problem hiding this comment.
https://github.com/llvm/llvm-project/blob/llvmorg-14.0.6/libcxx/include/__config#L423-L424
These line are removed on llvmorg-15.0.6, I didn't find requirement in README or in documentation so I don't know minimal version of gcc. You are right I will add a guard.
There was a problem hiding this comment.
I think It is not the correct way to require a recent version of clang because it has reference to llvm11 (from darwin stdenv):
$ nix-store -qR $(nix-store -qd /nix/store/n58zxqjx4lz785swd6xxcxv03la5lhfv-libcxxabi-15.0.6) | grep llvm
/nix/store/yww9hd80ikq72zrvkdk36picpqgvsh8b-llvm-11.1.0.src.tar.xz.drv
/nix/store/6wv8acdd7bm7npixgr3rk86mf55fj47c-llvm-config-link-static.patch
/nix/store/2mkz2cmqwg45xv7xfsj9idsb39maipmr-llvm-11.1.0.drv
/nix/store/cpnly3bmw6blsryxfgp2hs8ig2jz2aaz-llvm-13.0.1.drv
EDIT: after using nix-store --tree -q I see no llvm11 as direct dependencies.
1fa1af9 to
fb8668c
Compare
|
|
|
Just for those who want to see if it builds for your platform:
|
|
I tested this patch on Linux with Zig (a tool which uses libllvm and libclang), and it seemed to work well. (#210324) |
|
Is this PR ready to merge? |
Really ? Can I have a build log or a way to build if I don't have arm machine. |
Here is the build log on ofborg: https://logs.nix.ci/?key=nixos/nixpkgs.209536&attempt_id=a26d3c8f-3178-4417-917a-bea85254c8c9 |
then why does hydra have all the checks set to green? edit: oh its the ones that are gray that didnt report a status back to github |
|
i cherry-picked this into https://github.com/milahu/nixpkgs/tree/flutter-engine/init Details |
Would you please check if this also happens on #194634? If so, it is most likely an upstream issue. |
|
works on my second try with could be an OOM error. i last saw these random "internal compiler error" in qt6.qtwebengine (based on chrome) (#178171), where $NIX_BUILD_CORES is ignored |
e5e43ea to
0ce1ad3
Compare
…opment/compilers/llvm/15 Signed-off-by: Et7f3 <cadeaudeelie@gmail.com>
0ce1ad3 to
8438df0
Compare
Co-authored-by: misuzu <neironyan@gmail.com>
Co-authored-by: misuzu <neironyan@gmail.com>
8438df0 to
8c01b12
Compare
|
It eval correctly on my machine so ofborg should be happy |
|
No new regression since 15.0.6 it did pass the same (disabled) tests on x86_64-darwin. |
|
still failing on darwin |
If it is about ofborg, ofborg stops because LLVM always takes more than an hour to build on darwin. This is perfectly normal. |
|
lldb is broken on x86_64-darwin: |
|
I tried building lldb on Darwin x86_64. I have patches which will likely fix the two issues mentioned in @misuzu's comment. They will take a while to build on my old laptop though. See attached:
EDIT: My Libsystem patch doesn't work yet. I'll continue working on it. EDIT: The lldb patch doesn't work either. 🙈 I'll fix it. |
I don't think lldb is blocking zig and I think I saw patch in the other PR to fix lldb on darwin (so I didn't investigated that build failure). Unless I am wrong what is blocking is test failure in main compiler. |
I thought this PR (#209536) is for LLVM, not Zig.
Which platform and package? I built LLVM on Linux x86_64 and got no test failures. |
We can just apply 1d3ca42 in the other PR. |
Yes this PR is only for LLVM. Zig need one PR that add llvm to be merged. Since you opened a PR for zig I supposed you main interest was to unlock the situation and have one PR merged. Ah my bad I saw #209536 (comment) if those are fixed upstream then pick what doesn't build sure. |
|
What's the difference between PRs #209536 and #194634? Do they both do the same thing? Are both being maintained? I originally built my Zig patch (#210324) on top of this PR (#209536) because this LLVM PR (#209536) was open and the other LLVM PR (#194634) was marked as a draft. But now the other one (#194634) is approved and seems to have some of the bug fixes already...? I'm confused.
Yes, I want my Zig patch in, and if that means getting your PR in (or another similar PR), I'd do whatever I can to help get that PR in too. |
Yes, they do the same thing. That PR has been marked as a draft, mainly because it has higher goals, like targeting more platforms, enabling LLVM tests on macOS, so it took months before the author finally succeeded. Since I'm now a committer (aha!), I think I'll merge it if Zig builds on my (slow) aarch64-linux machine. |
|
Closing in favor of #194634. |
Do you get it on master ? Even with one jobs at the time ? if yes open an issue please, so we don't loose it here. |
Description of changes
I have taken example of primos pr #162104 if it is a command I put it in commit message otherwise I do it in a separate commit. so commit 2 and 4 are by hand. I also updated patch to fix line number
fix #191132
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)nixos/doc/manual/md-to-db.shto update generated release notes@nektro