gruntjs / grunt

Grunt: The JavaScript Task Runner
http://gruntjs.com/
Other
12.27k stars 1.5k forks source link

grunt.file.copy not compatible with grunt-contrib-copy in case of copying symlink #1741

Closed nejindal closed 2 years ago

nejindal commented 2 years ago

grunt-contrib-copy appends fileName in destPath but grunt.file.copy works fine if fileName is not appended.

evilaliv3 commented 2 years ago

@vladikoff: §I'm actually encountering this same issue withing @GlobaLeaks

We were using grunt-contrib-copy as well and it seems that the update of grunt introduced a failure on it.

Within our implementation we have one symlink called "license.txt", and when it is copied it tries to copy it in "dest/license.txt/license.txt" duplicating the name of the file and using it as directory path.

vladikoff commented 2 years ago

@evilaliv3 hey could you try grunt 1.5.1 , the point release fix should address it

evilaliv3 commented 2 years ago

Yes! I confirm this @vladikoff

This issue can be closed!

evilaliv3 commented 2 years ago

@vladikoff: I think that actually the patch has still some defects.

Apparently the copy is now copying the link so that the typical build will end linking files that are not included into the build.