[libvirt] [PATCH 2/2] src: More cleanup of some system headers already contained in internal.h

[Erik Skultety]

On Wed, Sep 19, 2018 at 05:42:18PM +0200, Michal Privoznik wrote:
> On 09/19/2018 12:22 PM, Erik Skultety wrote:
> > All of the ones being removed are pulled in by internal.h. The only
> > exception is sanlock which expects the application to include <stdint.h>
> > before sanlock's headers, because sanlock prototypes use fixed width
> > int, but they don't include stdint.h themselves, so we have to leave
> > that one in place.
> >
> > Signed-off-by: Erik Skultety <eskultet at redhat.com>
> > ---
>
> Is there an automated way to verify this? I don't expect anybody going
> one file after another just to see if this is correct.

That would be madness. In fact what I did after sed'ing this was trying to
compile it until it passed.

>
> I think the fact that this would pass travis/jenkins could be enough.
> But there has to be a better way.

That's in fact what I did, I built the repo on ubuntu-18, centos-7, fedora-28,
rawhide, freebsd-11. I also verified with mingw and Clang.

Erik