diff options
| author | Joey Hess <joeyh@joeyh.name> | 2017-10-15 13:00:29 -0400 |
|---|---|---|
| committer | Joey Hess <joeyh@joeyh.name> | 2017-10-15 13:00:29 -0400 |
| commit | 96415fe18ede53fafb9054992beb0b703a4f0e9c (patch) | |
| tree | f041c98a635a411aca68a5a2ace1541186d87a8e /doc/todo | |
| parent | 6747b76585407be1f08c2de55b430b9dcfaa54c9 (diff) | |
comment
Diffstat (limited to 'doc/todo')
| -rw-r--r-- | doc/todo/Ignore_lost+found_directory/comment_1_77d239c2f73d23aa28a4db6806d1bbdb._comment | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/doc/todo/Ignore_lost+found_directory/comment_1_77d239c2f73d23aa28a4db6806d1bbdb._comment b/doc/todo/Ignore_lost+found_directory/comment_1_77d239c2f73d23aa28a4db6806d1bbdb._comment new file mode 100644 index 00000000..80efdeec --- /dev/null +++ b/doc/todo/Ignore_lost+found_directory/comment_1_77d239c2f73d23aa28a4db6806d1bbdb._comment @@ -0,0 +1,21 @@ +[[!comment format=mdwn + username="joey" + subject="""comment 1""" + date="2017-10-15T16:52:44Z" + content=""" +I feel that modifying `dirCruft` to include "lost+found" is a layering +violation, because it could change the behavior of a lot of things besides +checking if a directory is empty "enough". Consider code that recurses +through a directory and sets permissions or owner. + +This calls for a new function, or a modification to a more specific +function, to handle the "lost+found" case. +`Propellor.Property.Chroot.Util.unpopulated` seems like a good place to +handle it. + +There are also some backup/restore properties, for Borg and Restic etc, that +only do a restore if a directory is empty or missing, and it seems to make +sense for them to also special-case handling of "lost+found". So it would +make sense to move `unpopulated` to a module that's not chroot-specific +and make those things also use it. +"""]] |