search

stpettersens / gulp-debian

:tropical_drink: Gulp plug-in to create a Debian package.
https://libraries.io/npm/gulp-debian
Other
7 stars 10 forks source link

Fix 'control directory has bad permissions 777 (must be >=0755 and <=0775)'. #24

Closed hydra closed 1 year ago

hydra commented 6 years ago

When attempting to build DPKG files on a WSL environment I encountered the above error, output from dpkg itself.

First I tried using the 'dirMode' option of outputFile but it didn't fix the issue, creating the control directory first fixes it.

https://www.npmjs.com/package/output-file#optionsdirmode

Note: PR recreated because I accidentally deleted my gulp-debian fork repo on github.

Note, diff looks big but it's just indentation.

with regards to the failing tests, can you take a look? I don't currently have time. I'm a super n00b when it comes to npm, node, etc. likely there's a test which needs to cleanup after itself before the next one runs as I think creating the directory fails if it already exists. If you have any code change suggestions let me know.

FYI, I'm here because a pre-exsting gulp task doesn't work for me on WSL, it uses the 'clean: true' option which is likely why this PR fixes my issues but causes the tests to file.

hydra commented 1 year ago

wow! almost 4 years since I submitted this PR.

stpettersens commented 1 year ago

Sorry, I was away from GitHub most of the time 4 years ago.

On Tue, 27 Dec 2022, 01:16 Dominic Clifton, @.***> wrote:

wow! almost 4 years since I submitted this PR.

— Reply to this email directly, view it on GitHub https://github.com/stpettersens/gulp-debian/pull/24#issuecomment-1365538384, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACWYGOU2MWZNLQFTAH37ODWPI7OTANCNFSM4FKKO6NA . You are receiving this because you modified the open/close state.Message ID: @.***>

hydra commented 1 year ago

@stpettersens that's ok, happy it's merged now. thanks!