Takao:

> It seems LC_PAPER is GNU extensions but not POSIX.
> http://www.gnu.org/software/hello/manual/autoconf/Special-Shell-Variables.html
>   
> 
> It may be better to use getenv("LC_PAPER") besides setlocale(LC_PAPER, 
> NULL) until the RFE is fixed.

Are you saying that we should modify evince to check the LC_PAPER 
environment variable, because it doesn't do so now?  And this would
allow people to set the default letter if they want?

Do you have any ideas where & how LC_PAPER should be set for users?

Brian


> Darren Kenny-san wrote (05/27/09 07:52 PM):
>> Hi Brian,
>>
>> On 26/05/2009 16:14, Brian Cameron wrote:
>>> Darren:
>>>
>>> If you needed to modify the patch to get it to backport, then I'd
>>> recommend updating it as a new attachment in the bug report and
>>> encourage the maintainers to put this patch into the next 2.26
>>> release.  I think it is a serious enough usability issue that 2.26
>>> users would appreciate this fix, especially if you already did all
>>> the work to prepare the patch.
>>
>> I'm not sure what you mean - I backported the fix from the GIT changes 
>> - and the
>> modifications were mainly just to get it to apply since other lines in 
>> the files
>> had all changed and as such it wouldn't apply using the patch command.
>>
>> I can't see any benefit of having the community make a change in 2.26 
>> when it's
>> already fixed and working there. If you mean putting it into the next 
>> 2.22
>> release then that's slightly different, and maybe worth mentioning, 
>> but I can't
>> imagine that there is likely to be a new 2.22 release of evince any 
>> time soon,
>> but I've updated the bug anyway :)
>>
>>> We should also file an enhancement request upstream so that the evince
>>> community can consider making the default letter size based on the
>>> user's locale setting, as Jim suggests later in the thread.  That
>>> seems a reasonable enhancement request that someone in the community
>>> might have an interest in working on.
>>
>> That already works upstream - Linux has the LC_PAPER locale variable - 
>> it's
>> Solaris that's lacking in this department, so needs to be fixed on 
>> Solaris...
>>
>> I've logged an RFE to track this request:
>>
>>     6845408 -  Solaris needs a mechanism to determine default print page
>>                    size based on locale.
>>
>> Thanks,
>>
>> Darren.
>> _______________________________________________
>> desktop-discuss mailing list
>> desktop-discuss at opensolaris.org
>>
> 


Reply via email to