#432 Support rpkg resultsdir configuration
Merged by onosek. Opened by oturpe.
oturpe/fedpkg use-layout-rpmdefines  into  master

Download 432.patch

Use rpkg layouts for rpmdefines

Earlier, rpkg's layout mechanism was bypassed and rpmdefines were set in a fixed way by fedpkg. This breaks the new results dir feature from https://pagure.io/rpkg/pull-request/540

Rpmdefines are changed to be read from selected layout here, including the new rpmfilename layout property.

Rpkg added support for new configuration option resultsdir in this rpkg commit. Default configuration is set: git_exclude to exclude the new results directory, and resultsdir option set of root, in order to preserve the current behavior by default and make the new subdir option opt-in.

NOTE: This change only works with a version of rpkg that includes the linked commit. The first working version will presumably be 1.63.

NOTE: I do not know what fedpkg-stage is and could not find any explanation either. Assuming it is some alternative version of fedpkg, I just set the new configuration identically for both.

EDIT: Ok, I found this blog post saying:

fedpkg-stage is identical to the regular fedpkg command, except it uses the staging configuration file

Should the staging configuration use the new option, i.e. results_dir=subdir?

rebased onto 26fa8a760bf725336cd732a6035792ad6078ca09

Commit 47579c59 fixes this pull-request

Pull-Request has been merged by onosek

Metadata