aurelia / bundler

A library for bundling JavaScript, HTML and CSS for use with SystemJS.
MIT License
37 stars 25 forks source link

fix to check for current path "." in file list #63

Closed donald-slagle closed 8 years ago

donald-slagle commented 8 years ago

Fixed the following error when running the bundler with html import enabled.

[11:45:10] Starting 'bundle'...
[11:45:11] 'bundle' errored after 1.25 s
[11:45:11] Error: EISDIR, illegal operation on a directory
    at Error (native)
    at Object.fs.readSync (fs.js:552:19)
    at Object.fs.readSync (d:\Project\node_modules\gulp-yu
idoc\node_modules\vinyl-fs\node_modules\graceful-fs\polyfills.js:218:23)
    at Object.fs.readSync (d:\Project\node_modules\gulp-yu
idoc\node_modules\yuidocjs\node_modules\graceful-fs\polyfills.js:218:23)
    at Object.fs.readFileSync (fs.js:389:28)
    at d:\Project\node_modules\aurelia-bundler\dist\html-i
mport-template-bundler.js:62:35
    at Array.forEach (native)
    at Object.bundle (d:\Project\node_modules\aurelia-bund
ler\dist\html-import-template-bundler.js:60:6)
    at _bundleHtmlImportTemplate (d:\Project\node_modules\
aurelia-bundler\dist\index.js:68:15)
    at d:\Project\node_modules\aurelia-bundler\dist\index.
js:54:18
ahmedshuhel commented 8 years ago

Thanks for the PR. A little more information will be greatly appreciated to get it merged. I have left some comment.

EisenbergEffect commented 8 years ago

@ahmedshuhel Was this fixed. Can it be closed?

ahmedshuhel commented 8 years ago

@EisenbergEffect I am not yet sure about it. I need to test it to find out exactly why we need to check for . here.

EisenbergEffect commented 8 years ago

How are we doing on this?

ahmedshuhel commented 8 years ago

@EisenbergEffect Could not complete test yet.

EisenbergEffect commented 8 years ago

@ahmedshuhel This PR has been here for months. Please address this.