[Bug 698051] Review Request: spim - An assembly language MIPS32 simulator

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Jerry James <loganjerry@xxxxxxxxx> 2011-11-23 17:42:34 EST ---
Excellent, we're almost there.  Rpmlint now says:

spim.x86_64: E: standard-dir-owned-by-package /usr/share/man
spim.x86_64: E: standard-dir-owned-by-package /usr/share/man/man1

That's because "%{_datadir}/*" in %files also catches %{_datadir}/man.  Change
that entry to "%{_datadir}/%{name}".

Also, I'd still like to encourage you to consider two items from comment 4,
namely:
- The spurious executable permissions in the debuginfo package can be fixed
with this snippet in %prep:
find . -type f -perm /0111 -print0 | xargs -0 chmod a-x
- Add some content to %doc, such as spim/README and the various HTML and PDF
files included in the distribution.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]