Build succeeded.

• pre-commit : SUCCESS in 1m 16s
• pytest : SUCCESS in 1m 21s
• rpm-mock-build : SUCCESS in 4m 43s

Tested locally and fepdkg local ran without any problem.

Doing a fedpkg-stage build led to: Building python-arrow-0.14.6-1.fc33 for rawhide[...] which will prevent users from doing the build unless they specify --skip-nvr-check once that build has been recorded in koji.

Build succeeded.

• pre-commit : SUCCESS in 1m 48s
• pytest : SUCCESS in 1m 35s
• rpm-mock-build : SUCCESS in 5m 28s

Add a blank line to separate the description from the file list please.

This won't accept the flags of the macro, which would be pasted verbatim into the release field. I know we didn't want to do "fancy" but I couldn't resist :wink:, so let's make it:

%autorel(e:s:hp) %{?-p:0.}1%{?-e:.%{-e*}}%{?-s:.%{-s*}}%{?dist}%{?-h:.1}

This will prepend 0. if for pre-releases, append .1 for hotfixes, and insert the arguments to -e and -s.

Sorry, this is becoming fancy. (Not really sorry.)

• You don't have to shell out to the date program to produce the current date, lua lets you do that via os.date().
• Changelog entries also want full names and email addresses and I fear we can't figure out the latter unless the user has set %packager in their ~/.rpmmacros which we can use verbatim if it exists.
• The release is missing from the entry.
• Calling date() and whoami() doesn't work, they're strings.
• The message can be much shorter. :wink:
• It's indented weirdly (mix of tabs and spaces).

How about:

%autochangelog %{lua:
    date = os.date("%a %b %d %Y")
    packager = rpm.expand("%{?packager}%{!?packager:John Doe <packager@example.com>}")
    evr = rpm.expand("%{?epoch:%{epoch}:}%{version}-%{release}")
    print("* " .. date .. " " .. packager .. " - " .. evr .. "\\n")
    print("- local build")
}

Doing a fedpkg-stage build led to: Building python-arrow-0.14.6-1.fc33 for rawhide[...] which will prevent users from doing the build unless they specify --skip-nvr-check once that build has been recorded in koji.

Yeah, fedpkg should do the equivalent of rpmautospec --quiet process-distgit --check ... and if it yields that %autorel is used, skip the check by itself.

Ok, for some reason, date() and whoami() were not strings for me, maybe some artefact of how IO works in embedded lua?

But I am all for making this more fancy :-)

Build succeeded.

• pre-commit : SUCCESS in 1m 09s
• pytest : SUCCESS in 1m 04s
• rpm-mock-build : SUCCESS in 4m 28s

Build succeeded.

• pre-commit : SUCCESS in 1m 09s
• pytest : SUCCESS in 1m 00s
• rpm-mock-build : SUCCESS in 5m 25s

Build succeeded.

• pre-commit : SUCCESS in 1m 06s
• pytest : SUCCESS in 55s
• rpm-mock-build : SUCCESS in 4m 22s

Build succeeded.

• pre-commit : SUCCESS in 1m 03s
• pytest : SUCCESS in 59s
• rpm-mock-build : SUCCESS in 4m 41s