ACK with comments: [PATCH 5/5] Debian: add Rules-Requires-Root field to control file

Alex Hung alex.hung at canonical.com
Fri Jun 18 20:02:19 UTC 2021


On 2021-06-18 6:57 a.m., Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> Clean up lintian warning, add Homepage

There is no Homepage in the patch (but in PATCH 4/5). Is this a typo?

I can fix this when applying though.

> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  debian/control | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/debian/control b/debian/control
> index 29d3c40f..6f05a93b 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -1,6 +1,7 @@
>  Source: fwts
>  Section: devel
>  Priority: optional
> +Rules-Requires-Root: no
>  Maintainer: Firmware Testing Team <fwts-devel at lists.ubuntu.com>
>  Uploaders: Colin King <colin.king at ubuntu.com>, Alex Hung <alex.hung at canonical.com>, Ivan Hu <ivan.hu at canonical.com>
>  Standards-Version: 3.9.7
> 

Acked-by: Alex Hung <alex.hung at canonical.com>



More information about the fwts-devel mailing list