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 <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[email protected]
         AssignedTo|[email protected]    |[email protected]
               Flag|                            |fedora-review?

--- Comment #1 from Daiki Ueno <[email protected]> 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
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to