FedoraReview

#359 mock.py fails to parse the rpmfusion config file
Closed: fixed by sergiomb. Opened by sergiomb.

Closed: fixed
Reopen Issue

after partially revert revert my commit [1] we can use it with mock configuration rpmfusion because these configuration use include state , for example [2] .

[1]
https://pagure.io/FedoraReview/pull-request/307#request_diff

[2]
fedora-review --other-bz https://bugzilla.rpmfusion.org -b 5163 -m fedora-rawhide-x86_64-rpmfusion_free

We should probably parse the file for include() and continue recursively until we find config_opts['root'] or config_opts["root"].

Added PR #363 to revert to previous behaviour and add a workaround for uninititialized config_opts.

Looks good , but I haven't tested it . Seems the best of 2 worlds , on mock project we haven't find any solution for include statement .... yet

this is fixed .
Thanks

Metadata Update from @sergiomb:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)
Metadata
None
None
None
None
None
Related Pull Requests
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.