https://bugzilla.redhat.com/show_bug.cgi?id=1304882 --- Comment #4 from awilliam@xxxxxxxxxx <awilliam@xxxxxxxxxx> --- Here's 4.3-7, with all of Neal's comments addressed, and a couple of other changes: - package review improvements: - * no need for worker to Requires(post) os-autoinst - * explain why tests are currently disabled - * fix a few macro invocations to use curly braces - * use directory macros where appropriate in scriptlets - * split apache configuration into a subpackage I forgot to put it in the changelog, but I also made the summaries and descriptions of the packages better, and dropped the superfluous Group tags (we're not really targeting EPEL with this ATM). I'm trying something new for my reviews too - keeping the old specs around and providing diffs for easy comparison. The latest spec will always be symlinked as 'openqa.spec', but you can look at a specific spec as 'openqa.spec.version-release', so: https://www.happyassassin.net/reviews/openqa/openqa.spec (links to https://www.happyassassin.net/reviews/openqa/openqa.spec.4.3-7 ) https://www.happyassassin.net/reviews/openqa/openqa-4.3-7.fc23.src.rpm https://www.happyassassin.net/reviews/openqa/4.3-6_4.3-7.diff (diff from 4.3-6 to 4.3-7) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review