https://bugzilla.rpmfusion.org/show_bug.cgi?id=2741
Alec Leamas <leamas.alec(a)gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |leamas.alec(a)gmail.com
--- Comment #1 from Alec Leamas <leamas.alec(a)gmail.com> 2013-04-03 21:56:46 CEST
---
That the review is stalled in BZ doesn't make this package eligible for
rpmfusion. After reading through 889505 is seems clear to me that this package
can go to main Fedora. If so, it certainly does not belong here in rpmfusion.
Please state why package would be eligible for rpmfusion, or just close this
bug.
That said:
- The Summary is too long, although the descriptions could certainly expand on
it.
- The %defattr are not required (unless you are heading for EPEL).
- One way to avoid rpmlint warnings about % in comments is to write it as %%.
- As other reviewers have pointed out you are not applying %optflags. This is
likely the cause for the missing debuginfo.
- You have not updated the changelog while fixing the spec. The recommended
way is to add a changelog entry for every new version you make public.
But, my best guess is that this should not be discussed here but rather in BZ
889505.
--
Configure bugmail:
https://bugzilla.rpmfusion.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
You are the assignee for the bug.