Allow additional Node configure args with PKG_BUILD_CONFIGURE_ARGS

[mladkau]

This PR adds the ability to specify additional build options for the node build via an environment variable.

For example:

export PKG_BUILD_PARAM="--partly-static"
pkg . --build -t node14-linux-x64

This can be used to address: vercel/pkg#555
The added functionality is a bit wider than /pull/72

[jesec]

@mladkau I made some updates: a. rename the env variable to PKG_BUILD_CONFIGURE_ARGS, 2. apply to windows build as well.

[robertsLando]

This needs to be documented in pkg and/or here. Maybe in this section: https://github.com/vercel/pkg#environment (waiting for a pr there before merge)

[jesec]

This needs to be documented in pkg and/or here. Maybe in this section: https://github.com/vercel/pkg#environment (waiting for a pr there before merge)

I lean toward hiding this as an advanced argument.

[robertsLando]

I understand it but we should start document more things in both pkg and pkg-fetch projects otherwise it could be hard for new devs to start developing or even users that are looking for those advanced options need to read the code in order to know them

I have spent many time looking at the code of both projects to undestand how they work at low level and some things are still obscure or not 100% clear to me

[leerob]

Any update here?

[robertsLando]

I think this could land but needs docs...

[jesec]

This needs to be documented in pkg and/or here. Maybe in this section: https://github.com/vercel/pkg#environment (waiting for a pr there before merge)

I lean toward hiding this as an advanced argument.

^

Documenting it means that we are supposed to support it in some ways, but we can't support every exotic Node configure flag (and combination of) out there. The custom binaries would not be supported by us in any way. As such, I think this should be a hidden undocumented option.

[robertsLando]

@leerob Could we merge this?

[robertsLando]

@jesec