Bug 433443

Builds OK for me; I suggest you ping the xemacs maintainer to get some
assistance with the xemacs bits.

Why do you redefine %debug_package?  This is a noarch package and so doesn't
have a debug package in any case.

I really don't see the point in having a separate doc directory with the same
COPYING file in every subpackage; it really isn't necessary.  So far we're not
blocking on it, but I have to say I really don't understand why you would do that.

Currently all of the xemacs stuff is cruft since it's not being built, but I see
no problem in leaving it there in the hope that it can be made to work. 
However, if it's not ever going to be built then you get to drop the -common
package (and really the package should then be called just "emacs-tuareg").

You might want to consider dropping something into emacs-startdir so that this
package will work without additional configuration.  I think you can just put
append-tuareg.el there, although I don't know quite enough about emacs to say
that with certainty.

The only thing I see that needs changing is the debug_package thing, which is
really minor so I'll go ahead and approve this and you can remove it when you
check in.

* source files match upstream:
   2f22834259a6b0c065a6528a519eda3d0164a9bf2de8a5f7c041d6a91777261b  
   tuareg-mode-1.45.6.tar.bz2
* package meets naming and versioning guidelines.
* package conforms to emacs packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
  emacs-common-tuareg-1.45.6-1.fc9.noarch.rpm
   emacs-common-tuareg = 1.45.6-1.fc9
  =

  emacs-tuareg-1.45.6-1.fc9.noarch.rpm
   emacs-tuareg = 1.45.6-1.fc9
  =
   emacs(bin) >= 22.1.50
   emacs-common-tuareg = 1.45.6-1.fc9

  emacs-tuareg-el-1.45.6-1.fc9.noarch.rpm
   emacs-tuareg-el = 1.45.6-1.fc9
  =
   emacs-tuareg = 1.45.6-1.fc9

* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -doc subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

APPROVED, just remove the debug_package define when you check in.

Thank you for quickly approving this.

debug_package: Force of habit .. I'll remove it.

xemacs: I've asked the upstream maintainer (of tuareg, not xemacs)
if he knows what is going on.  No reply yet.

I'll request CVS, remove debug_package, temporarily restructure
without xemacs as you suggest, and hopefully upstream will get back
to me about xemacs support.

New Package CVS Request
=======================
Package Name: emacs-common-tuareg
Short Description: Emacs mode for editing ocaml
Owners: rjones
Branches: F-8
InitialCC: rjones
Cvsextras Commits: yes

cvs done.