-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Eddy Petrișor wrote: > Frans Pop wrote: >> On Tuesday 09 January 2007 19:07, Eddy Petrişor wrote: >>> Attached is a patch that closes a few open issues with the ppp-udeb. >>> The most important are #402450 (not bringing up the interface after >>> install and not saving a correspondent section in the target >>> /etc/network/interfaces) and the handling of bad login information >>> being typed. >> Patch looks better than previous version. Still some comments. > > Ok. > >> Cheers, >> FJP >> >> I think these earlier comments are still valid: >> ! I still have some reservations regarding this patch. >> ! - netcfg's base-installer script already copies the /etc/network/* >> ! files to the target system; what does the interfaces file in the d-i >> ! environment contain if ppp-udeb is run? > > That file is not touched by ppp-udeb. > >> ! as the netcfg base-installer script will still be run, this is >> ! relevant. > > Should I consider adding the pppoe related information in there? > >> ! - The 50config-target-ppp script is not idempotent: it will >> ! add the same section again and again if it is run more than once. > > Yes, I know, I am aware of this issue and was aware of it from the start. > The main problem is that the interfaces file does not allow comments at the > end of lines with useful > information, so doing something like I did for the /etc/resolv.conf file is > not possible. > > I am not 100% sure, but I think I tested di's sed if it supports deleting > sections like : > > sed '/^# Start of ppp-udeb section$/,/^# End of ppp-udeb section$/d' > > And it didn't.
I could have been on crack that day, this works, actually. I have created an improved version of the patch. >>> Issues still open: >>> - hostname is not set (I think I can force netcfg to run the mandatory >>> steps, but I didn't had the time to do it during the winter holidays) >>> and does not yet blend well with netcfg >> I don't think you should want to use netcfg for that in the current >> situation. ppp-udeb should be more or less self-sufficient. >> IMO this issue is important. >> >> What about the /etc/hosts file? Is that being created? > > I was thinking of setting netcfg's templates to choose "Do not configure the > network at this time" > and make it skip directly to the hostname question. I did some tests this > morning, but they are > inconclusive yet. Still I am confident I can do this. I have tested a few combinations of netcfg templates setting, but no luck yet. Still I don't understand why this didn't help: <eddyp> isn't there any way to make netcfg to choose the "Do not configure the network at this time" option by default? <eddyp> I looked in the annex about preseeding, but I need to do it proramatically <eddyp> programmaticaly, even <eddyp> I tried db_setting "disable_dhcp", "dhcp_failed" and "dhcp_options" I will try to dig further into netcfg and see if there's any way to achieve this without user interaction. The main idea is to let the network unconfigured, but let netcfg to set hosts, lo and hostname. >> Other comments >> ============== >> As you are introducing new templates, you should allow translators the >> opportunity to update their translations! > > Yes, this is a issue I am aware of, but, to be honest, I was put off by your > initial comments > regarding these latest changes we are talking about and I wasn't sure it > would worth the effort > since I felt they might not end up in Etch anyway. I just submitted a BR that should centralize all translations, and I will contact the translators. >> + * ppp-udeb: the interface is now raised in the installed system >> Interfaces are not "raised", but "brought up" or maybe "activated". > > ok, I'll correct that. fixed >> + * Added myself to Uploades with maintainer's consent. >> s/Uploades/Uploaders/ fixed, thanks >> Why not just initialize RET=0, then you can replace the ugly "0$RET" by >> "$RET" for both tests. > > Because I didn't thought of it :-D that way from the start (I built that up > from some different code > logic). Changed. -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (GNU/Linux) iD8DBQFFq5EIY8Chqv3NRNoRAmchAJ43gqx++xW59J5/KY9xuKEAMvkJcACfRiKC m+cbhlgpxUOp0k59lJsnsOc= =8Uvb -----END PGP SIGNATURE-----
ppp_2.4.4rel-5.patch.gz
Description: GNU Zip compressed data
