[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[Bug 181404] Review Request: emacs-muse

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: emacs-muse


------- Additional Comments From jonathan underwood gmail com  2006-02-16 10:49 EST -------
Thanks for taking a look at the package, Kevin.

Regarding the package name issue:
fedora-extras package naming guidelines state that 

"If a new package is considered an "addon" package that enhances or adds a new
functionality to an existing Fedora Core or Fedora Extras package without being
useful on its own, its name should reflect this fact.

The new package ("child") should prepend the "parent" package in its name, in
the format: %{parent}-%{child}."

muse is an add-on package for emacs (it doesn't function without emacs), and I
think I have have followed the convention accordingly. This is the same as for
other emacs add-on packages, for example emacs-auctex. It's not the same though
as packages that used to be in core, eg. mew, which IMHO the fact that they
don't follow fedora-extras guidelines is a bug.

Regarding Xemacs:
I could produce xemacs-muse and emacs-muse from the same spec file, however this
brings complexity and makes it a bigger job to maintain. Complexity comes
because I would have to do two makes and make installs, with both makes in the
%install section of the spec (see eg. muse) - this is messy, I think

Also, I would end up with emacs-muse, xemacs-muse, emacs-muse-el, xemacs-muse-el
packages, from a emacs-muse src rpm - I'm not sure this is desireable, a SRPM
producing packages with different names.

I don't wish to be disagreeable, but I think I'm doing the right thing with
regard to naming, or the guidelines should be re-written.

Since eg. emacs-auctex doesn't support xemacs, I don't think a lack of an xemacs
package should be a blocker - I am prepared to add in the xemacs stuff though,
once we've resolved the package naming issue.

Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]