When setting lib to project root, don't introduce extra slashes

[mstade]

If you specify -L <project root> when running ez-build, you'll run into the situation of module names including an extra slash. To fix this we simply check if opts.lib is set (it'll always be relative to project root, so it'll be empty) and if not we only use the project name as the module root.

[mstade]

@FabienDeshayes this should solve your problem.

[FabienDeshayes]

:+1:

[Poetro]

:+1:

[FabienDeshayes]

I think there are 3 issues with this release (I'm on 0.3.1-fix-extra-slash-in-module-root.1):

• the optimised-modules.json is a bit mad and doesn't ignore all the correct files:

[
  "am-authentication/c:/dev/work/authentication/a2",
  "am-authentication/c:/dev/work/authentication/ez-build-debug",
  "am-authentication/c:/dev/work/authentication/from-params",
  "am-authentication/c:/dev/work/authentication/hello",
  "am-authentication/c:/dev/work/authentication/README",
  "am-authentication/c:/dev/work/authentication/to-params"
]

• running in interactive mode and enter Ctrl + C generates an error with some more info in ez-build-debug.log. Extract:

Rejection #1:
{ [Error: ENOENT: no such file or directory, open 'c:\dev\work\authentication\s']
  errno: -4058,
  code: 'ENOENT',
  syscall: 'open',
  path: 'c:\\dev\\work\\authentication\\s',
  build: { input: 's' } }

//etc 12 times, with a different letter each time

• it seems that even the simplest one doesn't work properly when requiring.

require('my-project/hello-world') // Object {}

Not sure about that last one as the output seems correct.

TBC next week..

[mstade]

Thanks @FabienDeshayes.

1. The modules part of the optimised-modules.json moniker is a bit of a misnomer – it's really just a listing of everything. Because you're not putting the output files in a dedicated lib directory, it'll include other files like the README and such as well. The paths however are obviously incorrect, they should be logical names, not absolute paths and certainly not in the middle of everything – I'll fix that. But other than that, it's not really doing anything wrong at this point.
2. I'm unable to repro this, anything else you can think of to trigger this?
3. I don't really know what this means, can you elaborate?

[mstade]

Bumping the minor version, since this adds new features (albeit backwards compatible.)