ACK: [[SCRIPT=remove_re|Re: [SRU][FOCAL][hwe-5.13[PATCH] UBUNTU: [Packaging] Re-enable dbgsyms]]

[Tim Gardner]

Acked-by: Tim Gardner <tim.gardner at canonical.com>

On 9/20/21 2:16 PM, Dimitri John Ledkov wrote:
> BugLink: https://bugs.launchpad.net/bugs/1939287
> 
> During v5.13 cycle, debug symbols building got fixed by running
> install step during build, instead of binary. Allowing to build
> multiple flavours with debug symbols without breaching disk space on
> Launchpad builder.
> 
> The packaging change that disabled dbgsyms unconditionally however
> remained in the hwe-5.13 packaging, but can now be safely reverted to
> reenable debug symbols builds.
> 
> Fixes: a175d9b3c0 ("UBUNTU: [Packaging] Initial packaging import from linux-hwe-5.13")
> Link: https://bugs.launchpad.net/bugs/1930713
> Signed-off-by: Dimitri John Ledkov <dimitri.ledkov at canonical.com>
> ---
>   debian/rules.d/0-common-vars.mk | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/debian/rules.d/0-common-vars.mk b/debian/rules.d/0-common-vars.mk
> index ef056f8426..3f045aa4d4 100644
> --- a/debian/rules.d/0-common-vars.mk
> +++ b/debian/rules.d/0-common-vars.mk
> @@ -77,7 +77,9 @@ endif
>   # The debug packages are ginormous, so you probably want to skip
>   # building them (as a developer).
>   #
> +ifeq ($(full_build),false)
>   skipdbg=true
> +endif
>   
>   abinum		:= $(shell echo $(revision) | sed -r -e 's/([^\+~]*)\.[^\.]+(~.*)?(\+.*)?$$/\1/')$(abi_suffix)
>   prev_abinum	:= $(shell echo $(prev_revision) | sed -r -e 's/([^\+~]*)\.[^\.]+(~.*)?(\+.*)?$$/\1/')$(abi_suffix)
> 

-- 
-----------
Tim Gardner
Canonical, Inc