Skip to content
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

llvmPackages: fix wasm with common opts #332170

Merged

Conversation

RossComputerGuy
Copy link
Member

@RossComputerGuy RossComputerGuy commented Aug 4, 2024

Description of changes

CC @spiage

Fixes things using WASM like Firefox.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added the 6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related label Aug 4, 2024
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 11-100 labels Aug 4, 2024
@RossComputerGuy
Copy link
Member Author

@ofborg build firefox

@spiage
Copy link

spiage commented Aug 4, 2024

@SuperSandro2000 @vcunat Hi! Can this two checks for firefox on darwin be skipped? This fix should fix issue with #330316
Both firefox and thunderbird builds fine with current commit on my machine and by ofborg too

@vcunat
Copy link
Member

vcunat commented Aug 4, 2024

No need to wait for this one, if you're confident that it's irrelevant. I think ofBorg can often get overloaded for darwin, so it takes time.

@spiage
Copy link

spiage commented Aug 4, 2024

No need to wait for this one, if you're confident that it's irrelevant. I think ofBorg can often get overloaded for darwin, so it takes time.

Thank you!

@RossComputerGuy can you break it?

@vcunat
Copy link
Member

vcunat commented Aug 4, 2024

Actually, firefox isn't even supported in nixpkgs for *-darwin.

@spiage
Copy link

spiage commented Aug 4, 2024

@vcunat could you please help me? How can I stop these two tests? Or can only @RossComputerGuy do this?

@RossComputerGuy
Copy link
Member Author

RossComputerGuy commented Aug 4, 2024

Sorry I just woke up, Firefox on Linux built at least which is all we need to prove the fix. The error which occurred happened during configuration and not at building, I just didn't have enough time to properly test this before I had to go to sleep.

@gador
Copy link
Member

gador commented Aug 4, 2024

PR LGTM. I got the config error on current master and the build for firefox is already way past it. I'm building firefox (and run nixpkgs-review on the buildserver) right now for x86_64-linux and can merge after they succeed. Or someone else merges before it ;-)

@RossComputerGuy RossComputerGuy merged commit e16b22c into NixOS:master Aug 4, 2024
31 of 33 checks passed
@RossComputerGuy RossComputerGuy deleted the fix/firefox-llvm-common-opts branch August 4, 2024 14:09
@spiage
Copy link

spiage commented Aug 4, 2024

Thank you! Every is fine now:
Version 15379 -> 15380:
firefox-unwrapped: -73.1 KiB
nixos-system-a7: 24.11.20240804.f099c76 → 24.11.20240804.e16b22c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: llvm/clang Issues related to llvmPackages, clangStdenv and related 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 11-100
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants