-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
cppcheck: put all resources back in bin folder #15424
Conversation
otherwise cppcheck does not find them fixes conan-io#15421
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.
I'll submit another PR after this one, there are several issues in test package logic and definition of an env var.
@@ -44,7 +44,7 @@ def generate(self): | |||
tc.variables["HAVE_RULES"] = self.options.have_rules | |||
tc.variables["USE_MATCHCOMPILER"] = "Auto" | |||
tc.variables["ENABLE_OSS_FUZZ"] = False | |||
tc.variables["FILESDIR"] = os.path.join(self.package_folder, "res").replace('\\', '/') | |||
tc.variables["FILESDIR"] = os.path.join(self.package_folder, "bin").replace('\\', '/') |
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.
maybe extend the test to check that nobody will break it again?
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.
unfortunately I don't think we can make a breaking test, because cppcheck stores the absolute path to this FILESDIR folder at build time and uses it as a fallback if the .cfg files cannot be found in the folder next to the cppcheck executable. It means we can only break by downloading an existing package binary into a package folder different from the one used during build :/
Conan v1 pipeline ✔️All green in build 1 ( |
otherwise cppcheck does not find them fixes conan-io#15421
otherwise cppcheck does not find them
fixes #15421
Specify library name and version: cppcheck/*