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

[Novimir Pablant]

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