[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 5/5] meson: add NSIS building
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v2 5/5] meson: add NSIS building |
Date: |
Wed, 26 Aug 2020 09:40:47 +0100 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Tue, Aug 25, 2020 at 09:58:27PM +0400, marcandre.lureau@redhat.com wrote:
> From: Marc-André Lureau <marcandre.lureau@redhat.com>
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> Makefile | 56 -----------------------------------
> meson.build | 25 ++++++++++++++++
> scripts/nsis.py | 78 +++++++++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 103 insertions(+), 56 deletions(-)
> create mode 100644 scripts/nsis.py
> diff --git a/scripts/nsis.py b/scripts/nsis.py
> new file mode 100644
> index 0000000000..e1c409344e
> --- /dev/null
> +++ b/scripts/nsis.py
> @@ -0,0 +1,78 @@
> +#!/usr/bin/env python3
> +#
> +# Copyright (C) 2020 Red Hat, Inc.
> +#
> +# SPDX-License-Identifier: GPL-2.0-or-later
> +
> +import argparse
> +import glob
> +import os
> +import shutil
> +import subprocess
> +import tempfile
> +
> +
> +def signcode(path):
> + cmd = os.environ.get("SIGNCODE")
> + if not cmd:
> + return
> + subprocess.run([cmd, path])
I know the existing makefile used $SIGNCODE env variable, but I can't
help thinking it would be better to have it as a configure arg, and
in turn a meson arg.
None the less, it isn't worse than what already exists so
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|