Date: Tue, 28 Feb 2006 12:11:11 -0800 From: Seth Kingsley <sethk@magnesium.net> To: John Baldwin <jhb@freebsd.org> Cc: freebsd-qa@freebsd.org Subject: Re: bin/93853: [PATCH]: Add country-specific keymap selection to sysinstall Message-ID: <20060228201111.GM2695@mail.meowfishies.com> In-Reply-To: <200602281419.53871.jhb@freebsd.org> References: <200602261910.k1QJAAmZ089790@freefall.freebsd.org> <200602281419.53871.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--zhtSGe8h3+lMyY1M Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Feb 28, 2006 at 02:19:52PM -0500, John Baldwin wrote: > Hmm, I tried this out. One thing I ran into just running it during multi= user > is that hitting Enter didn't work in the country selection dialog. I had= to > hit 'O' for Ok for some reason. It works if I pick a non-default country > actually, so it's something in the logic for the default country case. H= mm, > I fixed this by fixing dmenuSetCountryVariable() to return DITEM_SUCCESS > rather than void in the DEFAULT_COUNTRY case. That's really odd that it > didn't flag that as a warning. Also, I fixed keymapMenuSelect() to corre= ctly > return a DITEM_* value if VAR_COUNTRY wasn't set (dmenuOpenSimple returns= a > boolean). Finally, I think that this should only popup during installs a= nd > if it is run as non-init it shouldn't pop this menu up (under the assumpt= ion > that in multiuser you've already set the keymap). Yeah, everything you've changed looks good. It occured to me last night as= I was using sysinstall to label a disk that it would be irritating for the country menu to come up post-install. Raising the WARNS to 2 shows that I also missed a restorescr(w) at the end = of configCountry(), and there are some orphaned functions sprinkled about. --=20 Seth Kingsley || sethk@magnesium.net --zhtSGe8h3+lMyY1M Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2 (FreeBSD) iD8DBQFEBK5fD1AymFxBOwgRAjldAJ9XkQ69Hu+O3k3/7jC5RuQqZYuTrACeMpFu XhtECICThPOCMtAyZaLO8dI= =Tcof -----END PGP SIGNATURE----- --zhtSGe8h3+lMyY1M--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20060228201111.GM2695>