-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
python-cffi: fix clang build #34158
Conversation
Builds fine without it now.
@GrahamcOfBorg build python3Packages.cffi python2Packages.cffi |
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.
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
I guess we should queue this for staging instead? |
Dunno, master is already broken anyway. |
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.
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
I'll merge master into staging so we don't have unnecessary rebuilds. |
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.
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
Motivation for this change
A lot of packages depend on this and broke.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)