Description of problem: plugins/generic.py seems to generate the text "Package consistently uses macro is (..)." ^^^^^^^^ in review.txt, which does not make sense! I suppose it should just say "Package consistently uses *macros* (..)." Version-Release number of selected component (if applicable): fedora-review-0.4.1-2.fc19 How reproducible: 100% Additional info: I see the same in the master and devel branches in git.
Hm... this should be fixed in 3daa8d6683, dated Jan 28, 2013. I certainly see it in older branches, but not in current devel. Could you please provide a more specific test case, including a bug and last commit in devel?
Sure: $ rpm -q fedora-review fedora-review-0.4.1-2.fc19.noarch $ rpm -ql fedora-review | LANG=C xargs grep -d skip "uses macro is" Binary file /usr/share/fedora-review/plugins/generic.pyc matches Binary file /usr/share/fedora-review/plugins/generic.pyo matches $ rpm -V fedora-review $ Perhaps the python files were not recompiled properly?
Same for rawhide
You're right it is fixed in upstream devel branch but still present in the master branch.
As explained in the wiki, the master branch is not really useful in the model we use - it's basically just the release branches collected over time. The important thing is that this is fixed in devel, the upcoming release. Since we agree on this I close this. Thanks for reporting the issue (and for all checking!)
Can we keep this open until it is available in current releases? When do you plan to update the f19 package say?
Fixed in 0.5.0 release.