https://bugzilla.redhat.com/show_bug.cgi?id=2292943
--- Comment #6 from Tom Rix trix@redhat.com --- (In reply to Tom Hughes from comment #4)
Thanks for the review.
I'll update the version but I'm not clear why you think adding %dir would be helpful? The directory is already added along with the files it contained and the only thing %dir does it to make rpm package the directory without any files it contains meaning they would then have to be listed separately, or is that what you're suggesting?
the fedora-review output had some (maybe spurious) warnings about dir ownership. i have used the %dir squash these warnings in my own spec files.