[PATCH] tm requires vm-easymenu but it is gone (Was Re: [Failure] The Package Smoketest)

Mats Lidell <[email protected]>
Newsgroups gmane.emacs.xemacs.build-reports,gmane.emacs.xemacs.patches
Organization The XEmacs Project
Message-ID <[email protected]>
>>>>> TheSmokeTest wrote:

TheSmokeTest> These are the results from the XEmacs Package Smoketest:
TheSmokeTest>      21.5-nomule    -- errors
TheSmokeTest>      21.5-mule      -- errors
TheSmokeTest>      21.4-mule      -- errors
TheSmokeTest>      21.4-nomule    -- errors

We have had this situation now for a while. The showstopper is that tm
requires vm-easymenu which is gone from vm. Removing the require fixes
the build (See patch below). A guick look reveals no other
dependencies to vm-easymenu so my guess is that it was only needed for
making vm happy and with vm no longer having this file it is safe to
drop the require.

Index: tm-vm.el
===================================================================
RCS file: /pack/xemacscvs/XEmacs/packages/xemacs-packages/tm/tm-vm.el,v
retrieving revision 1.2
diff -u -r1.2 tm-vm.el
--- tm-vm.el	18 Dec 2002 23:30:33 -0000	1.2
+++ tm-vm.el	21 Apr 2008 18:47:41 -0000
@@ -39,7 +39,6 @@
   (require 'tm-mail)
   (require 'vm)
   (require 'vm-window)
-  (require 'vm-easymenu)
   (require 'vm-macro))
 
 (require 'tm-edit)

Yours
-- 
%% Mats
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.