bug#3122: Error saving commit messages for bazaar (bzr).

[Novimir Pablant <mir@amicitas.com>]

I installed Emacs-23.0.92.1 and tried reproducing the error.

I still get the error, but now the file is saved anyway.  This is 
certainly an improvement.

Still using emacs23 I tried the two commands:

M-: (file-readable-p 
"c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate") RET

Responds t

M-: (insert-file-contents 
"c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate") RET

Gives the error (from the Backtrace):
Debugger entered--Lisp error: (error "IO error reading 
c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate: 
Permission denied")
 
insert-file-contents("c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate")
   eval((insert-file-contents 
"c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate"))
   eval-expression((insert-file-contents 
"c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate") 
nil)
   call-interactively(eval-expression nil nil)


Novimir


On 4/26/2009 8:02 PM, Stefan Monnier wrote:
>> vc-bzr-registered: IO error reading
>> c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate:
>> Permission denied
>
>> Bazaar version 1.13
>> Emacs version 22.3
>
> Can you try and reproduce this bug with the Emacs-23 pretest?
>
> Also, as far as I can tell, the code in vc-bzr.el (in Emacs-22.3 as
> well as in the upcoming Emacs-23) checks (file-readable-p dirstate)
> before reading the file, so could you check:
>
>    M-: (file-readable-p "c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate") RET
>
> to see if it indeed returns t rather than nil.  And then try
>
>    M-: (insert-file-contents "c:/library/code/python/annotatex/releases/test/.bzr/checkout/dirstate") RET
>
> to check that it indeed signals the error you're seeing.
>
> If indeed, file-readable-p claims the file is readable, but
> insert-file-contents signals a "permission denied", then the problem
> seems to be in the C code (or even deeper such that there's no way for
> Emacs to predict that insert-file-contents will signal an error).
>
>
>          Stefan
>