[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Add backup option to convert-ly (Issue 3572) (issue 14040043)
From: |
Eluze |
Subject: |
Re: Add backup option to convert-ly (Issue 3572) (issue 14040043) |
Date: |
Fri, 4 Oct 2013 12:56:05 -0700 (PDT) |
dak wrote
> Eluze <
> eluzew@
> > writes:
>
>> now reading a little bit more I found that "A path is an unambiguous
>> pointer
>> to a resource in the file system. It can either point to a file or to a
>> directory."
>>
>> this seems exactly what we don't want, we don't deal with directories
>> here -
>> so it's sufficient to test if it is an existing /file/:
>> *os.path.isfile(back_up)* should be used solely.
>
> Wrong. os.path.exists (back_up) should be used solely since we are
> looking for an _unused_ file name. If there is a directory called
> file.ly.~1~ we have to skip this just as much in our search as if there
> is a file called file.ly.~1~ as we don't want to overwrite either
> existing files or existing directories.
ok - so because it's not possible to have a directory and a file with the
same name in the same location we have to check for the path-name...
thanks for your help in clarifying!
Eluze
--
View this message in context:
http://lilypond.1069038.n5.nabble.com/Add-backup-option-to-convert-ly-Issue-3572-issue-14040043-tp151484p151850.html
Sent from the Dev mailing list archive at Nabble.com.
Re: Add backup option to convert-ly (Issue 3572) (issue 14040043), dak, 2013/10/04
Re: Add backup option to convert-ly (Issue 3572) (issue 14040043), dak, 2013/10/04