Subject: Re: [bug#70923] [PATCH v4 07/11] import: hackage: Use 'maybe-list-field' procedure.

From: Lars-Dominik Braun <[email-masked]>
Date: Tue, 10 Dec 2024 18:35:26 +0100

Hi,

> --- a/tests/hackage.scm
> +++ b/tests/hackage.scm
> @@ -67,7 +67,7 @@ (define test-cabal-3
>    if impl(ghc>=7.2&&<7.6)
>      Build-depends: ghc-b
>    if impl(ghc == 7.8)
> -    Build-depends: 
> +    Build-depends:
>        HTTP       >= 4000.2.5 && < 4000.3,
>        mtl        >= 2.0      && < 3
>  ")
> @@ -86,7 +86,7 @@ (define test-cabal-4
>    if impl(ghc>=7.2&&<7.6)
>      Build-depends: ghc-b
>    if impl(ghc == 7.8)
> -    Build-depends: 
> +    Build-depends:
>        HTTP       >= 4000.2.5 && < 4000.3,
>        mtl        >= 2.0      && < 3
>  ")
> @@ -101,7 +101,7 @@ (define test-cabal-5
>  license: BSD3
>  library
>    if impl(ghc == 7.8)
> -    Build-depends: 
> +    Build-depends:
>        HTTP       >= 4000.2.5 && < 4000.3,
>    if impl(ghc -any)
>      Build-depends: mtl        >= 2.0      && < 3
> @@ -126,7 +126,7 @@ (define test-cabal-6
>    if impl(ghc>=7.2&&<7.6)
>      Build-depends: ghc-b
>    if impl(ghc == 7.8)
> -    Build-depends: 
> +    Build-depends:
>        HTTP       >= 4000.2.5 && < 4000.3,
>        mtl        >= 2.0      && < 3
>  ")

why are these changes to the tests required? Having a space after the
colon is legal as far as I know.

Lars