Add missing iostream header #1283

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

Open

rurban wants to merge 1 commit into jbeder:master from SpexAI:iostream

rurban commented May 27, 2024

No description provided.


          Add missing iostream header

ca5d463

Owner

jbeder commented May 27, 2024

Why?

Author

rurban commented May 27, 2024 via email

Try with gcc 14 Jesse Beder ***@***.***> schrieb am Mo., 27. Mai 2024, 16:33:

Why? — Reply to this email directly, view it on GitHub <#1283 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAKGUKKEFYIIXKD75S26O3ZEMXTHAVCNFSM6AAAAABIK4L4SCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZTGQ3TQNZRGI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

Owner

jbeder commented May 27, 2024

It's customary to do things like post the exact error message.

Author

rurban commented Jun 3, 2024

Ok, this has to be reworked.
It fails on my ubuntu 24.04 clang-tidy with missing iostream:

/usr/local/include/yaml-cpp/parser.h:10:10: error: 'iostream' file not found [clang-diagnostic-error]
   10 | #include <iostream>

but my fedora core 40 with gcc 14 warns about a missing iostream header. will check there

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

Labels

None yet