Thanks! Worked for me 2 : ) From: "Mihai, Irina" <Irina.Mihai@windriver.com> Date: Wednesday, October 24, 2018 at 3:01 PM To: "Bailey, Henry Albert (Al)" <Al.Bailey@windriver.com>, "Martinez Landa, Hayde" <hayde.martinez.landa@intel.com>, "Penney, Don" <Don.Penney@windriver.com>, "starlingx-discuss@lists.starlingx.io" <starlingx-discuss@lists.starlingx.io> Subject: RE: Problems building on master It works for me :) Thanks, Al! Irina ________________________________ From: Bailey, Henry Albert (Al) Sent: Wednesday, October 24, 2018 3:55 PM To: Mihai, Irina; Martinez Landa, Hayde; Penney, Don; starlingx-discuss@lists.starlingx.io Subject: RE: Problems building on master I pushed a fix for review. The build passed in my env. Others can cherry pick and try in theirs while waiting for Zuul. https://review.openstack.org/#/c/613148/ Al From: Mihai, Irina [mailto:Irina.Mihai@windriver.com] Sent: Wednesday, October 24, 2018 2:53 PM To: Martinez Landa, Hayde; Penney, Don; starlingx-discuss@lists.starlingx.io Subject: Re: [Starlingx-discuss] Problems building on master Hey, For what's worth, I also saw that today in my build env. I'm building in a centos lxc container. Irina ________________________________ From: Martinez Landa, Hayde [hayde.martinez.landa@intel.com] Sent: Wednesday, October 24, 2018 2:48 PM To: Penney, Don; starlingx-discuss@lists.starlingx.io Subject: Re: [Starlingx-discuss] Problems building on master Hi Don, %if %{with_python2} CFLAGS="$RPM_OPT_FLAGS" %{__python2} setup.py build %py2_build_wheel Well I tried running it removing the line with the macro “py2_build_wheel” and it worked But whit this line on, it fails with the error message: libvirt-python.spec:131: bad %if condition From: "Penney, Don" <Don.Penney@windriver.com> Date: Wednesday, October 24, 2018 at 1:38 PM To: "Martinez Landa, Hayde" <hayde.martinez.landa@intel.com>, "starlingx-discuss@lists.starlingx.io" <starlingx-discuss@lists.starlingx.io> Subject: RE: Problems building on master I think you meant this update: https://review.openstack.org/#/c/612517/ I don’t understand why you’d be seeing an issue, though. It built fine in my environment, and it shouldn’t impact any %if in the spec From: Martinez Landa, Hayde [mailto:hayde.martinez.landa@intel.com] Sent: Wednesday, October 24, 2018 2:31 PM To: starlingx-discuss@lists.starlingx.io Subject: [Starlingx-discuss] Problems building on master While trying to build on master I encountered the following issue: 17:09:42 b4: patching file SPECS/libvirt-python.spec 17:09:42 b4: ‘/localdisk/designer/haydemtzl/starlingx/cgcs-root/stx/stx-integ/virt/libvirt-python/centos/patches/0001-Fix-setup-import-to-support-building-wheel.patch’ -> ‘/localdisk/loadbuild/haydemtzl/starlingx/std/srpm_assemble/libvirt-python/rpmbuild/SOURCES/./0001-Fix-setup-import-to-support-building-wheel.patch’ 17:09:42 b4: SPEC file: /localdisk/loadbuild/haydemtzl/starlingx/std/srpm_assemble/libvirt-python/rpmbuild/SPECS/libvirt-python.spec 17:09:42 b4: SRPM build directory: /localdisk/loadbuild/haydemtzl/starlingx/std/srpm_assemble/libvirt-python/rpmbuild 17:09:42 b4: TIS_PATCH_VER: 2 17:09:42 b4: error: parse error in expression 17:09:42 b4: error: /localdisk/loadbuild/haydemtzl/starlingx/std/srpm_assemble/libvirt-python/rpmbuild/SPECS/libvirt-python.spec:131: bad %if condition 17:09:42 b4: ERROR: srpm_assemble (1296): rpmbuild failed: rpmbuild -bs /localdisk/loadbuild/haydemtzl/starlingx/std/srpm_assemble/libvirt-python/rpmbuild/SPECS/libvirt-python.spec --define='%_topdir /localdisk/loadbuild/haydemtzl/starlingx/std/srpm_assemble/libvirt-python/rpmbuild' --define='_tis_dist .tis' --undefine=dist 17:09:42 b4: ERROR: build_dir_srpm (802): failed to assemble srpm for 'libvirt-python-4.7.0-1.fc28' 17:09:42 ERROR: reaper (1302): Failed to build src.rpm from source at 'b4' 17:09:42 ######## Wed Oct 24 17:09:42 UTC 2018: build-srpm-parallel --std failed with rc=1 Wed Oct 24 17:09:42 UTC 2018: build-srpm-parallel --std failed with rc=1 Whit the help of Erich for debugging this, We think it come from this change [0] that affects the specific line mentioned on the error above: [0] https://review.openstack.org/#/c/612519/ I was able to reproduce this in 3 diff environments, including our Jenkins automation setup.