[libvirt-dbus PATCH] meson: Exclude some files from releases

[Daniel P. Berrangé]

On Wed, May 06, 2020 at 12:16:24PM +0200, Andrea Bolognani wrote:
> Both .mailmap and .travis.yml only make sense in the context of a
> git checkout, so we should not include them in releases.

Does it really matter if we include them though ? AFAIK they don't cause
any harm, and it feels like we're just creating extra work for ourselves
by deciding to exclude them.  The same can be said of the .gitlab-ci.yml
file, and supporting things we'll be introducing like the dockerfiles.

One of the things I like about meson is that we can essentially stop
worrying about what goes into the dist tarball, and simply let it have
the entire of git contents (aside from the couple of generated files
we added). So my gut feeling says to not bother trying to exclude files.

> 
> Signed-off-by: Andrea Bolognani <abologna at redhat.com>
> ---
>  meson.build     | 9 +++++++++
>  tools/nodist.sh | 5 +++++
>  2 files changed, 14 insertions(+)
>  create mode 100755 tools/nodist.sh
> 
> diff --git a/meson.build b/meson.build
> index e765ed6..341abe9 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -263,6 +263,15 @@ if git
>      foreach file : [ 'libvirt-dbus.spec', 'AUTHORS.rst' ]
>          meson.add_dist_script('tools/dist.sh', meson.build_root(), file)
>      endforeach
> +
> +    nodist_files = [
> +        '.mailmap',
> +        '.travis.yml',
> +    ]
> +
> +    foreach file : nodist_files
> +        meson.add_dist_script('tools/nodist.sh', file)
> +    endforeach
>  endif
>  
>  
> diff --git a/tools/nodist.sh b/tools/nodist.sh
> new file mode 100755
> index 0000000..f820147
> --- /dev/null
> +++ b/tools/nodist.sh
> @@ -0,0 +1,5 @@
> +#!/bin/sh
> +
> +FILE=$1
> +
> +rm -f $MESON_DIST_ROOT/$FILE
> -- 
> 2.25.4
> 

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