Version 3.4.2 of mocha just got published.

This version is covered by your current version range and after updating it in your project the build failed.

As mocha is “only” a devDependency of this project it might not break production or downstream projects, but “only” your build or test tools – preventing new deploys or publishes.

I recommend you give this issue a high priority. I’m sure you can resolve this 💪

Your Greenkeeper Bot 🌴

cc @twokul got a sec good sir?

Version 2.4.0 of debug just got published.

This version is covered by your current version range and after updating it in your project the build failed.

As debug is a direct dependency of this project this is very likely breaking your project right now. If other packages depend on you it’s very likely also breaking them.
I recommend you give this issue a very high priority. I’m sure you can resolve this 💪

Your Greenkeeper Bot 🌴

I am fairly new to broccoli and have been trying to write a simple plugin to use with ember.js. I used broccoli-caching-writer in my index.js as described on the github page:

        var CachingWriter = require('broccoli-caching-writer');

       module.exports = CachingWriter.extend({
      init: function(inputTrees, options)
      {   
              console.log('Initializing plugin with tree');
              console.log(inputTrees);
               console.log(options);
              this.inputTrees = inputTrees;

        },
       updateCache: function(srcPaths, destDir) {
          console.log('updateCache called with srcPaths and destDir');
          console.log(srcPaths);
          console.log(destDir);
       }
      });

I then imported the plugin into my ember app (that uses ember CLI) and configured the following in my .brocfile

         var plugin = require('broccoli-my-plugin');
         var merge = require('broccoli-merge-trees');

        pluginTree = new svgSpriter(['images'], {some: 'options'});
        ....
        ....
        module.exports = merge([app.toTree(),pluginTree]);

      Running the above with ember build command gives the following output (paths edited for privacy reasons):

Build failed.
     Cannot read property 'images/.DS_Store' of undefined
    TypeError: Cannot read property 'images/.DS_Store' of undefined
    at CoreObject.proto.shouldBeIgnored (/node_modules/broccoli-svg-sprite/node_modules/broccoli-caching-writer/index.js:135:33)
   at CoreObject.proto.keyForTree (/node_modules/broccoli-svg-sprite/node_modules/broccoli-caching-writer/index.js:277:14)
   at CoreObject.<anonymous> (/node_modules/broccoli-caching-writer/index.js:267:21)
at Array.map (native)
  at CoreObject.proto.keyForTree (/node_modules/broccoli-caching-writer/index.js:266:24)
at /node_modules/broccoli-caching-writer/index.js:87:20
at lib$rsvp$$internal$$tryCatch (/node_modules/broccoli-svg-sprite/node_modules/broccoli-caching-writer/node_modules/rsvp/dist/rsvp.js:489:16)
at lib$rsvp$$internal$$invokeCallback (/node_modules/broccoli-svg-sprite/node_modules/broccoli-caching-writer/node_modules/rsvp/dist/rsvp.js:501:17)
at lib$rsvp$$internal$$publish (/node_modules/broccoli-svg-sprite/node_modules/broccoli-caching-writer/node_modules/rsvp/dist/rsvp.js:472:11)
at lib$rsvp$asap$$flush (/node_modules/broccoli-caching-writer/node_modules/rsvp/dist/rsvp.js:1290:9)

It seems like the plugin is trying to check whether to ignore the path or not, but the options passed to caching writer does not have filterfromcache option defined, so the above should work? Not sure if I'm missing something?

If building a filter plugin, with a 1 to 1 mapping, it would be useful to have a diff of changed files. It seems like the current caching writer has all the info it needs to expose that information to an extending plugins but I'm unclear as to what the most friendly api would look like.

As I see it there are two sublet complexities in exposing this information

• Simply returning a list of added and changed files is easy but deleted files should also be returned
• The plugin still needs to return a complete list of files, including the files not rebuilt, so as not to break broccoli subsequent rebuilds.

Are there any thoughts to implement something like this?

I recently started getting this error whenever I added ember-cli-compass-compiler to my project. It pops up every time I try to save a file and it kills everything. I dont know if the issue originates here, but it after the symlink-copy stack trace.

I have an open thread on the discuss ember board

EEXIST, file already exists '/Users/jdillon/Documents/.../app/tmp/class-tmp_cache_dir-SjogQwVL.tmp'
Error: EEXIST, file already exists '/Users/jdillon/Documents/.../app/tmp/class-tmp_cache_dir-SjogQwVL.tmp'
    at Object.fs.symlinkSync (fs.js:741:18)
    at symlink (/Users/jdillon/Documents/.../app/node_modules/ember-cli/node_modules/symlink-or-copy/index.js:82:14)
    at Function.symlinkOrCopySync [as sync] (/Users/jdillon/Documents/.../app/node_modules/ember-cli/node_modules/symlink-or-copy/index.js:58:5)
    at /Users/jdillon/Documents/.../app/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/node_modules/broccoli-caching-writer/index.js:103:21
    at lib$rsvp$$internal$$tryCatch (/Users/jdillon/Documents/.../app/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:489:16)
    at lib$rsvp$$internal$$invokeCallback (/Users/jdillon/Documents/.../app/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:501:17)
    at lib$rsvp$$internal$$publish (/Users/jdillon/Documents/.../app/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:472:11)
    at Object.lib$rsvp$asap$$flush [as _onImmediate] (/Users/jdillon/Documents/.../app/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:1290:9)
    at processImmediate [as _immediateCallback] (timers.js:345:15)

I'm not sure if this should be reported here or somewhere else.

Readme says inputFiles is an array:

options = options || {};
// options.inputFiles === array of globs, to consider for the cache key

However looking into the code, the default is an empty hash:

this._inputFiles = options.inputFiles || {};

At the least these two don't agree with each other - which one is correct?

In the readme example it uses
// options.inputFiles === array of globs, to consider for the cache key

In the text below it uses
cacheInclude (default: []): An array of regular expressions...

Which one is right? Are they both right? Are they subtly different features?

turns out, we are invalidating dirs, even if excluded content is all that changed.

While working on a plugin that uses broccoli-caching-writer I was following the srcDir and destDir conventions, but I ran into trouble. I discovered that broccoli-caching-writer 1) Copies all option values to the plugin instance (see here) and 2) ultimately causes conflicts with destDir (see here) if an extend plugin attempts to utilize it.

I'm not sure if you plan to have destDir as a promised aspect of the API, but if not, I'd like to propose we change the variable _destDir.

isaacs/rimraf#69

Version 2.6.7 of debug just got published.

This version is covered by your current version range and after updating it in your project the build failed.

debug is a direct dependency of this project this is very likely breaking your project right now. If other packages depend on you it’s very likely also breaking them.
I recommend you give this issue a very high priority. I’m sure you can resolve this 💪

Your Greenkeeper Bot 🌴

Seems odd that the default isn't to assume anything in inputNodes can invalidate cache. If you run with defaults, nothing gets cached at all.

Running ember serve I get

broccoli-caching-writer  cache eviction due to:
- {directory, ., 16822, 0, 0, 1428421480363}
- {undefined} +0ms

This is on a W8.1 machine if that matters.
Any idea of what this might be or how to track it down?

just not part of the cache key.

@rwjblue thoughts?

Version 0.3.2 of walk-sync just got published.

This version is covered by your current version range and after updating it in your project the build failed.

walk-sync is a direct dependency of this project this is very likely breaking your project right now. If other packages depend on you it’s very likely also breaking them.
I recommend you give this issue a very high priority. I’m sure you can resolve this 💪

Your Greenkeeper Bot 🌴

own files included likely by mistake:

• tests: 13
• appveyor.yml 1

dependencies (127):

• broccoli-kitchen-sink-helpers 51
• broccoli-plugin 17
• rimraf 56

-- node both 2x and 3x will need to be updated.

• im working on this right now.
  https://ci.appveyor.com/project/embercli/broccoli-caching-writer/branch/master

If I understood correctly only trees as a whole are cached and invalidated. It would be more efficient to keep track of each file individually, so only actually changed files get updated. The whole tree would still have to be invalidated nevertheless.

But this is probably hard to implement, considering how updateCache(srcDirs, destDir) works right now. We would need a second method like updateFile(srcFile, destFile) that is called individually per file.

Is this out of scope and should be made a seperate plugin?

When installing a newer version of lodash (tested with v3.5.0) for use with Ember (v1.10.0) + broccoli-sass (v0.4.0), running ember s blows up with the following error:

Path or pattern "bower_components/lodash/dist/lodash.min.js" did not match any files
Error: Path or pattern "bower_components/lodash/dist/lodash.min.js" did not match any files
    at Object.multiGlob (/Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/node_modules/broccoli-kitchen-sink-helpers/index.js:202:13)
    at Class.module.exports.CachingWriter.extend.addFiles (/Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/concat-with-maps.js:74:13)
    at Class.module.exports.CachingWriter.extend.updateCache (/Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/concat-with-maps.js:52:12)
    at /Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/broccoli-sourcemap-concat/node_modules/broccoli-caching-writer/index.js:92:34
    at lib$rsvp$$internal$$tryCatch (/Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:489:16)
    at lib$rsvp$$internal$$invokeCallback (/Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:501:17)
    at lib$rsvp$$internal$$publish (/Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:472:11)
    at Object.lib$rsvp$asap$$flush [as _onImmediate] (/Users/someUser/Sites/projectName/node_modules/ember-cli/node_modules/promise-map-series/node_modules/rsvp/dist/rsvp.js:1290:9)
    at processImmediate [as _immediateCallback] (timers.js:354:15)

Version 0.3.3 of walk-sync was just published.

This version is covered by your current version range and after updating it in your project the build failed.

walk-sync is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Your Greenkeeper Bot 🌴

    if (canUseInputFiles(this._inputFiles)) {
      this.debug('using inputFiles directly');
      files = this._inputFiles.filter(shouldNotBeIgnored, this).map(keyForFile, this);
    } else {
      this.debug('walking %o', this.inputFiles);
      files = walkSync.entries(dir,  this.inputFiles).filter(entriesShouldNotBeIgnored, this).map(keyForEntry, this);
    }

Should both of these references to this.inputFiles be this._inputFiles instead?

I'm not sure if you all will support this, but unfortunately, the broccoli-caching-writer does not work inside of a shared folder from VirtualBox. This prohibits the use of ember-cli inside of a Vagrant virtual machine.

The short of it is, Virtualbox does not allow hard links on shared filesystems. It does allow symlinks (with a customization made to it's configuration).

Because the broccoli-caching-writer uses fs.linkSync instead of fs.symlinkSync, it won't work.

Here are a few references to what I'm talking about:

https://coderwall.com/p/b5mu2w
https://www.virtualbox.org/ticket/818
https://www.virtualbox.org/ticket/10085
ember-cli/ember-cli#607

Stack trace that occurs:

[vagrant@localhost ui]$ ember build
version: 0.0.34
Build failed.
EPERM, operation not permitted '/srv/ui/tmp/caching_writer-tmp_dest_dir-STEiDosg.tmp/assets/qunit-notifications.js'
Error: EPERM, operation not permitted '/srv/ui/tmp/caching_writer-tmp_dest_dir-STEiDosg.tmp/assets/qunit-notifications.js'
    at Object.fs.linkSync (fs.js:753:18)
    at linkAndOverwrite (/srv/ui/node_modules/ember-cli/node_modules/broccoli-file-mover/node_modules/broccoli-caching-writer/index.js:104:8)
    at linkRecursivelySync (/srv/ui/node_modules/ember-cli/node_modules/broccoli-file-mover/node_modules/broccoli-caching-writer/index.js:97:5)
    at linkRecursivelySync (/srv/ui/node_modules/ember-cli/node_modules/broccoli-file-mover/node_modules/broccoli-caching-writer/index.js:91:7)
    at linkRecursivelySync (/srv/ui/node_modules/ember-cli/node_modules/broccoli-file-mover/node_modules/broccoli-caching-writer/index.js:91:7)
    at /srv/ui/node_modules/ember-cli/node_modules/broccoli-file-mover/node_modules/broccoli-caching-writer/index.js:43:5
    at tryCatch (/srv/ui/node_modules/ember-cli/node_modules/rsvp/dist/commonjs/rsvp/-internal.js:163:16)
    at invokeCallback (/srv/ui/node_modules/ember-cli/node_modules/rsvp/dist/commonjs/rsvp/-internal.js:172:17)
    at publish (/srv/ui/node_modules/ember-cli/node_modules/rsvp/dist/commonjs/rsvp/-internal.js:150:13)
    at flush (/srv/ui/node_modules/ember-cli/node_modules/rsvp/dist/commonjs/rsvp/asap.js:51:9)
    at process._tickCallback (node.js:419:13)[vagrant@localhost ui]$ 

Works if you change linkSync calls to symlinkSync, but i'm unaware of the implications with doing so.

I preferred naming this issue "novaugust is bad at javascript", but thought that was a given.

If you can't symlink, the testCanLink test can blow up because there's not actually any catching going on.
Whoops.

I have no idea how this ever worked in the first place :(

A demo of the fix

> try{fs.unlinkSync('bullshit.pffft');} finally { console.log('meow') }
meow
Error: ENOENT, no such file or directory 'bullshit.pffft'
    at Object.fs.unlinkSync (fs.js:765:18)
    at repl:1:8
    at REPLServer.self.eval (repl.js:110:21)
    at repl.js:249:20
    at REPLServer.self.eval (repl.js:122:7)
    at Interface.<anonymous> (repl.js:239:12)
    at Interface.EventEmitter.emit (events.js:95:17)
    at Interface._onLine (readline.js:202:10)
    at Interface._line (readline.js:531:8)
    at Interface._ttyWrite (readline.js:760:14)
> try{fs.unlinkSync('bullshit.pffft');}catch(error){console.error(error);} finally { console.log('meow') }
{ [Error: ENOENT, no such file or directory 'bullshit.pffft']
  errno: 34,
  code: 'ENOENT',
  path: 'bullshit.pffft',
  syscall: 'unlink' }
meow
undefined

my original stack trace, from ember-cli

Building...
ENOENT, no such file or directory '/operator/client/node_modules/ember-cli/node_modules/broccoli-caching-writer/canLinkDest.tmp'
Error: ENOENT
 no such file or directory '/operator/client/node_modules/ember-cli/node_modules/broccoli-caching-writer/canLinkDest.tmp'
    at Object.fs.unlinkSync (fs.js:765:18)
    at testCanLink (/operator/client/node_modules/ember-cli/node_modules/broccoli-caching-writer/index.js:183:8)
    at Object.<anonymous> (/operator/client/node_modules/ember-cli/node_modules/broccoli-caching-writer/index.js:11:15)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/operator/client/node_modules/ember-cli/node_modules/broccoli-file-remover/index.js:3:21)

Version 2.6.2 of rimraf just got published.

This version is covered by your current version range and after updating it in your project the build failed.

rimraf is a direct dependency of this project this is very likely breaking your project right now. If other packages depend on you it’s very likely also breaking them.
I recommend you give this issue a very high priority. I’m sure you can resolve this 💪

Your Greenkeeper Bot 🌴

Version 3.6.0 of rsvp just got published.

This version is covered by your current version range and after updating it in your project the build failed.

rsvp is a direct dependency of this project this is very likely breaking your project right now. If other packages depend on you it’s very likely also breaking them.
I recommend you give this issue a very high priority. I’m sure you can resolve this 💪

Your Greenkeeper Bot 🌴

Version 1.3.1 of broccoli-plugin was just published.

This version is covered by your current version range and after updating it in your project the build failed.

broccoli-plugin is a direct dependency of this project, and it is very likely causing it to break. If other packages depend on yours, this update is probably also breaking those in turn.

Your Greenkeeper Bot 🌴

Example: https://github.com/silver-curve/broccoli-archiver

build function creates a zip file from the input tree

Expected zip file is created in both output folder and cache folder
Actual zip file is created in output folder only. Cache folder is empty

Adding the following options makes no difference:

    options.cacheInclude = [/.*/];
    options.inputFiles = ['**/*'];