backintime icon indicating copy to clipboard operation
backintime copied to clipboard

Allow disabling of Host/User/Profile etc nesting

Open auwsom opened this issue 3 years ago • 8 comments

Hi, love Backintime, as it both does incremental backup and has smart retention. One small gripe I have is that it buries files way down in nested directories. Files are 9 directories down just to take a look at them. This is largely because of the Host/User/Profile options in Setting which cannot be left blank for some reason. I understand their purpose, but I am the only user on this single host with only one profile (KISS).

The next set of inception levels are one nested directory for each of the subdirectories a file lives in, even when there is only one base directory needed. Could this also be optional? I dont see a use case for when a user who selects a base dir, needing to have its backup buried in every parent dir to it.

I know this is a minor thing, but I actually went back through evaluating about 20 backup applications I did a year or two ago to land back on backintime, mostly because of my memory of how buried the files are. I'm hoping that somehow helps with the restore process, but this structure could be optional and/or kept in the other backup config files.

Thank you

auwsom avatar Sep 22 '22 02:09 auwsom

Dear @auwsom thank you for your report.

Personally I agree and have the same problem with the folder structure. But it is a minor problem and won't change in the near feature because there are much other things to do.

But I vote for leave it open as a feature request.

Idea: Maybe we can workaround that structure with offering some "smart" symlinks. This would keep the modifications to the code small.

buhtz avatar Sep 22 '22 05:09 buhtz

I've been bothered by the same thing, thanks for the report :)

emtiu avatar Sep 22 '22 08:09 emtiu

I'm giving this a higher priority, because it evidently confuses and annoys many users – see #1001 and https://github.com/bit-team/backintime/issues/1176#issuecomment-1257000016.

emtiu avatar Sep 24 '22 18:09 emtiu

Related to:

  • #20 (introduced Host/User/Profile)
  • #145 (made entries mandatory)
  • #1001

aryoda avatar Oct 05 '22 21:10 aryoda

Somewhere I saw an old issue or comment in the code (or a commit) for a very old version <= 1.1.x which implemented this feature for a reason (the fields were not mandatory before I guess)...

Perhaps we find this old comment again to understand why it was introduced...

aryoda avatar Oct 11 '22 23:10 aryoda

Somewhere I saw an old issue or comment in the code (or a commit) for a very old version <= 1.1.x which implemented this feature for a reason (the fields were not mandatory before I guess)...

Perhaps we find this old comment again to understand why it was introduced...

Looking through git blame for config.py, it was probably introduced in this commit: 3561d0f39487198826e497c8371f0c16429da553

It was released in version 1.0.0, and described like this in the Changelog: https://github.com/bit-team/backintime/blob/b4dd2d1737c5031d264d667c18f6987a1916762b/CHANGES#L539

During development, there was a faulty implementation, which was reported as Launchpad bug #588393, which also contains a description of the way it's supposed to work.

There used to be an option called "Auto Host/User/Profile-ID" to automatically assign these values (instead of manually inputting them, as currently implemented). This was removed with commit https://github.com/bit-team/backintime/commit/7de2f028767b2a5f8be2d2c0c42e69cd4a88f817, released in 1.1.0: https://github.com/bit-team/backintime/blob/b4dd2d1737c5031d264d667c18f6987a1916762b/CHANGES#L298

There also an explanation of this change in the FAQ: Back In Time doesn't find my old Snapshots on my new Computer

emtiu avatar Oct 13 '22 10:10 emtiu

So it looks like Host/User/Profile-ID was still required, but it had an optional disabling of autofill.

There could be an option for "flat" backup. Essentially, user specifies target and destination only.

The restore wizard would just not be used in that scenario, unless a flat restore destination was built in.

auwsom avatar Oct 13 '22 16:10 auwsom