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

python-cffi: fix clang build #34158

Merged
merged 2 commits into from Jan 22, 2018
Merged

python-cffi: fix clang build #34158

merged 2 commits into from Jan 22, 2018

Conversation

LnL7
Copy link
Member

@LnL7 LnL7 commented Jan 22, 2018

Motivation for this change

A lot of packages depend on this and broke.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

@LnL7 LnL7 requested a review from FRidh as a code owner January 22, 2018 19:00
@Mic92
Copy link
Member

Mic92 commented Jan 22, 2018

@GrahamcOfBorg build python3Packages.cffi python2Packages.cffi

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

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

Success for system: x86_64-darwin

    % (self.verifier.get_module_name()))

testing/cffi0/test_verify2.py::test_win32_calling_convention_3
  /private/tmp/nix-build-python3.6-cffi-1.11.4.drv-0/cffi-1.11.4/cffi/vengine_cpy.py:192: UserWarning: reimporting '_cffi__xc5b5e469x752fb969' might overwrite older definitions
    % (self.verifier.get_module_name()))

-- Docs: http://doc.pytest.org/en/latest/warnings.html
===== 1822 passed, 143 skipped, 4 xfailed, 194 warnings in 331.91 seconds ======
/nix/store/nnwyqpka2s54y7w5mma8c383ghfxgfpx-python3.6-cffi-1.11.4
/nix/store/ich6qlkwc000xsg1ss1i05c8f0fdl9ly-python2.7-cffi-1.11.4

@Mic92
Copy link
Member

Mic92 commented Jan 22, 2018

I guess we should queue this for staging instead?

@LnL7
Copy link
Member Author

LnL7 commented Jan 22, 2018

Dunno, master is already broken anyway.

Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

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

Success for system: x86_64-linux

    % (self.verifier.get_module_name()))

testing/cffi0/test_verify2.py::test_ffi_gc_size_arg
  /tmp/nix-build-python3.6-cffi-1.11.4.drv-0/cffi-1.11.4/cffi/vengine_cpy.py:192: UserWarning: reimporting '_cffi__xa155022ax7f1150e2' might overwrite older definitions
    % (self.verifier.get_module_name()))

-- Docs: http://doc.pytest.org/en/latest/warnings.html
====== 1876 passed, 89 skipped, 4 xfailed, 202 warnings in 200.96 seconds ======
/nix/store/q4hx38x4fkbgj3ick114mflicyh25q48-python3.6-cffi-1.11.4
/nix/store/ibgvw4nhlvc0kpd88b789qbynhnjym5z-python2.7-cffi-1.11.4

@LnL7
Copy link
Member Author

LnL7 commented Jan 22, 2018

Estimating rebuild amount by counting changed Hydra jobs.
   6494 x86_64-darwin
   6896 x86_64-linux

I'll merge master into staging so we don't have unnecessary rebuilds.

@LnL7 LnL7 merged commit 7c84ddb into NixOS:master Jan 22, 2018
@LnL7 LnL7 deleted the cffi-clang branch January 22, 2018 19:54
Copy link

@GrahamcOfBorg GrahamcOfBorg left a comment

Choose a reason for hiding this comment

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

Success for system: aarch64-linux

  /nix/store/7wdhdh91j6fn9323rx45sbc4f01wk1sm-expat-2.2.5
  /nix/store/mhlap6kz8zcr3bhw52v8j8zgwl5srzpw-python3.6-cffi-1.11.4
  /nix/store/mn2c28fa7mp8ck3p74n80nls6g2w7flw-python3-3.6.4
  /nix/store/r5c2fd1mn4wksbyzxqgda8whdqk3k6z5-python2.7-cffi-1.11.4
copying path '/nix/store/r5c2fd1mn4wksbyzxqgda8whdqk3k6z5-python2.7-cffi-1.11.4' from 'https://cache.nixos.org'...
copying path '/nix/store/7wdhdh91j6fn9323rx45sbc4f01wk1sm-expat-2.2.5' from 'https://cache.nixos.org'...
copying path '/nix/store/mn2c28fa7mp8ck3p74n80nls6g2w7flw-python3-3.6.4' from 'https://cache.nixos.org'...
copying path '/nix/store/mhlap6kz8zcr3bhw52v8j8zgwl5srzpw-python3.6-cffi-1.11.4' from 'https://cache.nixos.org'...
/nix/store/mhlap6kz8zcr3bhw52v8j8zgwl5srzpw-python3.6-cffi-1.11.4
/nix/store/r5c2fd1mn4wksbyzxqgda8whdqk3k6z5-python2.7-cffi-1.11.4

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.

None yet

3 participants