Change the behavior of file.recurse so that it will log an error if it p...

lib/grunt/file.js

@@ -211,7 +211,13 @@ file.recurse = function recurse(rootdir, callback, subdir) {
  var abspath = subdir ? path.join(rootdir, subdir) : rootdir;
  fs.readdirSync(abspath).forEach(function(filename) {
  var filepath = path.join(abspath, filename);
- if (fs.statSync(filepath).isDirectory()) {
+ var isDirectory = false;
+ try {
+ isDirectory = fs.statSync(filepath).isDirectory();
+ } catch (e) {
+ grunt.log.error(e);
+ }
+ if (isDirectory) {
  recurse(rootdir, callback, unixifyPath(path.join(subdir || '', filename || '')));
  } else {
  callback(unixifyPath(filepath), rootdir, subdir, filename);

package.json

@@ -1,7 +1,7 @@
 {
  "name": "grunt",
  "description": "The JavaScript Task Runner",
- "version": "0.4.6-0",
+ "version": "0.4.6-1",
  "author": "\"Cowboy\" Ben Alman (http://benalman.com/)",
  "homepage": "http://gruntjs.com/",
  "repository": {
@@ -74,4 +74,4 @@
  "semver": "2.1.0",
  "shelljs": "~0.2.5"
  }
-}
+}

test/fixtures/symlinks/bad-symlink

@@ -0,0 +1 @@
+thing-to-link-to.txt

test/grunt/file_test.js

@@ -723,6 +723,23 @@ exports['file'] = {
 
  test.done();
  },
+ 'recurse bad symlink': function(test) {
+ var rootdir = 'test/fixtures/symlinks';
+
+ var expected = {};
+ expected[rootdir+'/bad-symlink'] = [rootdir, 'bad-symlink'];
+
+ var actual = {};
+ try{
+ grunt.file.recurse(rootdir, function(abspath, rootdir, subdir, filename) {
+ actual[abspath] = [rootdir, filename];
+ });
+ } catch (e) {
+ // We log an error if the file doesn't exist, expected behavior
+ }
+ test.deepEqual(actual, expected, 'file recurse will process broken symlinks');
+ test.done();
+ },
  'recurse': function(test) {
  test.expect(1);
  var rootdir = 'test/fixtures/expand';