Takeshi Asano wrote: > Hi all, > > Could you please review two webrevs for localizations (translations): > > (1) Regular localization update > > Webrev: > https://cr.opensolaris.org/action/browse/pkg/asano/7173451 > Bug Reports: > 7173451 IPS UI Localization update as of S11.1 build 15 (nearly build 16) > (May 2012) > 7140339 Online help omf Yelp files need localized > > Three folders are there. Please use convenient one for you. > First one is standard webrev, but majority of colored lines > are of comment lines automatically generated. > Third one is same webrev, but removed emphasis with color > from udiff comment line diffs. > Second one is not a webrev but a set of plain diffs to > focus only on string updates. > All are in UTF-8; please set browser encoding to UTF-8 if > you see garbled characters. > > (2) Localization bugfix in Czech (from Javier Acosta) > > Webrev: > https://cr.opensolaris.org/action/browse/pkg/asano/7122826 > Bug Report: > 7122826 [pkg command] missing space when run pkg install command in czech > locale > > All are in UTF-8; please set browser encoding to UTF-8 if > you see garbled characters.
Both look fine to me. I'd like to see 7122826 moved to solaris/pkg/gui, since that's where we've been keeping the other l10n bugs, though if you generally want to have these bugs in solaris/sysadmin/l10n, that's fine; I just want it consistent. In the future, could you make the second "webrev" in unified diff format, rather than normal diff format? Much easier to read for me. Thanks, Danek _______________________________________________ pkg-discuss mailing list [email protected] http://mail.opensolaris.org/mailman/listinfo/pkg-discuss
