Re: [libvirt] [PATCH v2 00/12] include result of bootstrap in git

2020-01-16 Thread Pavel Hrdina 
On Wed, Jan 15, 2020 at 06:47:41PM +, Daniel P. Berrangé wrote:
> On Thu, Jan 09, 2020 at 03:53:10PM +0100, Pavel Hrdina wrote:
> > This patch series is motivated by the effort to adopt Meson as our
> > build system but all of it improves the current build system on its own.
> > 
> > It moves the burden to run bootstrap/autoreconf to developers which
> > means it will save some time for others when there is any change to
> > the build system.
> > 
> > Since one of the patches is slightly bigger here is link to git:
> > 
> >  branch 
> 
> FYI, I am getting really close to being able to post another
> series which entirely kills off gnulib. I'm down to 4 modules
> left today, and a bunch of testing. If all goes well I'll
> likely be done by end of this week.
> 
> Most of these patches are still OK to push, but I'm thinking
> it is nicer to hold off on pushing the really big patch 10
> which actually commits all the gnulib files.

Well nice work.  I was not hoping to get rid of gnulib before the
Meson rewrite, but it will make things easier so I'm perfectly fine to
wait for that.  I'll do review ASAP once the patches are on list.

Pavel


signature.asc
Description: PGP signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v2 00/12] include result of bootstrap in git

2020-01-15 Thread Daniel P . Berrangé 
On Thu, Jan 09, 2020 at 03:53:10PM +0100, Pavel Hrdina wrote:
> This patch series is motivated by the effort to adopt Meson as our
> build system but all of it improves the current build system on its own.
> 
> It moves the burden to run bootstrap/autoreconf to developers which
> means it will save some time for others when there is any change to
> the build system.
> 
> Since one of the patches is slightly bigger here is link to git:
> 
>  branch 

FYI, I am getting really close to being able to post another
series which entirely kills off gnulib. I'm down to 4 modules
left today, and a bunch of testing. If all goes well I'll
likely be done by end of this week.

Most of these patches are still OK to push, but I'm thinking
it is nicer to hold off on pushing the really big patch 10
which actually commits all the gnulib files.


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

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH v2 00/12] include result of bootstrap in git

2020-01-09 Thread Pavel Hrdina 
This patch series is motivated by the effort to adopt Meson as our
build system but all of it improves the current build system on its own.

It moves the burden to run bootstrap/autoreconf to developers which
means it will save some time for others when there is any change to
the build system.

Since one of the patches is slightly bigger here is link to git:

 branch 

Changes in v2:
- some patches removed

Pavel Hrdina (12):
  secret: move virSecretGetSecretString into virsecret
  configure.ac: add check for getegid function
  bootstrap.conf: drop gnulib tests from libvirt
  bootstrap.conf: always copy files
  bootstrap.conf: declare bootstrap sync in configuration file
  bootstrap.conf: disable VC ignore files
  bootstrap.conf: stop creating AUTHORS file
  syntax-check: remove deleted daemon directory from space_indent_check
  autogen.sh: fix autoreconf step
  gnulib: store result of bootstrap in git
  autogen.sh: remove --dry-run option
  autogen.sh: remove --no-git and --gnulib-srcdir options

 .git-module-status | 2 +
 .gitignore |18 -
 INSTALL|   368 +
 Makefile.am| 2 +-
 Makefile.in|  2193 +
 aclocal.m4 |  1717 +
 autogen.sh |82 +-
 bootstrap.conf |27 +-
 build-aux/compile  |   348 +
 build-aux/config.guess |  1667 +
 build-aux/config.sub   |  1793 +
 build-aux/depcomp  |   791 +
 build-aux/install-sh   |   529 +
 build-aux/ltmain.sh| 11149 
 build-aux/missing  |   215 +
 build-aux/syntax-check.mk  |23 +-
 build-aux/test-driver  |   148 +
 config.h.in|  2288 +
 configure  | 55547 +++
 configure.ac   | 2 +-
 docs/Makefile.in   |  2484 +
 docs/compiling.html.in |13 -
 examples/Makefile.in   |  2364 +
 gnulib/lib/Makefile.in |  3705 ++
 gnulib/lib/_Noreturn.h |40 +
 gnulib/lib/accept.c|52 +
 gnulib/lib/alloca.c|   478 +
 gnulib/lib/alloca.in.h |71 +
 gnulib/lib/arg-nonnull.h   |26 +
 gnulib/lib/arpa_inet.in.h  |   150 +
 gnulib/lib/asnprintf.c |34 +
 gnulib/lib/assure.h|37 +
 gnulib/lib/binary-io.c |39 +
 gnulib/lib/binary-io.h |77 +
 gnulib/lib/bind.c  |49 +
 gnulib/lib/c++defs.h   |   316 +
 gnulib/lib/chown.c |   151 +
 gnulib/lib/cloexec.c   |83 +
 gnulib/lib/cloexec.h   |38 +
 gnulib/lib/close.c |71 +
 gnulib/lib/connect.c   |56 +
 gnulib/lib/dup2.c  |   235 +
 gnulib/lib/errno.in.h  |   279 +
 gnulib/lib/fchown-stub.c   |16 +
 gnulib/lib/fcntl.c |   627 +
 gnulib/lib/fcntl.in.h  |   392 +
 gnulib/lib/fd-hook.c   |   116 +
 gnulib/lib/fd-hook.h   |   119 +
 gnulib/lib/filename.h  |54 +
 gnulib/lib/float+.h|   147 +
 gnulib/lib/float.c |33 +
 gnulib/lib/float.in.h  |   188 +
 gnulib/lib/fseek.c |30 +
 gnulib/lib/fseeko.c|   164 +
 gnulib/lib/fstat.c |94 +
 gnulib/lib/gai_strerror.c  |92 +
 gnulib/lib/getaddrinfo.c   |   473 +
 gnulib/lib/getdelim.c  |   147 +
 gnulib/lib/getdtablesize.c |   124 +
 gnulib/lib/getgroups.c |   131 +
 gnulib/lib/gethostname.c   |   104 +
 gnulib/lib/getline.c   |27 +
 gnulib/lib/getpass.c   |   231 +
 gnulib/lib/getpass.h   | 4 +
 gnulib/lib/getpeername.c   |49 +
 gnulib/lib/getsockname.c   |49 +
 gnulib/lib/gettext.h   |   301 +
 gnulib/lib/getugroups.c|   128 +
 gnulib/lib/getugroups.h|19 +
 gnulib/lib/glthread/threadlib.c|73 +
 gnulib/lib/gnulib.mk   |  2629 +
 gnulib/lib/inet_ntop.c |   260 +
 gnulib/lib/intprops.h