Re: [PATCH] gnu: Add cmake.

[Ludovic Courtès]

Cyril Roelandt <address@hidden> skribis:

> * gnu/packages/cmake.scm: New file.
> * gnu/packages/patches/cmake-fix-tests.patch: New file.
> * Makefile.am: Add them.

Thanks!  Can you add a comment at the top of the patch saying what it
does, why, with a pointer to any relevant mailing list discussion or
documentation?

Ideally all patches should eventually be submitted upstream, but
sometimes it’s not desirable, because the changes would too specific or
intrusive.  State it in the patch if that is the case.

Other than that, looks good to me, so feel free to push then.

It would be great to have a ‘cmake-build-system’, which would be based
on ‘gnu-build-system’ but with specific ‘configure’ and ‘test’ phases.
See guix/build-system/perl.scm and related files for an example.  :-)

Ludo’.