On Mon, Nov 25, 2019 at 10:06:22PM +0000, Stuart Henderson wrote: > +cc semarie@, wondering if you have any suggestions?
CONFIGURE_STYLE = cargo is missing. so cargo isn't configured properly. I will take look to cargo.devel.mk. cargo has, since few releases, a new --offline flag, which should avoid the need to use pf(4) to catch network problem (cargo will just abort without trying network). > I imported this, but unfortunately when I tested before import, it was > on a machine not setup with _pbuild blocked from network access - now I'm > trying to build on another machine I see that it downloads during the build > which is not allowed. > > Also it seems some of the crates listed by modcargo-gen-crates are missing? > > Diff below adds the crates but there's still a problem as it tries to > fetch: > > ---------- > ===> Building for ncspot-0.1.1p0 > Updating crates.io index > warning: spurious network error (2 tries remaining): failed to connect to > github.com: Permission denied; class=Os (2) > warning: spurious network error (1 tries remaining): failed to connect to > github.com: Permission denied; class=Os (2) > error: failed to fetch `https://github.com/rust-lang/crates.io-index` > > Caused by: > failed to connect to github.com: Permission denied; class=Os (2) > ---------- > > If I allow it to have network, after that it appears to re-download the > various other crates that it should already find setup by cargo.port.mk.. > > I suspect part of the problem is to do with patching Cargo.toml, maybe > we want something like "MODCARGO_FEATURES = portaudio_backend > cursive/pancurses-backend" instead of patching, but even if I remove > the patch I still see it attempt and fail to fetch crates.io-index. Also features things could be improved. It was added when no port using it, so it isn't really exerciced. But it should be possible to avoid patching Cargo.toml to have the same result. Thanks. -- Sebastien Marie
