Looks sane to me, although PKGDB2_URL and KOJI_REPO seem unused.
They are used, but elsewhere in the code and since I set them with the global, I didn't have to change anything :)
Ah, sure. Still looks good to me :).