On Wed, May 13, 2020 at 11:45:26PM +0200, Miro Hrončok wrote: > On 10. 04. 20 9:03, Pierre-Yves Chibon wrote: > > On Fri, Apr 10, 2020 at 01:58:20AM +0200, Miro Hrončok wrote: > > > On 09. 04. 20 15:43, Pierre-Yves Chibon wrote: > > > > To remove some of the warnings thrown by `fedpkg` or to simply keep > > > > `rpmbuild` > > > > working locally, you will have to install the `rpmautospec-rpm-macros` > > > > package > > > > available in your nearest bodhi (it's still hot from the oven at the > > > > time of > > > > writing this email so it hasn't made it yet to updates-testing). > > > > > > There is also this warning: > > > > > > $ fedpkg-stage build > > > Could not execute build: Package 3dprinter-udev-rules-0.2.2-1.fc33 has > > > already been built > > > Note: You can skip this check with --skip-nvr-check. See help for more > > > info. > > > > > > Because locally, it always thinks I'm at release 1.fc33, but this has been > > > already built. With --skip-nvr-check, it works, obviously. > > > > > > I suppose this could be fixed (workarounded?) in fedpkg (it could default > > > to > > > --skip-nvr-check if it finds the %autorel macro in the spec). > > > > Yes we're aware of this one and I thought we documented it but apparently we > > only covered the part about the missing macros. > > > > I'll add this to the doc, thanks! > > I have found a slight issue with this approach. > > 1. Packager A clones package P (has %autorel) > 2. Packager B pushes+builds some changes in package P > 3. Packager A runs `fedpkg build --skip-nvr-check` without pulling first > > At 3, the old version of the package gets rebuilt and gets a higher EVR. > > See the build/3dprinter-udev-rules-0-0.2.2-7.fc33 and ...-6.fc33 tags in: > > https://src.stg.fedoraproject.org/rpms/3dprinter-udev-rules/commits/master
It took me a little time to find it back, but I knew we had documented this as this is something we are aware off: https://docs.pagure.org/fedora-infra.rpmautospec/peculiarities.html#multiple-builds-from-a-given-commit I wonder if we could teach fedpkg to check if there are new commits in a branch before it runs a fedpkg build and issue an error message if that is the case (with the --I-know-what-I-am-doing argument of course). Pierre _______________________________________________ devel mailing list -- devel@lists.fedoraproject.org To unsubscribe send an email to devel-le...@lists.fedoraproject.org Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org
