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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
> > ---
>
> 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

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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux