public inbox for speakup@linux-speakup.org
 help / color / mirror / Atom feed
* speakup_setlocale concerns
@  William Hubbs
   ` Steve Holmes
  0 siblings, 1 reply; 2+ messages in thread
From: William Hubbs @  UTC (permalink / raw)
  To: speakup mailing list

All,

I just looked this over and I have a couple of concerns about it.

The first is that the way it uses speakupconf save might be considered
broken, because, it doesn't allow me to specify which save directory I
want to use.  My suggestion for a solution to that would be to remove
the calls to speakupconf save from speakupconf_setlocale and just warn
the user that they need to use speakupconf save after running this
script.

Also, why not  make this part of speakupconf by adding another command
to it, like "speakupconf locale", which would check the LANG variable
and load the correct settings?  I think if we do that we can remove
speakup_setlocale entirely.

What do the rest of you think?

Thanks,

William


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: speakup_setlocale concerns
   speakup_setlocale concerns William Hubbs
@  ` Steve Holmes
  0 siblings, 0 replies; 2+ messages in thread
From: Steve Holmes @  UTC (permalink / raw)
  To: speakup

Hey, if speakupconf can handle the additional feature without bloating
it too much, and if this results in eliminating another module or
script, then it's a good idea.  I think given a bit of explanation of
how the locale deal works, I could probably work it into speakupconf
unless someone else would rather do it.  I've never done anything with
locale stuff as I'm not an i18n sort of guy.

On Sun, Oct 17, 2010 at 05:44:12PM -0500, William Hubbs wrote:
> All,
> 
> I just looked this over and I have a couple of concerns about it.
> 
> The first is that the way it uses speakupconf save might be considered
> broken, because, it doesn't allow me to specify which save directory I
> want to use.  My suggestion for a solution to that would be to remove
> the calls to speakupconf save from speakupconf_setlocale and just warn
> the user that they need to use speakupconf save after running this
> script.
> 
> Also, why not  make this part of speakupconf by adding another command
> to it, like "speakupconf locale", which would check the LANG variable
> and load the correct settings?  I think if we do that we can remove
> speakup_setlocale entirely.
> 
> What do the rest of you think?
> 
> Thanks,
> 
> William
> 
> _______________________________________________
> Speakup mailing list
> Speakup@braille.uwo.ca
> http://speech.braille.uwo.ca/mailman/listinfo/speakup

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~ UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
 speakup_setlocale concerns William Hubbs
 ` Steve Holmes

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).