[Bug] org-attach-set-directory

[Bug] org-attach-set-directory

[Florian Lindner]

Hello,

doing C-a s <newdir> on an entry with an existing attachement produces

make-directory: Die Datei existiert bereits: /home/florian/test/data/<newdir>

(translation: File already exists)

Actually, I implemented this functionality with the help of Nicolas Goaziou, but it seems an org mode or emacs upgrade broke it.

I will try to debug that further, but right now, my time is very limited, only undershot by my elisp knowledge.

Maybe someone can more quickly fix that... ;-)

Best,
Florian

Backtrace:

Debugger entered--Lisp error: (file-already-exists "Die Datei existiert bereits" "/home/florian/test/mydata")
  make-directory-internal("/home/florian/test/mydata")
  make-directory("/home/florian/test/mydata" nil)
  copy-directory("/home/florian/test/data/c1/32d552-b7bb-43b1-b0d8-c57c00d61f9b" "/home/florian/test/mydata" t nil t)
  org-attach-set-directory(nil)
  funcall-interactively(org-attach-set-directory nil)
  call-interactively(org-attach-set-directory)
  org-attach()
  funcall-interactively(org-attach)
  call-interactively(org-attach nil nil)
  command-execute(org-attach)

Org mode version 9.1.13 (9.1.13-elpa @ /home/florian/.emacs.d/elpa/org-20180716/)

Re: [Bug] org-attach-set-directory

[Nicolas Goaziou]

Hello,

Florian Lindner <mailinglists@xgm.de> writes:

> doing C-a s <newdir> on an entry with an existing attachement produces
>
> make-directory: Die Datei existiert bereits: /home/florian/test/data/<newdir>
>
> (translation: File already exists)
>
> Actually, I implemented this functionality with the help of Nicolas Goaziou, but it seems an org mode or emacs upgrade broke it.
>
> I will try to debug that further, but right now, my time is very limited, only undershot by my elisp knowledge.
>
> Maybe someone can more quickly fix that... ;-)
>
> Best,
> Florian
>
> Backtrace:
>
> Debugger entered--Lisp error: (file-already-exists "Die Datei existiert bereits" "/home/florian/test/mydata")
>   make-directory-internal("/home/florian/test/mydata")
>   make-directory("/home/florian/test/mydata" nil)
>   copy-directory("/home/florian/test/data/c1/32d552-b7bb-43b1-b0d8-c57c00d61f9b"
> "/home/florian/test/mydata" t nil t)


It looks like "/home/florian/test/mydata" is not a valid directory name.
It should end with a forward slash.


Regards,

-- 
Nicolas Goaziou