Re: [PATCH] builddeb: Fix packaged file permissions

From: Frans Pop
Date: Fri Sep 04 2009 - 14:55:50 EST


Added linux-kbuild to CC.

On Friday 04 September 2009, Ben Hutchings wrote:
> The packaged files' permissions are implicitly dependent on the
> current umask. Not only can this result in surprisingly restrictive
> permissions, but dpkg-deb will fail if the DEBIAN directory does not
> have permissions 755 or 775. Therefore, set the umask to 022 before
> building a package.
>
> Signed-off-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx>
Acked-by: Frans Pop <elendil@xxxxxxxxx>

> ---
> scripts/package/builddeb | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/scripts/package/builddeb b/scripts/package/builddeb
> index 8b357b0..24cfd80 100644
> --- a/scripts/package/builddeb
> +++ b/scripts/package/builddeb
> @@ -12,6 +12,8 @@
>
> set -e
>
> +umask 022
> +
> create_package() {
> local pname="$1" pdir="$2"
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/