Re: [Tails-dev] Please review bugfix/default_search_engines

Delete this message

Reply to this message
Author: intrigeri
Date:  
To: The Tails public development discussion list
Subject: Re: [Tails-dev] Please review bugfix/default_search_engines
Hi,

Ague Mill wrote (26 Sep 2012 10:43:31 GMT) :
> These directories match Iceweasel localization packages. For 'es':
> [...]


Now I understand better -- given it was not clear to me right from the
beginning, perhaps this rationale should be part of the branch
history?

Anyway, I tested this branch, and indeed it works both for locales
that get their own searchplugins directory (e.g. es-MX) and for
locales that don't have an iceweasel l10n package (e.g. es-NI):
both get Spanish Startpage as a default search engine.

This is great :)

> Do you have any idea of how to do it better?


> The only one that comes up to my mind is that we move all search
> plugins somewhere else in the chroot tree and use a local hook to
> add links in directories corresponding to given the installed
> localization package set. It feels unnecessarily complicated.


I agree it is a bit complicated, but I disagree on the "unnecessarily"
part: I think it's the only way to fix this bug in a robust way, that
is, to avoid seeing this bug re-appear, without us noticing, as soon
as the list of iceweasel localization packages changes (e.g. if fr_BE
appears, or if es_MX disappears). We don't want to test every language
setting at release time for such regressions.

>> > f9d73a5 Be consistent when giving a locale to check.torproject.org
>>
>> OK, great. (FTR, the previous setting made sense when our syslinux
>> menu allowed to pick "Portuguese", and that's all -- considering there
>> are many more Portuguese speakers in Brasil than in Portugal.)
>>
>> I have a feeling that this commit is too much or too little, and
>> causes a tiny regression for Brasilian users -- while we're at it, we
>> should add support for pt-BR in our branding extension.


> Yes, that'd be the way to go.


Added to the branch, then (commit 5ba8642).

Cheers!
--
intrigeri
| GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc
| OTR fingerprint @ https://gaffer.ptitcanardnoir.org/intrigeri/otr.asc