summaryrefslogtreecommitdiff
path: root/doc/todo/support_tarball_source_images
diff options
context:
space:
mode:
Diffstat (limited to 'doc/todo/support_tarball_source_images')
-rw-r--r--doc/todo/support_tarball_source_images/comment_1_6c019767a6a678d7d9f7ad924e948d94._comment17
-rw-r--r--doc/todo/support_tarball_source_images/comment_2_2d620f837f825f3041d9c66612e2ab4c._comment8
-rw-r--r--doc/todo/support_tarball_source_images/comment_3_411e4884c47fa6c371e6c6d2c5472752._comment8
-rw-r--r--doc/todo/support_tarball_source_images/comment_4_df41bdafff9277fb105ea2da0b0af5d9._comment37
-rw-r--r--doc/todo/support_tarball_source_images/comment_5_fec2c4bf3d0ea806c94b0720b5e80aea._comment8
-rw-r--r--doc/todo/support_tarball_source_images/comment_6_7f06f7f03d943649d24b8c5708bbb952._comment7
6 files changed, 85 insertions, 0 deletions
diff --git a/doc/todo/support_tarball_source_images/comment_1_6c019767a6a678d7d9f7ad924e948d94._comment b/doc/todo/support_tarball_source_images/comment_1_6c019767a6a678d7d9f7ad924e948d94._comment
new file mode 100644
index 00000000..192c1a69
--- /dev/null
+++ b/doc/todo/support_tarball_source_images/comment_1_6c019767a6a678d7d9f7ad924e948d94._comment
@@ -0,0 +1,17 @@
+[[!comment format=mdwn
+ username="joey"
+ subject="""comment 1"""
+ date="2015-10-20T16:06:52Z"
+ content="""
+I've introduced a ChrootBootstrapper type class, of which Debootstrapped
+is just one member. Everything that bootstraps chroots uses this
+type class, so you can write a separate module that uses buildroot, or
+whatever. And you're welcome to contribute such a module to propellor.
+
+As far as having propellor generate a tarball of a chroot, the way to do
+that is something like:
+
+ chroot dir `onChange` mktarball dir
+
+Ie, compose the chroot creation property with one that tars it up.
+"""]]
diff --git a/doc/todo/support_tarball_source_images/comment_2_2d620f837f825f3041d9c66612e2ab4c._comment b/doc/todo/support_tarball_source_images/comment_2_2d620f837f825f3041d9c66612e2ab4c._comment
new file mode 100644
index 00000000..9da5739e
--- /dev/null
+++ b/doc/todo/support_tarball_source_images/comment_2_2d620f837f825f3041d9c66612e2ab4c._comment
@@ -0,0 +1,8 @@
+[[!comment format=mdwn
+ username="joey"
+ subject="""comment 2"""
+ date="2015-10-20T16:27:07Z"
+ content="""
+I've done all I plan to do about this particular todo, so look forward to
+modules being contributed for other ways to bootstrap chroots.
+"""]]
diff --git a/doc/todo/support_tarball_source_images/comment_3_411e4884c47fa6c371e6c6d2c5472752._comment b/doc/todo/support_tarball_source_images/comment_3_411e4884c47fa6c371e6c6d2c5472752._comment
new file mode 100644
index 00000000..65027028
--- /dev/null
+++ b/doc/todo/support_tarball_source_images/comment_3_411e4884c47fa6c371e6c6d2c5472752._comment
@@ -0,0 +1,8 @@
+[[!comment format=mdwn
+ username="https://mathstuf.id.fedoraproject.org/"
+ nickname="mathstuf"
+ subject="comment 3"
+ date="2015-10-21T03:57:19Z"
+ content="""
+Here's my attempt at adding support for this: <https://github.com/mathstuf/propellor/tree/add-tarball-system>. `MinimalTarball` may be too verbose. Maybe `TarballImage`?
+"""]]
diff --git a/doc/todo/support_tarball_source_images/comment_4_df41bdafff9277fb105ea2da0b0af5d9._comment b/doc/todo/support_tarball_source_images/comment_4_df41bdafff9277fb105ea2da0b0af5d9._comment
new file mode 100644
index 00000000..21c39d32
--- /dev/null
+++ b/doc/todo/support_tarball_source_images/comment_4_df41bdafff9277fb105ea2da0b0af5d9._comment
@@ -0,0 +1,37 @@
+[[!comment format=mdwn
+ username="joey"
+ subject="""comment 4"""
+ date="2015-10-21T21:07:53Z"
+ content="""
+I don't think it makes sense to add MinimalTarball as an operating System.
+
+If you make ChrootTarball contain the FilePath to the tarball,
+then ChrootBootstrapper ChrootTarball instance doesn't need to look
+at the system parameter at all; instead it can just look inside the
+ChrootTarball to get the path to it, and extract that.
+
+So, the user would write, say:
+
+ bootstrapped (System (Debian Unstable) "amd64") (ChrootTarball "/tmp/debian.tar.gz") "/chroot"
+
+What might make sense it to change the first parameter of boostrapped
+to Maybe System, for cases where you don't want to specify it.
+
+----
+
+It would probably also be good to document what kind of tarball is
+expected. Ie, that it can be compressed, and should not consist
+of a single subdirectory, but one subdirectory for each top-level
+root directory.
+
+----
+
+Also, as a minor point, I don't think you need to pass standardPathEnv;
+it should be fine to use boolSystem with the regular enviroment as
+far as I can see. In fact, you could probably simplify extractTarball
+by implementing it like this:
+
+ check (unpopulated target) $
+ cmdProperty "tar" params
+ `requires` File.dirExists target
+"""]]
diff --git a/doc/todo/support_tarball_source_images/comment_5_fec2c4bf3d0ea806c94b0720b5e80aea._comment b/doc/todo/support_tarball_source_images/comment_5_fec2c4bf3d0ea806c94b0720b5e80aea._comment
new file mode 100644
index 00000000..4b999d73
--- /dev/null
+++ b/doc/todo/support_tarball_source_images/comment_5_fec2c4bf3d0ea806c94b0720b5e80aea._comment
@@ -0,0 +1,8 @@
+[[!comment format=mdwn
+ username="https://mathstuf.id.fedoraproject.org/"
+ nickname="mathstuf"
+ subject="comment 5"
+ date="2015-10-23T03:13:55Z"
+ content="""
+Updated. Thanks for the review. Much cleaner now :) .
+"""]]
diff --git a/doc/todo/support_tarball_source_images/comment_6_7f06f7f03d943649d24b8c5708bbb952._comment b/doc/todo/support_tarball_source_images/comment_6_7f06f7f03d943649d24b8c5708bbb952._comment
new file mode 100644
index 00000000..0de08eb7
--- /dev/null
+++ b/doc/todo/support_tarball_source_images/comment_6_7f06f7f03d943649d24b8c5708bbb952._comment
@@ -0,0 +1,7 @@
+[[!comment format=mdwn
+ username="joey"
+ subject="""comment 6"""
+ date="2015-10-23T05:31:07Z"
+ content="""
+Came out very nice indeed. Merged, thanks!
+"""]]