[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] gnu: Add cmake.

From: Ludovic Courtès
Subject: Re: [PATCH] gnu: Add cmake.
Date: Thu, 14 Mar 2013 18:39:10 +0100
User-agent: Gnus/5.130005 (Ma Gnus v0.5) Emacs/24.2 (gnu/linux)

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

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.  :-)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]