bug#23692: 25.1.50; Package.el gets confused if file has no autoloads (could be a bug in update-directory-autoloads)

["Clément Pit--Claudel" <clement.pitclaudel@live.com>]

[-- Attachment #1.1: Type: text/plain, Size: 982 bytes --]

Yup, works for me!

On 2016-06-08 21:47, Stefan Monnier wrote:
>>>> When the a single-package file has no autoloads, the call to
>>>> update-directory-autoloads leaves the generated autoload files
>>>> modified, but unsaved. This causes package.el to prompt the user
>>>> about killing a modified file.
> 
> Does the patch below fix the problem for you?
> 
> 
>         Stefan
> 
> 
> diff --git a/lisp/emacs-lisp/autoload.el b/lisp/emacs-lisp/autoload.el
> index 6473e31..9cf96fd 100644
> --- a/lisp/emacs-lisp/autoload.el
> +++ b/lisp/emacs-lisp/autoload.el
> @@ -1111,7 +1111,8 @@ write its autoloads into the specified file instead."
>  
>        ;; Don't modify the file if its content has not been changed, so `make'
>        ;; dependencies don't trigger unnecessarily.
> -      (when changed
> +      (if (not changed)
> +          (set-buffer-modified-p nil)
>          (let ((version-control 'never))
>            (save-buffer)))
>  
> 
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]