Upgrade to node v10 generates errors


#1

I’ve upgraded node: node -v
v10.1.0

Now php bakery build-assets reports this:

$ php bakery build-assets

UserFrosting's Assets Builder
=============================

Installing npm dependencies
---------------------------

> npm install
up to date in 3.404s
[!] 40 vulnerabilities found [4797 packages audited]
    Severity: 22 low | 4 moderate | 14 high
    Run `npm audit` for more detail


Installing assets bundles
-------------------------

> npm run uf-assets-install

> @userfrosting/[email protected] uf-assets-install /home/jose/yovoy/aig/sinma/repos/UF/build
> gulp bower-install

gulp[5156]: ../src/node_contextify.cc:628:static void node::contextify::ContextifyScript::New(const v8::FunctionCallbackInfo<v8::Value>&): Assertion `args[1]->IsString()' failed.
 1: node::Abort() [gulp]
 2: 0x557220a7229b [gulp]
 3: node::contextify::ContextifyScript::New(v8::FunctionCallbackInfo<v8::Value> const&) [gulp]
 4: v8::internal::FunctionCallbackArguments::Call(v8::internal::CallHandlerInfo*) [gulp]
 5: 0x557220c7ce97 [gulp]
 6: 0x557220c7d68f [gulp]
 7: 0xc1f1ae0427d

Testing assets installation
---------------------------

 [OK] Assets install looks successful

#2

Well. That’s a new one. If it worked in versions prior to 10, the obvious solution is to downgrade. Can’t say anything about a long term solution right now (not unless another contributor feels like taking a stab at it).


#3

Confirmed bug. I have opened an issue here: https://github.com/userfrosting/UserFrosting/issues/882. So, let us continue discussion in the issue tracker.


#4

I’ve added my workaround here.

php bakery build-assets -c now works as expected.