Skip to content

Making the recursive dependency an optional feature #13766

@andygrove

Description

@andygrove

Is your feature request related to a problem or challenge?

The addition of the recursive crate as a dependency in #13310 may cause issues for some downstream projects.

For example, in the DataFusion Comet subproject, we started to see CI failures when running with miri.

error: unsupported operation: can't call foreign function `rust_psm_stack_pointer` on OS `linux`
    --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/psm-0.1.24/src/lib.rs:319:14
     |
319  |     unsafe { rust_psm_stack_pointer() }
     |              ^^^^^^^^^^^^^^^^^^^^^^^^ can't call foreign function `rust_psm_stack_pointer` on OS `linux`
  

For now, we disabled the miri checks, but we would prefer to keep them and opt out of the recursive feature when testing in CI.

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions