Allow offline mode via environment variable

[theCapypara]

Allow offline mode via environment variable

[theCapypara]

This does indeed work, thank you!

If anybody needs a reference: flathub/org.skytemple.Randomizer@2b3d4ce#diff-3e722fa1f0525d390439912146aef963826ca681721f98919ae5d98910855071R197-R240

[theCapypara]

Maturin uses config-settings, setuptools-rust could probably do the same.

[davidhewitt]

The most portable way seems to be to send a PR to cargo to support CARGO_OFFLINE directly, if you're open to that?

[davidhewitt]

Alternatively maybe something like #327 (comment) using the config settings.

[theCapypara]

If CARGO_OFFLINE would be OK I'd be interested in giving implementing this a try.

I think this sounds like the easiest and also most expected solution. It's also really easy to implement in any CI pipeline (such as Flatpak manifests) without needing to reach into the requirements of the package that uses setuptools-rust (given that pip doesn't strip environment variables... not sure if it does with build isolation).

[davidhewitt]

Actually now that I look at this again I realise what you want is already present - setting CARGO_NET_OFFLINE=true should set offline mode.

See https://doc.rust-lang.org/cargo/reference/config.html#netoffline

[theCapypara]

Oh, really?
I looked for an option like this. Now it's been a short while, but I wonder how I missed this, or if I tried and it didn't work. I'll try again and update.