Re: [PATCH v4] allow disabling ACL

2014-04-04 Thread Peter Wu
On Friday 04 April 2014 17:24:04 David Heidelberger wrote: > >> /* Finally, add the read+execute ACL for $USER */ > >> +#ifdef HAVE_ACL > >> if (!add_acl (mount_dir, uid, error)) > >> +#else > >> + if (chown (mount_dir, -1, gid) == -1) > > > > add_acl seems to repo

[PATCH v5] allow disabling ACL

2014-04-04 Thread David Heidelberger
Only for review, patch in attachment. From 7536fa9b490540edb10a4debe24c99e3f9ca4980 Mon Sep 17 00:00:00 2001 From: David Heidelberger Date: Thu, 13 Mar 2014 21:28:42 +0100 Subject: [PATCH v5] allow disabling ACL This patch provide option to build and run udisks without ACL. v2: as replacement o

Re: [PATCH v4] allow disabling ACL

2014-04-04 Thread David Heidelberger
Dne 2014-04-04 16:08, Peter Wu napsal: Hi David, The overall patch looks fine to me, but it got mangled by your mail client. Some minor nitpicks/comment below. mail client, yes :( On Friday 04 April 2014 15:47:56 David Heidelberger wrote: From 310d26a9ad845c0a320692c873562b37c94cd102 Mon S

Re: [PATCH v3] allow disabling ACL

2014-04-04 Thread Peter Wu
Hi David, The overall patch looks fine to me, but it got mangled by your mail client. Some minor nitpicks/comment below. On Friday 04 April 2014 15:47:56 David Heidelberger wrote: > From 310d26a9ad845c0a320692c873562b37c94cd102 Mon Sep 17 00:00:00 2001 > From: David Heidelberger > Date: Thu, 1

[PATCH v3] allow disabling ACL

2014-04-04 Thread David Heidelberger
From 310d26a9ad845c0a320692c873562b37c94cd102 Mon Sep 17 00:00:00 2001 From: David Heidelberger Date: Thu, 13 Mar 2014 21:28:42 +0100 Subject: [PATCH v3] allow disabling ACL This patch provide option to build and run udisks without ACL. v2: as replacement of ACL is used chown call v3: do not cha