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.

Sign up for GitHub

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

Jump to bottom

{fedora-{38,39},ubuntu-mantic,debian-trixie}-minimal: pyzmq build failure #36027

Closed
mkoeppe opened this issue Aug 3, 2023 · 1 comment · Fixed by #36663
Closed

{fedora-{38,39},ubuntu-mantic,debian-trixie}-minimal: pyzmq build failure #36027

mkoeppe opened this issue Aug 3, 2023 · 1 comment · Fixed by #36663
Labels
c: packages: standard
Milestone
sage-10.2

Comments

@mkoeppe
Copy link
Contributor

mkoeppe commented Aug 3, 2023 

[pyzmq-24.0.1]     In file included from /usr/include/c++/13/ext/alloc_traits.h:34,
  [pyzmq-24.0.1]                      from /usr/include/c++/13/bits/basic_string.h:39,
  [pyzmq-24.0.1]                      from /usr/include/c++/13/string:54,
  [pyzmq-24.0.1]                      from /usr/include/c++/13/bits/locale_classes.h:40,
  [pyzmq-24.0.1]                      from /usr/include/c++/13/bits/ios_base.h:41,
  [pyzmq-24.0.1]                      from /usr/include/c++/13/ios:44,
  [pyzmq-24.0.1]                      from /usr/include/c++/13/istream:40,
  [pyzmq-24.0.1]                      from /usr/include/c++/13/sstream:40,
  [pyzmq-24.0.1]                      from bundled/zeromq/src/zmtp_engine.cpp:41:
  [pyzmq-24.0.1]     /usr/include/c++/13/bits/alloc_traits.h: In instantiation of 'struct std::__allocator_traits_base::__rebind<zmq::secure_allocator_t<unsigned char>, unsigned char, void>':
  [pyzmq-24.0.1]     /usr/include/c++/13/bits/alloc_traits.h:94:11:   required by substitution of 'template<class _Alloc, class _Up> using std::__alloc_rebind = typename std::__allocator_traits_base::__rebind<_Alloc, _Up>::type [with _Alloc = zmq::secure_allocator_t<unsigned char>; _Up = unsigned char]'
  [pyzmq-24.0.1]     /usr/include/c++/13/bits/alloc_traits.h:228:8:   required by substitution of 'template<class _Alloc> template<class _Tp> using std::allocator_traits< <template-parameter-1-1> >::rebind_alloc = std::__alloc_rebind<_Alloc, _Tp> [with _Tp = unsigned char; _Alloc = zmq::secure_allocator_t<unsigned char>]'
  [pyzmq-24.0.1]     /usr/include/c++/13/ext/alloc_traits.h:126:65:   required from 'struct __gnu_cxx::__alloc_traits<zmq::secure_allocator_t<unsigned char>, unsigned char>::rebind<unsigned char>'
  [pyzmq-24.0.1]     /usr/include/c++/13/bits/stl_vector.h:88:21:   required from 'struct std::_Vector_base<unsigned char, zmq::secure_allocator_t<unsigned char> >'
  [pyzmq-24.0.1]     /usr/include/c++/13/bits/stl_vector.h:425:11:   required from 'class std::vector<unsigned char, zmq::secure_allocator_t<unsigned char> >'
  [pyzmq-24.0.1]     bundled/zeromq/src/curve_client_tools.hpp:64:76:   required from here
  [pyzmq-24.0.1]     /usr/include/c++/13/bits/alloc_traits.h:70:31: error: static assertion failed: allocator_traits<A>::rebind_alloc<A::value_type> must be A
  [pyzmq-24.0.1]        70 |                         _Tp>::value,
  [pyzmq-24.0.1]           |                               ^~~~~

https://github.com/sagemath/sage/actions/runs/5706565894/job/15486087970#step:8:2141
@mkoeppe
Copy link
Contributor Author

mkoeppe commented Aug 8, 2023

Upgrading to pyzmq-25.1.0 does not fix this

@mkoeppe mkoeppe changed the title fedora-{38,39}-minimal: pyzmq build failure {fedora-{38,39},ubuntu-mantic}-minimal: pyzmq build failure Nov 6, 2023
@mkoeppe mkoeppe added this to the sage-10.2 milestone Nov 6, 2023
@mkoeppe mkoeppe changed the title {fedora-{38,39},ubuntu-mantic}-minimal: pyzmq build failure {fedora-{38,39},ubuntu-mantic,debian-trixie}-minimal: pyzmq build failure Nov 6, 2023
vbraun pushed a commit to vbraun/sage that referenced this issue Nov 9, 2023
dfde868 
sagemathgh-36663: Upgrade `zeromq` to 4.3.5, `pyzmq` to 25.1.1, patch out broken tests in setup
    
<!-- ^^^^^
Please provide a concise, informative and self-explanatory title.
Don't put issue numbers in there, do this in the PR body below.
For example, instead of "Fixes sagemath#1234" use "Introduce new method to
calculate 1+1"
-->
<!-- Describe your changes here in detail -->
Patches generated from
zeromq/pyzmq@main...mkoeppe:pyzmq:sage_fixes

<!-- Why is this change required? What problem does it solve? -->
<!-- If this PR resolves an open issue, please link to it here. For
example "Fixes sagemath#12345". -->
- Fixes sagemath#36027
- Part of sagemath#35092
<!-- If your change requires a documentation PR, please link it
appropriately. -->

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->
<!-- If your change requires a documentation PR, please link it
appropriately -->
<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
<!-- Feel free to remove irrelevant items. -->

- [x] The title is concise, informative, and self-explanatory.
- [ ] The description explains in detail what this PR is about.
- [ ] I have linked a relevant issue or discussion.
- [ ] I have created tests covering the changes.
- [ ] I have updated the documentation accordingly.

### ⌛ Dependencies

<!-- List all open PRs that this PR logically depends on
- sagemath#12345: short description why this is a dependency
- sagemath#34567: ...
-->

<!-- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->
    
URL: sagemath#36663
Reported by: Matthias Köppe
Reviewer(s): John H. Palmieri, Matthias Köppe
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
c: packages: standard
Projects
None yet
Milestone
sage-10.2
Development

Successfully merging a pull request may close this issue.

Upgrade zeromq to 4.3.5, pyzmq to 25.1.1, patch out broken tests in setup mkoeppe/sage
1 participant
@mkoeppe