Skip to content

[R][Packaging] Address CRAN check NOTE about portable file names #46947

Closed

Description

@amoeba

Describe the enhancement requested

We're seeing a NOTE about non-portable file paths in recent CI jobs,

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.

Component(s)

Packaging, R

Activity

  1. amoeba commented on Jun 30, 2025

    @amoeba
    MemberAuthor

    Quote from the relevant section in WRE (link):

    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.

  2. amoeba commented on Jun 30, 2025

    @amoeba
    MemberAuthor

    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.

  3. added this to the 21.0.0 milestone on Jun 30, 2025
  4. added a commit that references this issue on Jul 1, 2025
  5. amoeba commented on Jul 1, 2025

    @amoeba
    MemberAuthor

    Issue resolved by pull request 46948
    #46948

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions