On Tue, 2023-01-31 at 17:52 +0900, Daiki Ueno wrote: > > Shouldn't this be a || instead of && ? Ie, allow to set either uid, gid > > or both ? Right now if I specify only one, we don't get a chown at all. > > > > Would this (yet untested) patch make sense ? I'll be testing it later > > this week and send a patch here or a github PR, whatever is preferred, > > but first maybe let me know if I'm completely off here ? > > Thank you for the patch! The proposed change looks sensible to me. I > can trigger the CI and give it a formal review once a PR is opened.
Thanks. I'll send a PR. Cheers, Ben.