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=701932 Daiki Ueno <dueno@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |dueno@xxxxxxxxxx AssignedTo|nobody@xxxxxxxxxxxxxxxxx |dueno@xxxxxxxxxx Flag| |fedora-review? --- Comment #1 from Daiki Ueno <dueno@xxxxxxxxxx> 2011-05-09 23:09:05 EDT --- Hi, thanks for packaging this! The Python packaging looks fine with me. One suggestion is that the package should follow the Emacs packaging guideline, since the description says "once started from Emacs". This sounds for me that the package is mainly used with Emacs. https://fedoraproject.org/wiki/Packaging:Emacs To migrate, the package will need to: - be renamed or split from pymacs to emacs-pymacs - byte-compile /usr/share/emacs/site-lisp/pymacs.el - use %{_emacs_site*dir} macros - add a startup file for autoloading -- 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