Comment 13 for bug 1171434

Revision history for this message
su_v (suv-lp) wrote :

On 2013-05-02 19:31 +0100, ~suv wrote:
> This issue needs to be backported to the stable release branch because
> the change which introduced it was also committed to 0.48.x in revision
> 9948.

Proposing to instead revert revision 9948 of the stable release branch:
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/RELEASE_0_48_BRANCH/revision/9948>

Reasons:
1) The committed change does break GUI options in extension dialogs as described in this report.
2) The committed change has no effect as claimed in the commit message on choosing the export area on the command line: the extension dialog for PDF/PS export in current stable do not use radiobuttons - it's still all booleans [*] (the fix for bug #499965 was not backported to the stable release branch) and works as expected.
3) Choosing (PDF) export area via command line is not broken with current stable release 0.48.4 (see bug #1102069 - the regression was introduced in trunk only).

@Johan - could you please revisit the changes you committed to the stable release branch in r9948, and consider reverting it? Else IMHO the risk is high that the next bug-fix release (0.48.5) will ship with yet another regression (this time one we even knew about before the release).

-----
[*] PDF export options in stable and trunk:
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/RELEASE_0_48_BRANCH/view/9948/src/extension/internal/cairo-renderer-pdf-out.cpp#L232>
<http://bazaar.launchpad.net/~inkscape.dev/inkscape/trunk/view/12322/src/extension/internal/cairo-renderer-pdf-out.cpp#L237>