https://bugzilla.redhat.com/show_bug.cgi?id=1304882



--- Comment #4 from awill...@redhat.com <awill...@redhat.com> ---
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@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to