[pkg-fso-maint] [Fwd: patch to install.sh (Debian at FreeRunner)]
luca at pca.it
Wed Feb 4 02:03:16 UTC 2009
On Mon, 02 Feb 2009 11:28:41 +0100, Joachim Breitner wrote:
> I think Steffen’s changes are fine, anyone disagree? And does someone
> want to comment on his remarts about kernel installation?
> Maybe we should remove vfat-support altogether and just have people
> install Qi or adjust their uboot environment?
I will comment after having (re)read the whole thread, which does not
mean I will do it today.
> -------- Weitergeleitete Nachricht --------
> Von: Steffen Moeller <steffen_moeller at gmx.de>
> I just ran through install.sh and did the following changes.
If you provide Git patches (with e.g. `git format-patch origin/master`),
one per each change, it would be less work for us and your name would
directly appear on `git log` output :-)
> Apparently dosfstools got updated in the meantime.
Applied , thank you!
> The description of what the apt target is doing I found too difficult
> (I understood you'd prepare some kind of proxy or so) - maybe you can
> come up with something even better.
It is difficult to find a better wording, because IMHO the actual one is
correct: it not only populates /etc/apt/sources.lists, but also sets
other APT-related stuff (APT::Install-Recommends  and pkg-fso-keyring
for SecureAPT ).
We can reflect what is printed when starting/ending this stage,
i.e. changing the description to "Configure APT", is it better?
> The final bit seems to be syntactically problematic.
> @@ -800,7 +799,7 @@
> # Stage unmount
> echo "Unmounting microSD card partitions"
> -if ! [ "$SINGLE_PART" = "true" ]; then
> +if [ "$SINGLE_PART" != "true" ]; then
> umount $INST_DIR/boot
> umount $INST_DIR
I, as well, prefer your syntax over the existing one and it is also
present at line 605: both corrected . The rationale is that having a
positive test is always preferable that testing for a non-positive one.
Gismo / Luca
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 314 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/pkg-fso-maint/attachments/20090204/8edcca7b/attachment-0001.pgp
More information about the pkg-fso-maint