todo updates

3 files changed, 9 insertions, 2 deletions

diff --git a/TODO b/TODO
index 2a69f10..ead4ef5 100644
--- a/TODO
+++ b/TODO
@@ -12,6 +12,7 @@ ALPHA PHASE: get some people to test and suggest ideas, while fixing bugs and re
 * find a way to make _cli_ask_checklist still userfriendly for long lists.. maybe we should just propose/ask to use dia if the list is too long
 * lib things sometimes call die_error but we go back to the menu.. or something.. i think
 * automatically configure grub for dm_crypt and lvm
+* move "/tmp/pacman.conf" to runtime directory and variablize everywhere
 
 
 BETA PHASE: try to get aif on the (beta) installcd as an experimental, alternative installer.
@@ -45,7 +46,7 @@ arguments to the procedure or asking a few questions.
 For automatic procedures, a profile could be the fundamental entitiy,
 whereas for interactive procedures it could provide some other defaults.
 * all dialog windows are equally sized.  noone cares, right?
-* we run mkinitcpio twice: one while installing kernel packages, once after
+* we run mkinitcpio twice: one while installing kernel packages, once after configuring system (mkinitcpio.conf)
 configuring the system (mkinitcpio.conf). can we optimize this?
 * core/interactive -> install packages -> dependency cycles with glibc.  not
 a problem? -> ASKDEV
diff --git a/src/aif.sh b/src/aif.sh
index 1b898d7..9bdea2d 100755
--- a/src/aif.sh
+++ b/src/aif.sh
@@ -271,6 +271,12 @@ module=
 procedure=
 
 
+# TODO: you cannot override $var_OPTS_STRING, nor process_args. because profile not sourced yet
+# we will hit '?)' and exit 5
+# solutions? don't make $var_OPTS_STRING overridable, source correct profile as early as possible so process_args gets known.  remove the '?)' catchall (eg put it in the default process_args) so we don't hit it accidentially
+# in that case -p needs to be the first option, but that's doable imho
+# an alternative would be to provide an argumentstring for the profile. eg aif -p profile -a "-a a -b b -c c"
+
 var_OPTS_STRING=":i:dlp:" # you can override this variable in your procedure.
 while getopts $var_OPTS_STRING OPTION
 do
diff --git a/src/core/libs/lib-software.sh b/src/core/libs/lib-software.sh
index af26137..6b4a1a8 100644
--- a/src/core/libs/lib-software.sh
+++ b/src/core/libs/lib-software.sh
@@ -26,7 +26,7 @@ run_mkinitcpio()
 installpkg() {
 	notify "Package installation will begin now.  You can watch the output in the progress window. Please be patient."
 	target_special_fs on
-	run_background pacman-installpkg "$PACMAN_TARGET -S $TARGET_PACKAGES" $TMP_PACMAN_LOG
+	run_background pacman-installpkg "$PACMAN_TARGET -S $TARGET_PACKAGES" $TMP_PACMAN_LOG #TODO: There may be something wrong here. See http://projects.archlinux.org/?p=installer.git;a=commitdiff;h=f504e9ecfb9ecf1952bd8dcce7efe941e74db946 ASKDEV (Simo)
 	follow_progress " Installing... Please Wait " $TMP_PACMAN_LOG
 
 	wait_for pacman-installpkg