PLAT-78282: mksnapshot from webruntime fails to run when host and tar…

[shr-project]

…get have different glibc version

show the whole command line used to call mksnapshot for easier debugging when it fails

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[MikyungKim]

It's included in #90. Closing this in favor of #90.

[shr-project]

It's included in #90. Closing this in favor of #90.

https://github.com/enactjs/dev-utils/pull/90/files#diff-3b66dd15d3130b884243a4f217accfb7c44ee30bef1a0e1329a822d551964c1eR146

doesn't seem to have the actual output as it was in this PR:

 \nstdout: "' + child.stdout + '"\nstderr: "' + child.stderr + '"

Does your version print any output from the mksnapshot call?

[MikyungKim]

@shr-project, Actually, the output from the mksnapshot call has already been showing. https://github.com/enactjs/cli/blob/master/commands/pack.js#L184

So with your suggestion, it will double print the output like below.

Snapshot blob generation "mksnapshot --random-seed=314159265 --startup-blob=snapshot_blob.bin --no-use_ic --abort_on_uncaught_exception --turbo_instruction_scheduling --no-native-code-counters main.js" in "dist" directory failed:
stdout: "null"
stderr: "null"
Failed to compile.
null
null

So I've removed the redundant information. Does this make sense to you?

[shr-project]

So I've removed the redundant information. Does this make sense to you?

OK, maybe that changed in last 2 years since I was debugging this and creating this PR.