Skip to content

Revert exprengine changes to fix CI on appveyor - #3668

Closed
pfultz2 wants to merge 2 commits into
cppcheck-opensource:mainfrom
pfultz2:ci-expr-engine-fix
Closed

Revert exprengine changes to fix CI on appveyor#3668
pfultz2 wants to merge 2 commits into
cppcheck-opensource:mainfrom
pfultz2:ci-expr-engine-fix

Conversation

@pfultz2

Copy link
Copy Markdown
Collaborator

No description provided.

@firewave

Copy link
Copy Markdown
Collaborator

I think it's just incomplete support for constexpr.

Since it was said that removing support for older Visual Studio versions is fine (see #3596 (comment)) we could probably keep this code and just disconnect Appveyor from this repository (but keep the configuration files until everything is moved to GitHub actions) since older GCC versions are fine.

@pfultz2

Copy link
Copy Markdown
CollaboratorAuthor

Alright, here is a PR that removes appveyor:

#3669

@danmar

Copy link
Copy Markdown
Collaborator

sorry for this trouble I try to fix it ASAP

@firewave

Copy link
Copy Markdown
Collaborator

Since this now builds we can close this.

@danmardanmar closed this Jan 4, 2022
Sign up for freeto 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.

3 participants

@pfultz2@firewave@danmar