[21:52] <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.
[22:14] <infinity> stgraber: How much does busybox grow if you do that?
[22:15] <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:16] <infinity> stgraber: Also, what provides ping, and can we drop it if busybox provides it?
[22:17] <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:19] <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:41] <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:42] <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:43] <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:49] <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:50] <cjwatson> maybe first decide whether you're talking about the initramfs or the installer :)
[23:13] <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:15] <cjwatson> *busybox-udeb, I guess
[23:16] <stgraber> right ;)
[23:16] <cjwatson> debian/config/pkg/udeb:783:CONFIG_PING=y
[23:17] <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:18] <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:19] <cjwatson> in fact it looks like it shares code with ping
[23:20] <cjwatson> so yeah, doit
[23:22] <stgraber> good, uploaded
[23:27] <stgraber> ok and with that, calling it a week, time to pack and change side of the atlantic ocean
[23:36] <cjwatson> temporary or permanent?  (I think you did tell me, I just forgot)