[libblockdev 0/4] Would like to use scons instead of make

Vratislav Podzimek vpodzime at redhat.com
Thu Jul 10 07:28:23 UTC 2014


On Wed, 2014-07-09 at 12:16 -0400, mulhern wrote:
> The primary motivation is that it may make the configuration part, which
> would otherwise be handle by autotools, easier. Also, it may actually be
> the case that scons is better. I've never written an SConstruct file before
> and the whole thing was surprisingly easy. The fact that the file is actually
> python is nice, and the automatic, and hopefully correct, handling of C
> dependencies is nice too.
> 
> One thing that confused me is scons -c (for clean). It cleans for the
> default target, only. I don't yet know how to clean for all targets,
> without specifying them all, or temporarily removing the default target.
> 
> The first patch is just a tiny fix I happened to notice.
> 
> Removing the Makefile may seem harsh, but we don't want two build systems
> going at once. If we need it back, because this does not work out,
> it's not hard to find.
> 
> mulhern (4):
>   Do not add some redundant const modifiers.
>   Add an SConstruct file largely equivalent to existing Makefile.
>   Update .gitignore for scons related files.
>   Remove Makefile to avoid duplicate updates.
> 
>  .gitignore               |   2 +
>  Makefile                 | 127 -------------------------------------
>  SConstruct               | 162 +++++++++++++++++++++++++++++++++++++++++++++++
>  boilerplate_generator.py |   4 +-
>  4 files changed, 166 insertions(+), 129 deletions(-)
>  delete mode 100644 Makefile
>  create mode 100644 SConstruct
I like this much better than autotools. Just please add README and
INSTALL files that would explain how to build and install the library
and plugins.

Once you're happy with the change, feel free to push it (with the PATCH
1/4 skipped). I'll try not to break things with additional patches in
the meantime.

Thanks for working on this!

-- 
Vratislav Podzimek

Anaconda Rider | RHCE | Red Hat, Inc. | Brno - Czech Republic




More information about the anaconda-patches mailing list