diff options
author | Sean Whitton <spwhitton@spwhitton.name> | 2018-05-13 13:15:53 -0700 |
---|---|---|
committer | Sean Whitton <spwhitton@spwhitton.name> | 2018-05-13 13:15:53 -0700 |
commit | 3c850fa31cecd647a92e3e75d7ec9870e8c8b34a (patch) | |
tree | 21a8065bafc73bdba56fe337927383a467488a8e /doc/todo | |
parent | 58884471926804f4abc4d31142e1ba01ee5d9ae3 (diff) | |
parent | 56c77e4cc9ef0899ae0d6c0f1f8b658ccb3ba44f (diff) |
Merge tag '5.3.6' into debian
tagging package propellor version 5.3.6
# gpg: Signature made Wed 09 May 2018 01:25:27 PM MST
# gpg: using RSA key 28A500C35207EAB72F6C0F25DB12DB0FF05F8F38
# gpg: Good signature from "Joey Hess <joeyh@joeyh.name>" [full]
# Primary key fingerprint: E85A 5F63 B31D 24C1 EBF0 D81C C910 D922 2512 E3C7
# Subkey fingerprint: 28A5 00C3 5207 EAB7 2F6C 0F25 DB12 DB0F F05F 8F38
Diffstat (limited to 'doc/todo')
3 files changed, 52 insertions, 0 deletions
diff --git a/doc/todo/depend_on_concurrent-output.mdwn b/doc/todo/depend_on_concurrent-output.mdwn index cf985166..c3641385 100644 --- a/doc/todo/depend_on_concurrent-output.mdwn +++ b/doc/todo/depend_on_concurrent-output.mdwn @@ -3,4 +3,27 @@ should be converted to a dependency. Waiting on concurrent-output reaching Debian stable. +> Well, it's in stable now. Not in oldstable yet, and propellor is still +> supporting the current oldstable, I believe.. --[[Joey]] + [[!tag user/joey]] + +> This was attempted again in 2018 and had to be reverted +> in [[!commit b6ac64737b59e74d4aa2d889690e8fab3772d2c6]]. +> +> The strange output I was seeing is the first line +> of "apt-cache policy apache2" (but not subsequent lines) +> and the ssh-keygen command run by `genSSHFP'` + +> Propellor also misbehaved in some other ways likely due to not seeing +> the command output it expected. In particular Git.cloned must have +> failed to see an origin url in git config output, because it nuked and +> re-cloned a git repo (losing data). +> +> So, it seems that readProcess was somehow leaking output to the console +> and also likely not providing it to the caller. +> +> The affected system had libghc-concurrent-output-dev 1.10.5-1 installed +> from debian. That is a somewhat old version and perhaps it was buggy? +> However, I have not had any luck reproducing the problem there running +> readProcess in ghci. --[[Joey]] diff --git a/doc/todo/factor_out_Grub.configured_for_any___47__etc__47__default_config.mdwn b/doc/todo/factor_out_Grub.configured_for_any___47__etc__47__default_config.mdwn new file mode 100644 index 00000000..16c791cd --- /dev/null +++ b/doc/todo/factor_out_Grub.configured_for_any___47__etc__47__default_config.mdwn @@ -0,0 +1,17 @@ +It would be useful to have a property to set key value pairs in /etc/default configs. The code is in Grub.configured. I have not written a patch yet because I am not sure what the module should be called. Possibilities are: + + & EtcDefault.set "anacron" "ANACRON_RUN_ON_BATTERY_POWER" "no" + +or maybe + + & ConfFile.hasShellSetting "/etc/default/anacron" ("ANACRON_RUN_ON_BATTERY_POWER", "no") + +Or possibly both of these, with the former implemented in terms of the latter. + +Notes: + +* The use of a tuple for the last two parameters ensures that the property can be used infix. + +* I think this property should deduplicate the config key after setting it. I.e. after uncommenting and modifying ANACRON_RUN_ON_BATTERY_POWER it should remove any further ANACRON_RUN_ON_BATTERY_POWER settings further down the config. This allows a seamless transition from just using File.containsLine to add to the end of the file. + +--spwhitton diff --git a/doc/todo/factor_out_Grub.configured_for_any___47__etc__47__default_config/comment_1_5039acea906faba7a0b33094028a475f._comment b/doc/todo/factor_out_Grub.configured_for_any___47__etc__47__default_config/comment_1_5039acea906faba7a0b33094028a475f._comment new file mode 100644 index 00000000..b4b924ac --- /dev/null +++ b/doc/todo/factor_out_Grub.configured_for_any___47__etc__47__default_config/comment_1_5039acea906faba7a0b33094028a475f._comment @@ -0,0 +1,12 @@ +[[!comment format=mdwn + username="joey" + subject="""comment 1""" + date="2018-05-03T16:46:45Z" + content=""" +Agreed on all points, also there are some +`File.containsLine` properties for /etc/default files elsewhere that +don't necessarily work correctly if a later line changes the value, +that could be converted to use this new property. + +Your name ideas sound fine to me. +"""]] |