->fixPermissionsWith*() requires sudo

[yann-eugone]

I decided to give a try to this bundle, and appreciate the initiative. But I had some problem using it in my environments. So I decided to open issues for it, if accepted, I could propose a PR.

Regarding the DefaultDeployer::doSetPermissions the only strategy that do not use sudo for fixing permissions is chmod.

Problem here is that in my case, sudo is not required at all (worst : if used, I have an error). Could sudo be optional ?

We could be adding a parameter $sudo = true to the fixPermissionsWith*() methods.

[zorn-v]

I think sudo is not need at all. If you deploy with this bundle you already have privileges (you are is owner), if not - you can fix it. I can deploy with user with no root privileges at all. Using sudo useless and make problems in case of this bundle.

BTW sudo may not be installed at all on some "red hat based" distro

[mcordoba]

That is the point... sudo is required by the fixPermissionsWithAcl methods and @yann-eugone is proposing to avoid this requirement.

More than one year later it looks like sudo is still required

[PWalkow]

Same problem here, my current server requires it to be ran as sudo. I guess it would be good to have a choice to run it with sudo. I've changed temporary my config to: ->fixPermissionsWithChown('someUser')