summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--doc/todo/Ignore_lost+found_directory.mdwn10
-rw-r--r--doc/todo/Ignore_lost+found_directory/comment_1_77d239c2f73d23aa28a4db6806d1bbdb._comment21
-rw-r--r--doc/todo/creates_mount_point_and_always_tries_to_mount_when_using_Fstab.mounted.mdwn10
3 files changed, 41 insertions, 0 deletions
diff --git a/doc/todo/Ignore_lost+found_directory.mdwn b/doc/todo/Ignore_lost+found_directory.mdwn
new file mode 100644
index 00000000..9b1469b3
--- /dev/null
+++ b/doc/todo/Ignore_lost+found_directory.mdwn
@@ -0,0 +1,10 @@
+Hello,
+
+I had a problem where propellor refused to create a chroot because the target
+directory was not empty. Actually, the directory contained `lost+found`.
+
+I made a patch to ignore this directory, do you see any bad side effect?
+
+You can pull the `ignore-lost-n-found` branch at `http://git.ni.fr.eu.org/nicolas/propellor.git`.
+
+Nicolas.
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.
+"""]]
diff --git a/doc/todo/creates_mount_point_and_always_tries_to_mount_when_using_Fstab.mounted.mdwn b/doc/todo/creates_mount_point_and_always_tries_to_mount_when_using_Fstab.mounted.mdwn
new file mode 100644
index 00000000..4732825f
--- /dev/null
+++ b/doc/todo/creates_mount_point_and_always_tries_to_mount_when_using_Fstab.mounted.mdwn
@@ -0,0 +1,10 @@
+Hello,
+
+this simple change creates mount point and always tries to mount when using
+Fstab.mounted.
+
+See [[forum/Fstab.mounted_does_not_mount_if_entry_exists]].
+
+Please pull `fstab-mounted` branch at `http://git.ni.fr.eu.org/nicolas/propellor.git`.
+
+Nicolas.