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

python3Packages.sphinxHook: Fix cross compilation #249157

Merged
merged 1 commit into from
Sep 9, 2023

Conversation

kip93
Copy link
Contributor

@kip93 kip93 commented Aug 14, 2023

Fixes build of wrapt. Also removed the now irrelevant workaround on beautifulsoup4.

Description of changes

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux (cross compiled from x86_64-linux)
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • 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/)
  • 23.11 Release Notes (or backporting 23.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.

@kip93
Copy link
Contributor Author

kip93 commented Aug 25, 2023

Changed target branch to staging and rebased due to number of rebuilds.

@kip93 kip93 force-pushed the chore/fix-sphinx-hook branch 2 times, most recently from 100ae35 to c7454bd Compare September 5, 2023 09:45
@Artturin
Copy link
Member

Artturin commented Sep 9, 2023

hooks/default change no longer necessary since #247245

nix-repl> pkgsCross.aarch64-multiplatform.__splicedPackages.python3Packages.sphinxHook.__spliced
{ buildBuild = «derivation /nix/store/85jbgw8qis73zzjb8jr42zm375m8r7nk-python3.10-sphinx-hook.drv»; buildHost = «derivation /nix/store/85jbgw8qis73zzjb8jr42zm375m8r7nk-python3.10-sphinx-hook.drv»; buildTarget = «derivation /nix/store/85jbgw8qis73zzjb8jr42zm375m8r7nk-python3.10-sphinx-hook.drv»; hostHost = «derivation /nix/store/dc6yabkisig7aifmvgynz41wni6jzbcf-python3.10-sphinx-hook.drv»; hostTarget = «derivation /nix/store/dc6yabkisig7aifmvgynz41wni6jzbcf-python3.10-sphinx-hook.drv»; }

@kip93
Copy link
Contributor Author

kip93 commented Sep 9, 2023

Just rebased with staging to be sure, but it still fails to cross compile wrap without this, with the same error

Copy link
Member

@Artturin Artturin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I forgot that my pr only fixed splicing python3Packages.callPackage { python }: while the hook splicing within the set (used by callPackage) is still broken and will eventually be fixed by #228139

the rebuilds are a bit weird so i'll check why that is happening

Failed due to using host sphinx instead of build one.
@Artturin
Copy link
Member

Artturin commented Sep 9, 2023

I checked and there's really no rebuilds, rebuild count is likely just because of a rebase gone wrong

@Artturin Artturin merged commit de63d2e into NixOS:master Sep 9, 2023
9 of 13 checks passed
kip93 added a commit to OroraTech/nixpkgs that referenced this pull request Sep 30, 2023
@kip93 kip93 deleted the chore/fix-sphinx-hook branch September 30, 2023 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants