You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
[R][Packaging] Address CRAN check NOTE about portable file names #46947
The note points to the issues being with some recent Arrow C++ check-ins:
2025-06-23T00:43:52.3782199Z * checking for portable file names ... NOTE
2025-06-23T00:43:52.3789516Z Found the following non-portable file paths:
2025-06-23T00:43:52.3790791Z arrow/tools/cpp/src/arrow/flight/sql/odbc/flight_sql/include/flight_sql/config/connection_string_parser.h
2025-06-23T00:43:52.3792457Z arrow/tools/cpp/src/arrow/flight/sql/odbc/flight_sql/include/flight_sql/ui/dsn_configuration_window.h
2025-06-23T00:43:52.3794282Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/attribute_utils.h
2025-06-23T00:43:52.3795845Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/encoding_utils.h
2025-06-23T00:43:52.3797434Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/odbc_connection.h
2025-06-23T00:43:52.3798970Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/odbc_descriptor.h
2025-06-23T00:43:52.3800482Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/odbc_environment.h
2025-06-23T00:43:52.3801924Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/odbc_handle.h
2025-06-23T00:43:52.3802941Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/odbc_statement.h
2025-06-23T00:43:52.3803798Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/odbc_impl/type_utilities.h
2025-06-23T00:43:52.3804631Z arrow/tools/cpp/src/arrow/flight/sql/odbc/odbcabstraction/include/odbcabstraction/spi/result_set_metadata.h
2025-06-23T00:43:52.3805301Z Tarballs are only required to store paths of up to 100 bytes and cannot
2025-06-23T00:43:52.3805750Z store those of more than 256 bytes, with restrictions including to 100
2025-06-23T00:43:52.3806104Z bytes for the final component.
2025-06-23T00:43:52.3806680Z See section ‘Package structure’ in the ‘Writing R Extensions’ manual.
These are new files (since #40939) so the new failure makes sense.
At the very least, we need to find a way to silence the NOTE but it would be good to check if this causes any other issues.
In addition, some applications on Windows can only work with path names of certain length, following an earlier limit in the Windows operating system. Packages are normally distributed as tarballs, and these have a limit on path lengths. So, to be friendly to users who themselves may want to use a relatively long path where they extract the package, and for maximal portability, 100 bytes.
Activity
amoeba commentedon Jun 30, 2025
Quote from the relevant section in WRE (link):
amoeba commentedon Jun 30, 2025
I don't think these files are currently or will be used in a source build so I think we can just add this path prefix to https://github.com/apache/arrow/blob/main/r/bootstrap.R#L54.
GH-46947: [R][Packaging] Add src/arrow/flight/sql/odbc to source excl…
amoeba commentedon Jul 1, 2025
Issue resolved by pull request 46948
#46948