From 96415fe18ede53fafb9054992beb0b703a4f0e9c Mon Sep 17 00:00:00 2001 From: Joey Hess Date: Sun, 15 Oct 2017 13:00:29 -0400 Subject: comment --- ...ment_1_77d239c2f73d23aa28a4db6806d1bbdb._comment | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) create mode 100644 doc/todo/Ignore_lost+found_directory/comment_1_77d239c2f73d23aa28a4db6806d1bbdb._comment (limited to 'doc') 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. +"""]] -- cgit v1.2.3