Skip to content

Conversation

@devernay
Copy link
Member

@devernay devernay commented Jan 7, 2022

What type of PR is this? (Check one of the boxes below)

  • New feature (non-breaking change which adds functionality)
  • My change requires a change to the documentation
    • I have updated the documentation accordingly

What does this pull request do?

Save backup versions of projects.
Resolves #562.

Show a few screenshots (if this is a visual change)

image

Have you tested your changes (if applicable)? If so, how?

yes

Copy link
Member

@YakoYakoYokuYoku YakoYakoYokuYoku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May I ask why the choice of the *.ntp.~<n>~ format? My first glance was that it looks kinda weird considering that A the usage of more than one tilde might be a little unintuitive in UNIX based systems and B is that those backups are still valid Natron Project files so ending them with .ntp is not unreasonable. Apart from those no more problems if we go forward with it.

A bit of a nit, however, we have to take into consideration the situation of slowdowns at the moment of autosaving (#738).

@devernay
Copy link
Member Author

devernay commented Jan 7, 2022

May I ask why the choice of the *.ntp.~<n>~ format?

@devernay
Copy link
Member Author

devernay commented Jan 7, 2022

Oh and also GNU utilities cp, mv, ln have the option --backup=numbered which use that same naming convention.
See http://www.jpeek.com/articles/linuxmag/2005-08/

@devernay devernay merged commit 1d1434d into RB-2.4 Jan 7, 2022
@YakoYakoYokuYoku
Copy link
Member

May I ask why the choice of the *.ntp. format?

blender uses .blend1 .blend2, etc. By default, these files should not be opened by blender without removing manually the number at the end.
...

Oh and also GNU utilities cp, mv, ln have the option --backup=numbered which use that same naming convention. See http://www.jpeek.com/articles/linuxmag/2005-08/

Fair enough LGTM then

@YakoYakoYokuYoku YakoYakoYokuYoku deleted the save-versions branch June 18, 2022 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants