On Tue, Dec 26, 2023 at 10:36:16PM +0100, Abou Al Montacir wrote:
> Please review this patch, and I'll upload it if you find it OK.

Reviewing M-A:same annotations is hard. Would you mind uploading to
unstable (as the multiarch hinter does not look at experimental) and if
it complains about M-A:same reupload with those M-A:same removed?

> diff --git a/debian/control b/debian/control
> index 118454ff..e8e7c0a8 100644
> --- a/debian/control
> +++ b/debian/control
> @@ -232,6 +232,7 @@ Description: Lazarus Components Library - command line 
> build tools
>  
>  Package: lcl-units-3.0
>  Architecture: any
> +Multi-Arch: same

Verify using m-a hinter.

>  Depends: lcl-gtk2-3.0 (= ${binary:Version}) | lcl-qt5-3.0 (= 
> ${binary:Version}),
>           ${fpc-abi:Depends},
>           ${misc:Depends},
> @@ -263,6 +264,7 @@ Description: Lazarus Components Library - backend 
> independent components
>  
>  Package: lcl-nogui-3.0
>  Architecture: any
> +Multi-Arch: same

Verify using m-a hinter.

>  Depends: fp-units-base,
>           fp-units-fcl,
>           fp-units-rtl,
> @@ -298,6 +300,7 @@ Description: Lazarus Components Library - no GUI backend
>  
>  Package: lcl-gtk2-3.0
>  Architecture: any
> +Multi-Arch: same

Verify using m-a hinter.

>  Depends: fp-units-base,
>           fp-units-fcl,
>           fp-units-gtk2,
> @@ -333,6 +336,7 @@ Description: Lazarus Components Library - GTK+ backend
>  
>  Package: lcl-qt5-3.0
>  Architecture: any
> +Multi-Arch: same

Verify using m-a hinter.

>  Depends: fp-units-base,
>           fp-units-fcl,
>           fp-units-rtl,
> @@ -521,8 +525,8 @@ Description: IDE for Free Pascal - Last Qt version 
> dependency package
>   currently just depends on the GTK+ version.
>  
>  Package: lcl
> -Architecture: all
> -Multi-Arch: foreign
> +Architecture: any
> +Multi-Arch: same

The architecture change looks good. Removing M-A:foreign looks good.
Verify M-A:same using the hinter.

>  Depends: lcl-3.0,
>           ${misc:Depends}
>  Description: Lazarus Components Library - LCL dependency package
> @@ -571,7 +575,8 @@ Description: Lazarus Components Library - command line 
> build tools dependency pa
>   applications.
>  
>  Package: lcl-units
> -Architecture: all
> +Architecture: any

Agreed.

> +Multi-Arch: same

Verify using m-a hinter. Not useful unless lcl-units-3.0 is M-A:same.

>  Depends: lcl-units-3.0,
>           ${misc:Depends}
>  Description: Lazarus Components Library - backend independent components 
> dependency package
> @@ -595,7 +600,8 @@ Description: Lazarus Components Library - backend 
> independent components depende
>   the package containing common components.
>  
>  Package: lcl-nogui
> -Architecture: all
> +Architecture: any

Agreed.

> +Multi-Arch: same

Verify using m-a hinter. Not useful unless lcl-nogui-3.0 is M-A:same.

>  Depends: lcl-nogui-3.0,
>           ${misc:Depends}
>  Description: Lazarus Components Library - no GUI backend dependency package
> @@ -620,7 +626,8 @@ Description: Lazarus Components Library - no GUI backend 
> dependency package
>   applications and command line tools.
>  
>  Package: lcl-gtk2
> -Architecture: all
> +Architecture: any

Agreed.

> +Multi-Arch: same

Verify using m-a hinter. Not useful unless lcl-gtk2-3.0 is M-A:same.

>  Depends: lcl-gtk2-3.0,
>           ${misc:Depends}
>  Description: Lazarus Components Library - GTK+ backend dependency package
> @@ -645,7 +652,8 @@ Description: Lazarus Components Library - GTK+ backend 
> dependency package
>   applications.
>  
>  Package: lcl-qt5
> -Architecture: all
> +Architecture: any

Agreed.

> +Multi-Arch: same

Verify using m-a:hinter. Not useful unless lcl-qt5-3.0 is M-A:same.

>  Depends: lcl-qt5-3.0,
>           ${misc:Depends}
>  Description: Lazarus Components Library - Qt backend dependency package
> diff --git a/debian/control.in b/debian/control.in
> index c7bf9c55..f85a8317 100644
> --- a/debian/control.in
> +++ b/debian/control.in

Skip as these are duplicate changes.

Helmut

Reply via email to