On Thu, May 07, 2020 at 12:04:10PM +0200, Andrea Bolognani wrote: > On Tue, 2020-05-05 at 12:40 +0100, Daniel P. Berrangé wrote: > > +.git_build_job_template: &git_build_job_definition > > + image: $CI_REGISTRY_IMAGE/ci-$NAME:latest > > + stage: builds > > + before_script: > > + - export MAKEFLAGS="-j$(getconf _NPROCESSORS_ONLN)" > > + - export SCRATCH_DIR="/tmp/scratch" > > + - export VROOT="$SCRATCH_DIR/vroot" > > + - export LD_LIBRARY_PATH="$VROOT/lib" > > + - export PATH="$VROOT/bin:$PATH" > > + - export PKG_CONFIG_PATH="$VROOT/lib/pkgconfig" > > + - export TEST_MAINTAINER=1 > > + script: > > + - pushd "$PWD" > > + - mkdir -p "$SCRATCH_DIR" > > + - cd "$SCRATCH_DIR" > > + - git clone --depth 1 https://gitlab.com/libvirt/libvirt.git > > + - mkdir libvirt/build > > + - cd libvirt/build > > + - ../autogen.sh --prefix="$VROOT" --without-libvirtd > > + - $MAKE install > > + - popd > > + - perl Build.PL > > + - perl Build > > + - perl Build test > > The Jenkins-based CI also runs > > perl Build install > > and I think we should keep that. I'll add that. > > Partially related point: I just realized that, among the projects > we've added to GitLab CI so far, I don't think there's a single one > where we're testing that we can successfully build RPMs. This is > something that CentOS CI does, and I think it's important that we > keep testing it on GitLab CI. The way we tested RPM builds was a bit gross in CentOS as we were hacking the spec to cut out the BuildRequires. I'd like to figure out a better way todo that which actually reflects the way we'll run builds for real. When testing against distro-provided libvirtd this is quite easy as we have the right devel packages present. Testing against GIT it is harder. > > > +centos-7-git-build: > > + <<: *git_build_job_definition > > + variables: > > + NAME: centos-7 > > + > > +centos-8-git-build: > > + <<: *git_build_job_definition > > + variables: > > + NAME: centos-8 > > + > > +debian-9-git-build: > > + <<: *git_build_job_definition > > + variables: > > + NAME: debian-9 > > Same question as libvirt-php: why are we not building against > distro-provided libvirt packages? Perl bindings only support exactly matching libvirt versions. 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 :|