Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can cargo expand be made available in the standard build environment? #2724

Open
freshtonic opened this issue Jan 17, 2025 · 1 comment
Open

Comments

@freshtonic
Copy link

Crate name

sqltk

Build failure link

https://docs.rs/crate/sqltk/0.4.0/builds/1636051

Additional details

sqltk's build.rs requires cargo expand to be available.

My testing indicates that it is not installed on the build environment, and lack of network access in build.rs prevents me from installing it.

Is it actually installed, and perhaps I'm invoking it incorrectly? If not, can I request that it be added to the standard build environment?

Kind regards,

James.

@syphar
Copy link
Member

syphar commented Jan 17, 2025

For now I don't see us adding a third party cargo extension to the compiler toolchain we use to build docs. Network access won't be allowed for security reasons.

From what I see, you could either drop down to directly using cargo rustc --profile=check -- -Zunpretty=expanded,

or do similar things as other more complex libraries, like using conditional compilation to make your docs build without the need of cargo expand. Often the public documented interface doesn't need the build script output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants