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

bump simplecpp to 1.3.2 #7299

Merged
merged 1 commit into from
Feb 12, 2025
Merged

bump simplecpp to 1.3.2 #7299

merged 1 commit into from
Feb 12, 2025

Conversation

danmar
Copy link
Owner

@danmar danmar commented Feb 12, 2025

No description provided.

@danmar danmar changed the title bump simplecpp to 1.3.0 bump simplecpp to 1.3.1 Feb 12, 2025
@danmar danmar force-pushed the bump-simplecpp branch 2 times, most recently from ce38f9d to 04e38c4 Compare February 12, 2025 14:39
@danmar
Copy link
Owner Author

danmar commented Feb 12, 2025

@firewave do you think that the selfcheck failure is caused by the simplecpp? I don't see how it can be?

@danmar
Copy link
Owner Author

danmar commented Feb 12, 2025 •
edited
Loading

@firewave do you think that the selfcheck failure is caused by the simplecpp? I don't see how it can be?

I can reproduce the preprocessorErrorDirective in path.cpp locally :-(

@firewave
Copy link
Collaborator

Looks like a bug. We define __GNUC__ for the selfcheck and there is a preprocessor check for that in the code that reports the error.

@danmar
Copy link
Owner Author

danmar commented Feb 12, 2025

Looks like a bug.

yes. I'll revert some changes in simplecpp and bump a new version..

@danmar danmar changed the title bump simplecpp to 1.3.1 bump simplecpp to 1.3.2 Feb 12, 2025
@danmar danmar merged commit ad530b0 into danmar:main Feb 12, 2025
60 checks passed
@danmar danmar deleted the bump-simplecpp branch February 23, 2025 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danmar @firewave