GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — selfcheck.sh: also run with system includes made available by firewave · Pull Request #438 · cppcheck-opensource/simplecpp

Conversation

@firewave

@firewave firewave commented Jun 11, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@firewave

This comment was marked as resolved.

Sign in to view
@firewave

firewave commented Jun 11, 2025

Copy link
Copy Markdown
Collaborator Author

This fails with

/usr/include/c++/15.1.1/ext/atomicity.h:39: syntax error: failed to evaluate #if condition, division/modulo by zero

The code in question:

#if __has_include(<sys/single_threaded.h>)
# include <sys/single_threaded.h>
#endif
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@firewave
firewave force-pushed the selfcheck-sys-x branch 2 times, most recently from 508ffaf to fdae274 Compare June 11, 2025 11:24
@firewave firewave mentioned this pull request Jun 12, 2025
syntax error caused by unconditional __has_include usage #439
Closed
@firewave
firewave force-pushed the selfcheck-sys-x branch 2 times, most recently from dd1cc3f to fbb316f Compare August 4, 2025 10:59
@firewave

firewave commented Aug 4, 2025

Copy link
Copy Markdown
Collaborator Author

It now fails with

/usr/include/c++/v1/__availability:78: syntax error: failed to evaluate #if condition, undefined function-like macro invocation: __has_feature( ... )

or

/usr/include/c++/v1/__assert:29: syntax error: failed to evaluate #if condition, undefined function-like macro invocation: __has_builtin( ... )
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

This was referenced Aug 4, 2025
syntax error caused by unconditional __has_builtin usage #473
Open
syntax error caused by unconditional __has_feature usage #474
Open
@firewave
firewave force-pushed the selfcheck-sys-x branch 8 times, most recently from 8af58e9 to a21478b Compare August 6, 2025 08:04
@firewave firewave mentioned this pull request Aug 6, 2025
headers included with relative paths are not found #482
Closed
@firewave
firewave force-pushed the selfcheck-sys-x branch 5 times, most recently from 34115d6 to d7eaeb7 Compare August 6, 2025 09:16
@firewave firewave mentioned this pull request Aug 6, 2025
syntax error caused by unconditional __has_include_next usage #484
Open
@firewave
firewave force-pushed the selfcheck-sys-x branch 5 times, most recently from eebcb44 to 2760f96 Compare August 7, 2025 09:35
@firewave
firewave marked this pull request as ready for review September 1, 2025 15:47
This was referenced Sep 1, 2025
feat: Ordered search paths for -I, -isystem, -F, -iframework + Darwin framework support (with legacy -I back-compat) #511
Open
includes not found on MSYS2 #524
Closed
syntax error caused by unconditional __has_cpp_attribute usage #534
Open
syntax error caused by unconditional __has_attribute usage #535
Open
syntax error caused by unconditional __has_extension usage #536
Open
This was referenced Sep 10, 2025
fix #14117: --no-check-unused-templates does not apply to templated functions with declaration specifiers cppcheck-opensource/cppcheck#7811
Draft
main.cpp: error out when file/path provided by -I or -include=, or the input does not exist #435
Merged
danmar
danmar reviewed Sep 10, 2025
View reviewed changes

@danmar danmar left a comment

Copy link
Copy Markdown
Collaborator

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.

hmm.. my problem with this could be that maybe there will be problem in CI and no problem locally .. or no problem in CI today and then we get problems a year from now.

are the headers in a github runner fixed so they are never changed? or can they be changed at anytime (from one day to the next)?

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@firewave

firewave commented Sep 10, 2025

Copy link
Copy Markdown
Collaborator Author

hmm.. my problem with this could be that maybe there will be problem in CI and no problem locally ..

Yes, that is a bit of a shortcoming. That might need some adjustment if encountered. But as we are just checking for errors and not the actual output it should be fine.

or no problem in CI today and then we get problems a year from now.

That can happen with anything in any workflow because the runners (and all other parts of the GitHub workflows) are constantly changing.

are the headers in a github runner fixed so they are never changed? or can they be changed at anytime (from one day to the next)?

No, they are not. But as mentioned above that is an issue we are constantly facing with any part of GitHub workflows.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@firewave

This comment was marked as resolved.

Sign in to view
@firewave
firewave force-pushed the selfcheck-sys-x branch from e779362 to b0b0573 Compare September 10, 2025 12:44
@firewave
firewave marked this pull request as draft September 11, 2025 09:51
@firewave

firewave commented Sep 11, 2025

Copy link
Copy Markdown
Collaborator Author

After integrating this with MinGW I have a few cleanups for this.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@firewave

firewave commented Sep 11, 2025

Copy link
Copy Markdown
Collaborator Author

Yes, that is a bit of a shortcoming. That might need some adjustment if encountered.

This can be improved upon later by getting the built-in include paths from the compiler (if possible).

I think I have that figured out as well.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@firewave

This comment was marked as resolved.

Sign in to view
@firewave
firewave force-pushed the selfcheck-sys-x branch 4 times, most recently from 81a82d0 to 6abcbe6 Compare September 16, 2025 18:09
@firewave firewave mentioned this pull request Sep 16, 2025
added MinGW workflow #475
Merged
firewave
firewave commented Sep 16, 2025
View reviewed changes
Comment thread selfcheck.sh
@@ -1,11 +1,111 @@
#!/bin/sh
#!/bin/bash

@firewave firewave Sep 16, 2025

Copy link
Copy Markdown
Collaborator Author

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.

Needed so the <<< syntax works.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@firewave
firewave force-pushed the selfcheck-sys-x branch from 6abcbe6 to 31088a9 Compare October 1, 2025 15:30
@firewave
firewave marked this pull request as ready for review October 1, 2025 15:30
firewave added 2 commits October 1, 2025 17:38
@firewave
selfcheck.sh: also run with system includes made available
196f245
@firewave
CI-unixish.yml: install unversioned libc++-dev package
35346ef
@firewave
firewave force-pushed the selfcheck-sys-x branch from 31088a9 to 35346ef Compare October 1, 2025 15:38
danmar
danmar approved these changes Oct 2, 2025
View reviewed changes
@firewave
firewave merged commit a5fdea9 into cppcheck-opensource:master Oct 2, 2025
1 check passed

Uh oh!

There was an error while loading. Please reload this page.

@firewave
firewave deleted the selfcheck-sys-x branch October 2, 2025 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danmar danmar danmar approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@firewave @danmar
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.