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.