| === CarlFK1 is now known as CarlFK | ||
| stgraber | cjwatson: if you're still around. Do you expect anything bad to happen if I turn on CONFIG_PING and CONFIG_PING6 in busybox-initramfs? I'm mostly interested in having ping6 really as ping is currently provided by something else in our installer though I guess it'd make sense to turn both on if I do that. | 21:52 |
|---|---|---|
| infinity | stgraber: How much does busybox grow if you do that? | 22:14 |
| stgraber | infinity: that's the next thing I need to check, I wouldn't expect a huge growth for just ping+ping6 but that's definitely on the list | 22:15 |
| infinity | stgraber: Also, what provides ping, and can we drop it if busybox provides it? | 22:16 |
| infinity | stgraber: Mostly, though, I suspect this is less of a concern for the installer specifically, and more of a concern for initrds growing in general. | 22:17 |
| infinity | (A potential concern for the installer if busybox ping doesn't provide the same interface as the other one being used, and we still need two) | 22:17 |
| infinity | stgraber: Third option, can whatever's providing ping also provide ping6, and we're just not doing so in the current udeb? | 22:17 |
| infinity | (From a "not related to the intaller" POV, mind you, I suspect people with NFS roots and the like might find ping/ping6 in busybox-initramfs handy as a diagnostic tool) | 22:19 |
| stgraber | infinity: yeah, I still need to figure out exactly what provides ping at the moment, its output is quite different "blah is alive!" and it returns instantly so I'm guessing having a "real" ping may break things in the installer | 22:41 |
| stgraber | having ping/ping6 would definitely be useful in the initrd, also for LTSP as debugging things with busybox-initramfs is kind of tricky (IIRC the most useful thing in there is netcat) | 22:42 |
| stgraber | anyway, I guess I'll just do a local build with the extra options to check the size and will then build a custom installer initrd to see if the installer explodes because of it | 22:43 |
| cjwatson | stgraber: sounds generally OK I guess; the installer doesn't care what's in busybox-initramfs, busybox-udeb is a separate build | 22:49 |
| cjwatson | and a separate config | 22:49 |
| cjwatson | maybe first decide whether you're talking about the initramfs or the installer :) | 22:50 |
| stgraber | ah, I assumed the installer was using the initramfs build, looks like I was wrong. I was mostly interested in the installer, so these changes would be for initramfs-udeb then :) | 23:13 |
| cjwatson | *busybox-udeb, I guess | 23:15 |
| stgraber | right ;) | 23:16 |
| cjwatson | debian/config/pkg/udeb:783:CONFIG_PING=y | 23:16 |
| cjwatson | looks like busybox already provides that then; turning on PING6 too seems like a no-brainer, go ahead | 23:17 |
| cjwatson | see 1:1.1.3-5ubuntu8 | 23:17 |
| stgraber | yeah, guess it helps looking at the right config file ;) turning PING6 on then and uploading that. I assume you're not too concerned about the size difference for the udeb? | 23:18 |
| cjwatson | not desperately | 23:18 |
| cjwatson | I mean, I care a little bit, but ping6 shouldn't be huge | 23:18 |
| cjwatson | in fact it looks like it shares code with ping | 23:19 |
| cjwatson | so yeah, doit | 23:20 |
| stgraber | good, uploaded | 23:22 |
| stgraber | ok and with that, calling it a week, time to pack and change side of the atlantic ocean | 23:27 |
| cjwatson | temporary or permanent? (I think you did tell me, I just forgot) | 23:36 |
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!