gruntjs / grunt-contrib-uglify

Minify files with UglifyJS.
http://gruntjs.com/
MIT License
1.48k stars 347 forks source link

Files are accidentally combined when ext is defined #341

Open helgatheviking opened 9 years ago

helgatheviking commented 9 years ago

When I try to find use the bit of code from the readme to dynamically uglify all the files in a folder:

// Project configuration.
grunt.initConfig({
  uglify: {
    my_target: {
      files: [{
          expand: true,
          cwd: 'src/js',
          src: '**/*.js',
          ext: '.min.js',
          dest: 'dest/js'
      }]
    }
  }
});

If the filenames have . in them uglify gets confused and concatenates files. For example: I have jquery.fitvids.js and jquery.fastclick.js and after running the uglify from above I end up only with jquery.min.js. However, if I drop the ext setting then I get 3 files as expected.

gothburz commented 9 years ago

I replicated your problem and has the same issue, it only worked correctly when I renamed the files without "jquery.". Just leave the files as .js extensions.

helgatheviking commented 9 years ago

Yes, it works when I rename the files to remove the "jquery". I think it would be ideal for the grunt plugin to get the file name from whatever preceeds .js instead of first character up until the first .

XhmikosR commented 8 years ago

You are welcome to make a PR.