search

SitePen / dojo-amd-converter

A legacy dojo to AMD conversion utility.
Other
11 stars 5 forks source link

Path parsing issues #28

Closed emecell closed 11 years ago

emecell commented 11 years ago

System: OSX 10.8.3 using node 0.10.2

The first call of parser.js#L13 fails for my system since the first call does not pass in any path so I patched it locally to:

path = pathUtil.join(parent, path || "");

but then parser.js#L54 truncates the path wrong so I had to remove the + 1 like so:

path = path.slice(config.srcDir.length);.

dylans commented 11 years ago

Should be pretty easy to fix... care to provide a pull request? Do you have a Dojo Foundation CLA on file?

emecell commented 11 years ago

Yup, I do have a CLA on file.

emecell commented 11 years ago

Hrm, looks like github created another issue. I'll close this one for now.