[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