[Patch] Fixes and features about live-initramfs
- Subject: [Patch] Fixes and features about live-initramfs
- From: steven at nchc.org.tw (Steven Shiau)
- Date: Tue, 12 Aug 2008 17:48:49 +0800
- Message-id: <[🔎] 48A15C81.10202@nchc.org.tw>
- In-reply-to: <mailman.2089.1218209207.14321.debian-live-devel@lists.alioth.debian.org>
- References: <mailman.2089.1218209207.14321.debian-live-devel@lists.alioth.debian.org>
> Steven Shiau wrote:
>> > Hi,
>
> Hi,
>
> a general note: please make our lives easier by sending individual patch
> files for each change. this way, merging/applying them is much easier
> (most easiest ist though to directly merge from a git tree of yours).
Sorry, my bad. Will do that if I submit another patch in the future.
>
>> > I modified some in live-initramfs to make it work with
>> > fetch=tftp://$IP/@FILE
>
> nice. this also requires tftp client to be installed in the initramfs,
> though (which i've added in the commit after the merge).
>
> also, your patch was slightly imperfect wrt/ to threatening the [
> "${quiet}" != "y" ] stanzas, and you should use '' for sed when not
> having variables inside the substitution, not "":
>
> bad: FOO="$(echo ${BAR} | sed "s|foo|bar|")"
> good: FOO="$(echo ${BAR} | sed 's|foo|bar|')"
>
> i changed both of these in the commit.
Thanks for telling me this. Will follow your advice.
>
>> > And about fixes:
>> > 1. Make the shared libraries in /root/lib and /root/usr/lib can be
>> > accessed in initramfs
>
> what is the reason/use-case for that? why do we need that?
The reason to do so is actually for the PATH in scripts/live, since
there is:
export
PATH="/root/usr/bin:/root/usr/sbin:/root/bin:/root/sbin:/usr/bin:/usr/sbin:
/bin:/sbin"
If some exe file in /root/usr/{bin,sbin} is run, and it's not static
linking, we need those *.so in /root/lib or /root/usr/lib. I found this
problem when I used live-helper and live-initramfs to create a Ubuntu
8.10 live CD.
>
>> > 2. In upstart, to run it is using "exec", not "respawn".
>> > For example, in /etc/event.d/tty1, it's:
>> > exec /sbin/getty 38400 tty1
>
> still, this won't work in debian. have you tried it? it gives a message
> about not having found exec.
I did try it, but I tried it in Ubuntu 7.10 and 8.04. With this patch,
it works for Ubuntu. I think we should find a better solution for this
to work with Debian.
>
>> > Another minor fix is about using udevsettle or "udevadm settle" in
>> > scripts/live-helpers.
>
> merged.
--
Steven Shiau <steven _at_ nchc org tw> <steven _at_ stevenshiau org>
National Center for High-performance Computing, Taiwan.
http://www.nchc.org.tw
Public Key Server PGP Key ID: 1024D/9762755A
Fingerprint: A2A1 08B7 C22C 3D06 34DB F4BC 08B3 E3D7 9762 755A
Reply to: