Quoting Pirate Praveen (2020-05-14 14:18:20) > On Thu, May 14, 2020 at 1:12 pm, Jonas Smedegaard <jo...@jones.dk> > wrote: > > Quoting Pirate Praveen (2020-05-05 15:11:31) > >> I'm planning to upload node-grunt-babel 8.0 in unstable (it is in > >> experimental currently). I have fixed all the reverse build > >> dependencies and this is the only remaining blocker. I have fixed > >> this package in babel7 branch of salsa. > > > > I doubt the patch was actually tested - I was unable to reproduce a > > succesful build. > > > > It built fine for me, attaching sbuild log.
Thanks, that was (not enough, but) very helpful: autopkgtest still fails for me, and I would appreciate if you could test if you agree that it fails for you as well. Turns out there are two issues here: With Babel 6, "babeljs --out-dir dist src/foo.js src/deeper/bar.js" creates dist/deeper/bar.js, where with Babel 7 that command instead creates dist/bar.js - your latest patch changes to an alternative form "babeljs --out-dir dist src" which works. Thanks for that! Another issue (also experienced with your earlier gulp patch) is that the _content_ produced by Babel 6 works with the (arguably crude and unsupported) autopkgtest, but with Babel 7 it fails like this: Error: Cannot find module ./utils at modRequire (/usr/share/javascript/webrtc-adapter/adapter.js:10:22) at Object.1 (/usr/share/javascript/webrtc-adapter/adapter.js:40:37) at req (/usr/share/javascript/webrtc-adapter/adapter.js:6:22) at modRequire (/usr/share/javascript/webrtc-adapter/adapter.js:11:14) at Object.0../adapter_factory.js (/usr/share/javascript/webrtc-adapter/adapter.js:26:24) at req (/usr/share/javascript/webrtc-adapter/adapter.js:6:22) at 0../adapter_factory.js (/usr/share/javascript/webrtc-adapter/adapter.js:2:20) at Object.<anonymous> (/usr/share/javascript/webrtc-adapter/adapter.js:14:3) at Module._compile (internal/modules/cjs/loader.js:778:30) at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10) The generated code is also suspicously much much smaller. This pattern works with Babel 6: babeljs --cwd src/deeper --out-dir dist src/foo.js src/deeper/bar.js ...but there is no "cwd" command-line option with Babel 7, it seems. -- * Jonas Smedegaard - idealist & Internet-arkitekt * Tlf.: +45 40843136 Website: http://dr.jones.dk/ [x] quote me freely [ ] ask before reusing [ ] keep private
signature.asc
Description: signature