gulp-zip change folder permission, from 755 to 666

[NotCoderJack]

Hi today my team member happened to meet this issue.

Normal:

But after gulp-zip:

and this issue only occurs on Windows, my machine is Mac, never has this problem.

Following is my code:

var gulp = require('gulp');
var gulpZip = require('gulp-zip');
var gulpIf = require('gulp-if');
var gulpChmod = require('gulp-chmod');
var gulpTap = require('gulp-tap');
var moment = require('moment');
var colors = require('colors[表情]fe');
// 打包
function isDirectory(file) {
    console.log(file, file.isDirectory())
    return file.isDirectory()
}
gulp.task('zip', function() {
    var timestamp = moment().format('YYYY-MM-DD_HHMMSS');
    var fileName = 'static-' + timestamp + '.zip';
    console.log(colors.green('*******COPY THIS PATH TO DEPLOY: ' + 'qly_industry/' + fileName))
    return gulp.src('./dist/**')
        // .pipe(gulpIf(isDirectory, gulpChmod(0o777)))
        .pipe(gulpTap(function(file) {
            if (file.isDirectory()) {
                file.stat.mode = parseInt(40755, 8)
            }
        }))
        .pipe(gulpZip(fileName))
        .pipe(gulp.dest('./dist'));
});

After I tried gulp-chmod and gulp-tap as other issues you mentioned to fix likely problem, still can't fix this issue!

---

IssueHunt Summary

jimbly has been rewarded.

Backers (Total: $30.00)

• issuehunt ($30.00)

Submitted pull Requests

• #117 Fix creating unusable zip files on Windows (missing executable bit on directories)

---

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

[ghjagus]

I met the problem too, is there any solutions?

[ghjagus]

I met the problem too, is there any solutions?

Finally, i use archiver replace gulp-zip

[IssueHuntBot]

@issuehunt has funded $30.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[xuliangzhan]

I used gulp-chmod to solve it.

gulp.src('dist/**')
    .pipe(chmod(0o755, 0o755))
    .pipe(zip(`dist.zip`))
    .pipe(gulp.dest('./'))

[Cristy94]

This is a really, really big issue. Any progress on it or suggestions? Should we use chmod as @xuliangzhan suggested?

[issuehunt-oss]

@sindresorhus has rewarded $27.00 to @jimbly. See it on IssueHunt

• 💰 Total deposit: $30.00
• 🎉 Repository reward(0%): $0.00
• 🔧 Service fee(10%): $3.00