Re: [O] function for cleaning org-attach directories

[Eric Abrahamsen]

Eric Abrahamsen <address@hidden> writes:

> Alan Schmitt <address@hidden> writes:
>
>> Hi Eric,
>>
>> On 2015-07-16 10:57, Eric Abrahamsen <address@hidden> writes:
>>
>>> I use org-attach a lot, and if you're not careful you can get a "data/"
>>> directory of many gigabytes. Not a problem, until you want to rsync it
>>> and it takes all day...
>>>
>>> I wrote this to clean my attach directories. I'm not sure how portable
>>> it is (and I'm really not pleased with the (concat attach-dir "/" d "/"
>>> d+)), but I'm posting it to see if it's useful to anyone.
>>>
>>> Comments/improvements welcome! If the final product is desirable, I can
>>> work it up as a patch.
>>
>> This would be most useful indeed. One quick question: why do you use
>> "rm" instead of "delete-file"?
>
> Because I was copy-pasting from org-attach! The real question is: why
> didn't I use `delete-directory' :)
>
> I'll do another version!
>
> E

Here we go, and this one ought to be a little more portable. I guess
I'll do it as a proper patch in a bit.

(defun org-attach-clean-dirs (&optional attach-dir clean-archived)
  (interactive)
  (let ((attach-dir
         (if attach-dir
             (file-name-as-directory attach-dir)
           (concat (file-name-as-directory org-directory)
                   org-attach-directory)))
        (valid-dir-re "\\`[0-9a-z-]+\\'")
        (org-id-search-archives (if clean-archived nil org-id-search-archives)))
    (dolist (d (directory-files attach-dir nil valid-dir-re))
      (dolist (d+ (directory-files
                   (concat attach-dir d) nil valid-dir-re))
        (let ((id (format "%s%s" d d+))
              (full-path (concat
                          attach-dir
                          (file-name-as-directory d)
                          d+)))
          (unless (org-id-find id)
            (delete-directory full-path t)))))))