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

Remove leftover usage of the __std_stream header #3663

Merged
merged 1 commit into from
Oct 1, 2023

Conversation

lazka
Copy link
Contributor

@lazka lazka commented Oct 1, 2023

2dd4fa8 removed all usage of __std_stream because it is no longer available with clang v17. That commit missed one place where the header was still used (only used when building with -DFMT_MODULE=ON). Remove it there too.

See #3654

2dd4fa8 removed all usage of __std_stream because
it is no longer available with clang v17. That commit missed one place
where the header was still used (only used when building with -DFMT_MODULE=ON).
Remove it there too.

See fmtlib#3654
@vitaut vitaut merged commit b3bf23f into fmtlib:master Oct 1, 2023
40 checks passed
@vitaut
Copy link
Contributor

vitaut commented Oct 1, 2023

Thanks!

ckerr pushed a commit to transmission/fmt that referenced this pull request Nov 7, 2023
2dd4fa8 removed all usage of __std_stream because
it is no longer available with clang v17. That commit missed one place
where the header was still used (only used when building with -DFMT_MODULE=ON).
Remove it there too.

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

Successfully merging this pull request may close these issues.

2 participants