Re: "make autoloads" fails

[Eli Zaretskii]

> From: Waiyian Chong <address@hidden>
> Date: Sun, 30 Jun 2013 23:30:00 +0800
> 
> Eli Zaretskii <address@hidden> writes:
> 
> > Here's the error message:
> >
> >   $ make autoloads
> >   EMACSLOADPATH=/d/gnu/bzr/emacs/trunk/lisp LC_ALL=C 
> > /d/gnu/bzr/emacs/trunk/src/emacs -batch --no-site-file --no-site-lisp -l 
> > autoload \
> >      --eval "(setq generate-autoload-cookie \";;;###cal-autoload\")" \
> >      --eval "(setq generated-autoload-file (unmsys--file-name 
> > \"/d/gnu/bzr/emacs/trunk/lisp/calendar/cal-loaddefs.el\"))" \
> >      --eval "(setq make-backup-files nil)" \
> >      -f batch-update-autoloads /d/gnu/bzr/emacs/trunk/lisp/calendar
> >   Wrong type argument: number-or-marker-p, nil
> ...
> >   Lisp Backtrace:
> >   "select-safe-coding-system" (0x82e494)
> >   "write-region" (0x82ed9c)
> >   "autoload-ensure-default-file" (0x82ef6c)
> >   "autoload-find-generated-file" (0x82f140)
> >   "update-directory-autoloads" (0x82f3e0)
> >   "apply" (0x82f3dc)
> >   "batch-update-autoloads" (0x82f5f0)
> >   "command-line-1" (0x82f7c8)
> >   "command-line" (0x82f9cc)
> >   "normal-top-level" (0x82fb50)
> 
> I encountered the same problem, and traced it back to the function
> `autoload-ensure-default-file', which seems to call `write-region'
> with wrong arguments.  Build succeeded after applying attached patch.

Thanks.  However, there's no problem with that call to write-region:
it is perfectly valid to pass to it a string.  The root cause was in
select-safe-coding-system, which didn't handle correctly the case of a
string instead of a region of buffer positions.

Should be fixed now on the trunk.