[OE-core] [PATCH] npm.bbclass: introduce NPM_PXORY
Burton, Ross
ross.burton at intel.com
Wed Jan 9 17:32:40 UTC 2019
On Wed, 9 Jan 2019 at 17:28, Alexander Kanavin <alex.kanavin at gmail.com> wrote:
> > + if [ -n "${NPM_PROXY}" ]; then
> > + npm config set proxy ${NPM_PROXY}
> > + npm config set https-proxy ${NPM_PROXY}
> > + else
> > + npm config set proxy null
> > + npm config set https-proxy null
> > + fi
>
> Isn't it better to set those from http[s]_proxy environment variables
> which are standard?
Please, unless there's a very good reason lets use the proxy variables
that we already export to various other bits of the system.
If npm will respect http_proxy in the environment, then just use
bb.utils.export_proxies.
Ross
More information about the Openembedded-core
mailing list