Skip to content
This repository has been archived by the owner on Aug 4, 2021. It is now read-only.

Error with entry file paths beginning with ./ #43

Closed
maoberlehner opened this issue Jun 8, 2016 · 2 comments
Closed

Error with entry file paths beginning with ./ #43

maoberlehner opened this issue Jun 8, 2016 · 2 comments
Labels

Comments

@maoberlehner
Copy link

maoberlehner commented Jun 8, 2016

Does not work:

rollup --config --output app/js/index.js ./js/index.js

Works:

rollup --config --output app/js/index.js js/index.js

rollup.config.js

import commonjs from 'rollup-plugin-commonjs';
import nodeResolve from 'rollup-plugin-node-resolve';

export default {
  plugins: [
    nodeResolve(),
    commonjs()
  ]
};

Error message:

Path must be a string. Received undefined
TypeError: Path must be a string. Received undefined
    at assertPath (path.js:7:11)
    at Object.resolve (path.js:1148:7)
    at resolveId$1 (/Users/moberlehner/Sites/vagrant/devbox/modularis/node_modules/rollup-plugin-node-resolve/dist/rollup-plugin-node-resolve.cjs.js:42:15)
    at /Users/moberlehner/Sites/vagrant/devbox/modularis/node_modules/rollup/src/utils/first.js:8:31
    at process._tickCallback (internal/process/next_tick.js:103:7)
    at Function.Module.runMain (module.js:577:11)
    at startup (node.js:160:18)
    at node.js:456:3

Is this an error or should I use paths not beginning with ./ in general?

ronzeidman added a commit to ronzeidman/ng2-ui-auth that referenced this issue Jun 11, 2016
updated rollup & typescript version + fixed rollup/rollup-plugin-node-resolve#43
@lammas
Copy link
Contributor

lammas commented Jun 17, 2016

Should we move to not using ./ in import paths or will this be fixed?

lammas added a commit to lammas/rollup-plugin-node-resolve that referenced this issue Jun 17, 2016
lammas added a commit to lammas/rollup-plugin-node-resolve that referenced this issue Jun 17, 2016
ronzeidman added a commit to ronzeidman/ng2-ui-auth that referenced this issue Jul 9, 2016
updated rollup & typescript version + fixed rollup/rollup-plugin-node-resolve#43
@TrySound
Copy link
Member

Fixed in 1.7.2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants