Re: [PATCH] gnu: Add tidy.

[Mark H Weaver]

David Thompson <address@hidden> writes:

> +(define-public tidy
> +  (package
> +    (name "tidy")
> +    (version "20091223")
> +    (source (origin
> +              (method cvs-fetch)
> +              (uri (cvs-reference
> +                    (root-directory
> +                     ":pserver:address@hidden:/cvsroot/tidy")
> +                    (module "tidy")
> +                    (revision "2009-12-23")))
> +              (sha256
> +               (base32
> +                "14dsnmirjcrvwsffqp3as70qr6bbfaig2fv3zvs5g7005jrsbvpb"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     '(#:phases (alist-cons-before
> +                 'configure 'bootstrap

Please do (alist-cons-after 'unpack ...) instead.  The reason is that we
need this to happen before the 'patch-/usr/bin/file' phase and other
similar passes that fix up 'configure', otherwise things are likely to
break on MIPS and ARM.

Looks good to me otherwise.

     Thanks!
       Mark