fix(r/sedonadb): Add -Wno-pedantic to avoid compile error #181
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.
Fix #164
Background
When
-pedanticflag is passed viaCFLAGS, the aws-lc-sys crate cannot be compiled.R CMD INSTALLis not the case, so the CI is fine without this. However, when using devtools or pak, this flag is passed via pkgbuild.Why? Pkgbuild's comment says this flag is to "enforce good coding practice" to the R package developers. Also, it says this flag is used on CRAN check. But, considering this package doesn't strongly aim for CRAN (at least at the moment), probably it's fine to ignore the warnings.
https://github.com/r-lib/pkgbuild/blob/d88519a177c18c6a2701e1eaf9fb22d37f9580b9/R/compiler-flags.R#L3-L5
Details
This pull request simply tries to override
-pedanticby-Wno-pedantic. In the currentMakevars.win.in, I usedsubstrto remove-pedanticflag. However, it seems appending this flag is simpler if this works.Some more tweaks on
Makevars.win.in:ALL_CPPFLAGSwithCFLAGS.CPPFLAGSis C Pre-Processor flags, so I thinkCFLAGSis appropriate here.AWS_LC_SYS_CFLAGS