paleolimbot opened a new issue, #657:
URL: https://github.com/apache/sedona-db/issues/657
We have a few warnings in R CMD check that would be nice to clear up:
```
Found the following hidden files and directories:
src/c/.clang-format
src/c/sedona-s2geography/s2geography/.clang-format
src/c/sedona-s2geography/s2geography/.editorconfig
src/c/sedona-s2geography/s2geography/.git
src/c/sedona-s2geography/s2geography/.pre-commit-config.yaml
src/c/sedona-s2geography/s2geometry/.git
src/c/sedona-s2geography/s2geometry/.travis.yml
src/c/sedona-s2geography/s2geography/.github
These were most likely included in error. See section ‘Package
structure’ in the ‘Writing R Extensions’ manual.
* checking for portable file names ... NOTE
Found the following non-portable file paths:
sedonadb/src/rust/sedona-geo-generic-alg/src/algorithm/line_measures/metric_spaces/euclidean/distance.rs
sedonadb/src/rust/sedona-geo-generic-alg/src/algorithm/line_measures/metric_spaces/euclidean/utils.rs
Tarballs are only required to store paths of up to 100 bytes and cannot
store those of more than 256 bytes, with restrictions including to 100
bytes for the final component.
```
For the hidden files I think we just remove these when we run `bootstrap.R`
to create the source tarball.
For the non-portable file paths it's probably just easiest to shorten the
name `sedona-geo-generic-alg` (we only need a few characters back).
While we're here we should see if we can ensure that the r-extended CI job
fails on notes or warnings we don't expect.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
