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

OS X 12.6 "chained fixups" warning #34533

Closed
jhpalmieri opened this issue Sep 14, 2022 · 14 comments
Closed

OS X 12.6 "chained fixups" warning #34533

jhpalmieri opened this issue Sep 14, 2022 · 14 comments

Comments

@jhpalmieri
Copy link
Member

The upgrade to OS X 12.6 leads to a number of failing doctests because of a warning

ld: warning: -undefined dynamic_lookup may not work with chained fixups

Let's ignore this warning when parsing doctest output.

Component: doctest framework

Author: John Palmieri

Branch: 67c914c

Reviewer: Volker Braun

Issue created by migration from https://trac.sagemath.org/ticket/34533

@jhpalmieri
Copy link
Member Author

comment:1

It would also be nice to know where this is coming from and deal with it at its root.

@jhpalmieri
Copy link
Member Author

Branch: u/jhpalmieri/chained_fixups

@jhpalmieri
Copy link
Member Author

New commits:

67c914ctrac 34533: silence warning during doctesting about "chained fixups"

@jhpalmieri
Copy link
Member Author

Commit: 67c914c

@jhpalmieri
Copy link
Member Author

comment:4

I'm making this a blocker since we currently have failing doctests on a supported platform.

@mkoeppe
Copy link
Contributor

mkoeppe commented Sep 14, 2022

comment:5

I don't know if this really needs to be a blocker

@jhpalmieri
Copy link
Member Author

comment:6

I'm not sure, either, but it would be best to have a release that passes doctests on all supported platforms.

@vbraun
Copy link
Member

vbraun commented Sep 14, 2022

Reviewer: Volker Braun

@vbraun
Copy link
Member

vbraun commented Sep 14, 2022

comment:8

Seems safe enough, and its unlikely to accidentally hide a real error

@jhpalmieri
Copy link
Member Author

comment:9

Thank you (and as I said earlier, it would be nice to understand why this has begun happening and deal with it rather than hide it).

@kwankyu
Copy link
Collaborator

kwankyu commented Sep 16, 2022

comment:10

For your info:

Still I don't understand the warning message and "why this has begun happening" :)

@mkoeppe mkoeppe modified the milestones: sage-9.8, sage-9.7 Sep 16, 2022
@vbraun
Copy link
Member

vbraun commented Sep 17, 2022

Changed branch from u/jhpalmieri/chained_fixups to 67c914c

@jhpalmieri
Copy link
Member Author

comment:13

Follow-up in #34577.

@jhpalmieri
Copy link
Member Author

Changed commit from 67c914c to none

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants