commit/leim: Jerry James: Byte compile all leim sources with 21.5. See xemacs-patches message with ID
                
            
            
                11 years, 1 month
            
            
                
                    
                     
                    
                    
                    Bitbucket
                
 
                
                    
                        
                            1 new commit in leim:
https://bitbucket.org/xemacs/leim/commits/c42c0c9d4234/
Changeset:   c42c0c9d4234
User:        Jerry James
Date:        2014-10-03 18:11:14+00:00
Summary:     Byte compile all leim sources with 21.5.  See xemacs-patches message with ID
<CAHCOHQnFmL6nmN3+aXaDxVS0wrQsBrw-+WM5LZakErndegbLNw(a)mail.gmail.com>.
Affected #:  2 files
diff -r 92387d42b154080b6c06f5cce403b26926b8c7bc -r c42c0c9d42345554333cbf7e45f9dffc1a3f718c ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2014-09-29  Jerry James  <james(a)xemacs.org>
+
+	* Makefile: move elisp files in EXTRA_SOURCES to ELCS_1 instead if
+	XEMACS_21_5 is set.
+
 2014-09-08  Norbert Koch  <viteno(a)xemacs.org>
 
 	* Makefile (VERSION): XEmacs package 1.37 released.
diff -r 92387d42b154080b6c06f5cce403b26926b8c7bc -r c42c0c9d42345554333cbf7e45f9dffc1a3f718c Makefile
--- a/Makefile
+++ b/Makefile
@@ -50,11 +50,17 @@
 # under 21.4. They error when loaded in an XEmacs that doesn't support the
 # characters needed.
 
-# #### Move these to ELCS_1 once 21.4 is deprecated.
+include ../../Local.rules
 
+ifeq ($(XEMACS_21_5),t)
+ELCS_1 += quail/cyrillic-tajik.elc quail/georgian.elc quail/hebrew.elc \
+	quail/ipa-21.5.elc quail/latin-ltx.elc quail/rfc1345.elc \
+	quail/sgml-input.elc quail/welsh.elc
+else
 EXTRA_SOURCES= quail/cyrillic-tajik.el quail/georgian.el quail/hebrew.el \
 	quail/ipa-21.5.el quail/latin-ltx.el quail/rfc1345.el \
 	quail/sgml-input.el quail/welsh.el
+endif
 
 ELCS_1_FILES = $(ELCS_1) $(ELCS_1:.elc=.el)
 ELCS_1_DEST = $(PACKAGE)/quail
Repository URL: https://bitbucket.org/xemacs/leim/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches
                        
                     
                    
                 
             
         
        
        
            
        
        
            
                
                    
                    
                    commit/cc-mode: acm: Fix Emacs bug #17463 (cc-langs.elc gets loaded at run-time).
                
            
            
                11 years, 1 month
            
            
                
                    
                     
                    
                    
                    Bitbucket
                
 
                
                    
                        
                            1 new commit in cc-mode:
https://bitbucket.org/xemacs/cc-mode/commits/e5decf42ebf6/
Changeset:   e5decf42ebf6
User:        acm
Date:        2014-10-02 20:17:16+00:00
Summary:     Fix Emacs bug #17463 (cc-langs.elc gets loaded at run-time).
cc-langs.el (c-no-parens-syntax-table): Rename the c-lang-const to
c-make-no-parens-syntax-table and correct the logic.
(c-no-parens-syntax-table): Correct the logic of the c-lang-defvar.
Affected #:  1 file
diff -r 62e1d14d01888223943fef31678bf2e565c453f3 -r e5decf42ebf693b24382cb0e108b61ea4d80076b cc-langs.el
--- a/cc-langs.el
+++ b/cc-langs.el
@@ -389,7 +389,7 @@
   (and (c-lang-const c++-make-template-syntax-table)
        (funcall (c-lang-const c++-make-template-syntax-table))))
 
-(c-lang-defconst c-no-parens-syntax-table
+(c-lang-defconst c-make-no-parens-syntax-table
   ;; A variant of the standard syntax table which is used to find matching
   ;; "<"s and ">"s which have been marked as parens using syntax table
   ;; properties.  The other paren characters (e.g. "{", ")" "]") are given a
@@ -397,18 +397,19 @@
   ;; even when there's unbalanced other parens inside them.
   ;;
   ;; This variable is nil for languages which don't have template stuff.
-  t  `(lambda ()
-	(if (c-lang-const c-recognize-<>-arglists)
-	    (let ((table (funcall ,(c-lang-const c-make-mode-syntax-table))))
-	      (modify-syntax-entry ?\( "." table)
-	      (modify-syntax-entry ?\) "." table)
-	      (modify-syntax-entry ?\[ "." table)
-	      (modify-syntax-entry ?\] "." table)
-	      (modify-syntax-entry ?\{ "." table)
-	      (modify-syntax-entry ?\} "." table)
-	      table))))
+  t  (if (c-lang-const c-recognize-<>-arglists)
+     `(lambda ()
+	(let ((table (funcall ,(c-lang-const c-make-mode-syntax-table))))
+	  (modify-syntax-entry ?\( "." table)
+	  (modify-syntax-entry ?\) "." table)
+	  (modify-syntax-entry ?\[ "." table)
+	  (modify-syntax-entry ?\] "." table)
+	  (modify-syntax-entry ?\{ "." table)
+	  (modify-syntax-entry ?\} "." table)
+	  table))))
 (c-lang-defvar c-no-parens-syntax-table
-	       (funcall (c-lang-const c-no-parens-syntax-table)))
+  (and (c-lang-const c-make-no-parens-syntax-table)
+       (funcall (c-lang-const c-make-no-parens-syntax-table))))
 
 (c-lang-defconst c-identifier-syntax-modifications
   "A list that describes the modifications that should be done to the
Repository URL: https://bitbucket.org/xemacs/cc-mode/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches
                        
                     
                    
                 
             
         
        
        
            
        
        
            
                
                    
                    
                    Fwd: Re: [PATCH 21.5] Fix signs in time zone string.
                
            
            
                11 years, 1 month
            
            
                
                    
                     
                    
                    
                    Adrian Aichner
                
 
                
                    
                        
                            (wasn't in the moderator queue either)
-------- Forwarded Message --------
Subject: 	Re: [PATCH 21.5] Fix signs in time zone string.
Date: 	Tue, 30 Sep 2014 12:38:25 +0200
From: 	Adrian Aichner <adrian.aichner(a)gmail.com>
To: 	xemacs-patches(a)xemacs.org
Hi Marcus, this patch looks wrong to me.
How could XEmacs and GNU Emacs both be wrong until now?
http://git.savannah.gnu.org/cgit/emacs.git/tree/src/editfns.c#n1937
suggests that they are not including a + character for positive offsets,
which surprised me.
But zone offsets > 0 do not include the - sign, which is the same logic
XEmacs uses before your patch.
Also, why are you removing the REST argument in passing without mention?
Adrian
On 30.09.2014 08:13, Marcus Crestani wrote:
> PATCH 21.5
>
> Currently, Fencode_time calculates wrong dates in respect to local
> timezones.  The problem is caused by switched signs in the code that
> builds the time zone string, the attached patch fixes this.
>
> I'll push in two days if nobody objects.
>
>
> diff --git a/src/ChangeLog b/src/ChangeLog
> --- a/src/ChangeLog
> +++ b/src/ChangeLog
> @@ -1,3 +1,9 @@
> +2014-09-30  Marcus Crestani  <crestani(a)informatik.uni-tuebingen.de>
> +
> +	* editfns.c (Fencode_time): Switch signs when calculating the time
> +	zone string. (The offset indicates the value one must add to the
> +	local time to arrive at UTC.)
> +
>   2014-09-23  Jerry James  <james(a)xemacs.org>
>
>   	* floatfns.c (round_two_bignum_1): Fix memory leak.
> diff --git a/src/editfns.c b/src/editfns.c
> --- a/src/editfns.c
> +++ b/src/editfns.c
> @@ -1155,7 +1155,7 @@
>   Year numbers less than 100 are treated just like other year numbers.
>   If you want them to stand for years in this century, you must do that yourself.
>
> -arguments: (SECOND MINUTE HOUR DAY MONTH YEAR &optional ZONE &rest REST)
> +arguments: (SECOND MINUTE HOUR DAY MONTH YEAR &optional ZONE)
>   */
>          (int nargs, Lisp_Object *args))
>   {
> @@ -1192,7 +1192,7 @@
>   	  int abszone = abs (XFIXNUM (zone));
>   	  /* #### I have no idea what this conforms to,
>   	     but the compiler has stopped whining. */
> -	  sprintf (tzbuf, "XXX%s%d:%02d:%02d", (XFIXNUM (zone) < 0) ? "-" : "+",
> +	  sprintf (tzbuf, "XXX%s%d:%02d:%02d", (XFIXNUM (zone) < 0) ? "+" : "-",
>   		   abszone / (60*60), (abszone/60) % 60, abszone % 60);
>   	  tzstring = tzbuf;
>   	}
>
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches
                        
                     
                    
                 
             
         
        
        
            
        
        
            
                
                    
                    
                    commit/XEmacs: Marcus Crestani: Fix calculation of time zone string in Fencode_time.
                
            
            
                11 years, 1 month
            
            
                
                    
                     
                    
                    
                    Bitbucket
                
 
                
                    
                        
                            1 new commit in XEmacs:
https://bitbucket.org/xemacs/xemacs/commits/36dddf9d90d1/
Changeset:   36dddf9d90d1
User:        Marcus Crestani
Date:        2014-10-02 08:19:00+00:00
Summary:     Fix calculation of time zone string in Fencode_time.
Affected #:  2 files
diff -r 7b42a97af782e406d9ebda1890a87479ca1000f5 -r 36dddf9d90d1c2b7fa2e60bb96c9f8bce365e924 src/ChangeLog
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,9 @@
+2014-09-30  Marcus Crestani  <crestani(a)informatik.uni-tuebingen.de>
+
+	* editfns.c (Fencode_time): Switch signs when calculating the time
+	zone string. (The offset indicates the value one must add to the
+	local time to arrive at UTC.)
+
 2014-09-23  Jerry James  <james(a)xemacs.org>
 
 	* floatfns.c (round_two_bignum_1): Fix memory leak.
diff -r 7b42a97af782e406d9ebda1890a87479ca1000f5 -r 36dddf9d90d1c2b7fa2e60bb96c9f8bce365e924 src/editfns.c
--- a/src/editfns.c
+++ b/src/editfns.c
@@ -1190,9 +1190,13 @@
       else if (FIXNUMP (zone))
 	{
 	  int abszone = abs (XFIXNUM (zone));
-	  /* #### I have no idea what this conforms to,
-	     but the compiler has stopped whining. */
-	  sprintf (tzbuf, "XXX%s%d:%02d:%02d", (XFIXNUM (zone) < 0) ? "-" : "+",
+	  /* We specify the time zone in offset notation (see `man
+	     tzset' for details).  The offset indicates the value one
+	     must add to local time to arrive at UTC.  Thus, we sign
+	     the offset with a `-' if the time zone is east of GMT; we
+	     sign the offset with a `+' if the time zone is GMT (then
+	     the offset is 0) or if the time zone is west of GMT. */
+	  sprintf (tzbuf, "XXX%s%d:%02d:%02d", (XFIXNUM (zone) < 0) ? "+" : "-",
 		   abszone / (60*60), (abszone/60) % 60, abszone % 60);
 	  tzstring = tzbuf;
 	}
Repository URL: https://bitbucket.org/xemacs/xemacs/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches
                        
                     
                    
                 
             
         
        
        
            
        
        
            
                
                    
                    
                    Don't bail out on :package-version in `defcustom'
                
            
            
                11 years, 1 month
            
            
                
                    
                     
                    
                    
                    Michael Sperber
                
 
                
                    
                        
                            
I'm (once again ...) porting clojure-mode, and needed this patch.
2014-09-27  Michael Sperber  <mike(a)xemacs.org>
	* custom.el (defcustom): 
	(custom-handle-keyword): Handle :package-version without bailing out.
Will push Monday if nobody objects.
-- 
Regards,
Mike
diff --git a/lisp/custom.el b/lisp/custom.el
--- a/lisp/custom.el
+++ b/lisp/custom.el
@@ -274,6 +274,15 @@
         VALUE should be a string specifying that the variable was
         first introduced, or its default value was changed, in Emacs
         version VERSION.
+:package-version
+        VALUE should be a list with the form (PACKAGE . VERSION)
+        specifying that the variable was first introduced, or its
+        default value was changed, in PACKAGE version VERSION.  This
+        keyword takes priority over :version.  The PACKAGE and VERSION
+        must appear in the alist `customize-package-emacs-version-alist'.
+        Since PACKAGE must be unique and the user might see it in an
+        error message, a good choice is the official name of the
+        package, such as MH-E or Gnus.
 :tag LABEL
         Use LABEL, a string, instead of the item's name, to label the item
         in customization menus and buffers.
@@ -458,6 +467,8 @@
 	 (custom-add-to-group value symbol type))
 	((eq keyword :version)
 	 (custom-add-version symbol value))
+	((eq keyword :package-version)
+	 (custom-add-package-version symbol value))
 	((eq keyword :link)
 	 (custom-add-link symbol value))
 	((eq keyword :load)
@@ -507,6 +518,10 @@
   "To the custom option SYMBOL add the version VERSION."
   (put symbol 'custom-version version))
 
+(defun custom-add-package-version (symbol version)
+  "To the custom option SYMBOL add the package version VERSION."
+  (put symbol 'custom-package-version version))
+
 (defun custom-add-load (symbol load)
   "To the custom option SYMBOL add the dependency LOAD.
 LOAD should be either a library file name, or a feature name."
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches
                        
                     
                    
                 
             
         
        
        
            
        
        
            
                
                    
                    
                    commit/XEmacs: sperber: Don't bail out on :package-version in `defcustom'.
                
            
            
                11 years, 1 month
            
            
                
                    
                     
                    
                    
                    Bitbucket
                
 
                
                    
                        
                            1 new commit in XEmacs:
https://bitbucket.org/xemacs/xemacs/commits/7b42a97af782/
Changeset:   7b42a97af782
User:        sperber
Date:        2014-09-27 12:16:11+00:00
Summary:     Don't bail out on :package-version in `defcustom'.
2014-09-27  Michael Sperber  <mike(a)xemacs.org>
	* custom.el (defcustom):
	(custom-handle-keyword): Handle :package-version without bailing out.
Affected #:  2 files
diff -r b527c25f36ce63931d68e37353ca9fce94f2890d -r 7b42a97af782e406d9ebda1890a87479ca1000f5 lisp/ChangeLog
--- a/lisp/ChangeLog
+++ b/lisp/ChangeLog
@@ -1,3 +1,8 @@
+2014-09-27  Michael Sperber  <mike(a)xemacs.org>
+
+	* custom.el (defcustom): 
+	(custom-handle-keyword): Handle :package-version without bailing out.
+
 2014-09-25  Jerry James  <james(a)xemacs.org>
 
 	* bytecomp.el (byte-compile-insert-header): do not insert
diff -r b527c25f36ce63931d68e37353ca9fce94f2890d -r 7b42a97af782e406d9ebda1890a87479ca1000f5 lisp/custom.el
--- a/lisp/custom.el
+++ b/lisp/custom.el
@@ -274,6 +274,17 @@
         VALUE should be a string specifying that the variable was
         first introduced, or its default value was changed, in Emacs
         version VERSION.
+:package-version
+        VALUE should be a list with the form (PACKAGE . VERSION),
+        where PACKAGE and VERSION are strings.
+
+        This specifies that the variable was first introduced, or its
+        default value was changed, in PACKAGE version VERSION.  This
+        keyword takes priority over :version.
+
+        Since PACKAGE must be unique and the user might see it in an
+        error message, a good choice is the official name of the
+        package, such as MH-E or Gnus.
 :tag LABEL
         Use LABEL, a string, instead of the item's name, to label the item
         in customization menus and buffers.
@@ -458,6 +469,8 @@
 	 (custom-add-to-group value symbol type))
 	((eq keyword :version)
 	 (custom-add-version symbol value))
+	((eq keyword :package-version)
+	 (custom-add-package-version symbol value))
 	((eq keyword :link)
 	 (custom-add-link symbol value))
 	((eq keyword :load)
@@ -507,6 +520,14 @@
   "To the custom option SYMBOL add the version VERSION."
   (put symbol 'custom-version version))
 
+(defun custom-add-package-version (symbol version)
+  "To the custom option SYMBOL add the package version VERSION."
+  (unless (and (consp version)
+	       (stringp (car version))
+	       (stringp (cdr version)))
+        (error "Invalid package version `%s'" value))
+  (put symbol 'custom-package-version version))
+
 (defun custom-add-load (symbol load)
   "To the custom option SYMBOL add the dependency LOAD.
 LOAD should be either a library file name, or a feature name."
Repository URL: https://bitbucket.org/xemacs/xemacs/
--
This is a commit notification from bitbucket.org. You are receiving
this because you have the service enabled, addressing the recipient of
this email.
_______________________________________________
XEmacs-Patches mailing list
XEmacs-Patches(a)xemacs.org
http://lists.xemacs.org/mailman/listinfo/xemacs-patches