https://bugzilla.redhat.com/show_bug.cgi?id=1305737 Jens Lody <fedora@xxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody@xxxxxxxxxxxxxxxxx |fedora@xxxxxxxxxxx Flags| |fedora-review? --- Comment #3 from Jens Lody <fedora@xxxxxxxxxxx> --- I take this. Still some minor issues: COPYING has to use %license-macro instead of %doc. The docs-folder should not be included completely, because it doubles the man-pages, includes the Makefile{.in,.am} and the bash- and zsh-completion-files. The latter should be packaged also (if possible), at least for the users convenience, but of course at the correct places. You should also add a comment, why you do not use the tests. And you should add a comment, why the main url is from Debian, but the package source from Ubuntu. The package does not install with fedora-review in rawhide (error 30), but if I remember correctly,this is not a packge but a fedora-review/mock-issue. It seems to work fine in el7. By the way, do you also plan to package pbuilder ? I use it to build Debian-packages of Code::Blocks on my CentOS 7 Server. I would use Reprepro to build the repo (if it is packkaged),currently I use a self-written script, which works quite good for my purposes. I also use pbuilder (quick hacked package there). -- 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review