bad-package-check

[Ulrich Mueller]

Hi,

Emacs 22.1.50 now displays a warning when loading the semantic.el that
comes with cedet-1.0_pre3. This is in spite of the semantic bug being
fixed in the Gentoo version long time ago. In this special case it is
no problem, since it is easy to upgrade to a newer cedet version.

However, shouldn't the version regexps in general be sandwiched
between \` and \' pairs? Then distros could stick an extra version
string on, and the check wouldn't trigger anymore.

Also, it shouldn't match version strings like "12.0pre3".

Ulrich


--- lisp/simple.el~     2007-11-02 18:31:23.828369000 +0100
+++ lisp/simple.el      2007-11-02 18:34:27.800303335 +0100
@@ -5633,7 +5633,7 @@
 (defconst bad-packages-alist
   ;; Not sure exactly which semantic versions have problems.
   ;; Definitely 2.0pre3, probably all 2.0pre's before this.
-  '((semantic semantic-version "2\\.0pre[1-3]"
+  '((semantic semantic-version "\\`2\\.0pre[1-3]\\'"
               "The version of `semantic' loaded does not work in Emacs 22.
 It can cause constant high CPU load.
 Upgrade to at least Semantic 2.0pre4 (distributed with CEDET 1.0pre4).")