Comment 21 for bug 1342175

Revision history for this message
Björn Michaelsen (bjoern-michaelsen) wrote :

@xtrusia: Did you ever even test that debdiff vs. 4.2? I doesnt even seem to compile:
- For one there seems to be an obvious typo in using both rMark.getMarkedRanges() and rMark.GetMarkedRanges() in the same patch
- Curiously this typo isnt even there in the upstream commit 91502a72c12c559442e8bf77c27a516b49c2a68d from which it was derived
- then the patch uses sc::ColRowSpan which does not seem to be available on LibreOffice 4.2.x yet

Here is an example build log failure:
https://launchpadlibrarian.net/187606247/buildlog_ubuntu-trusty-amd64.libreoffice_1%3A4.2.7~rc2-0ubuntu1_FAILEDTOBUILD.txt.gz

Im not too happy to have wasted some time to rebase the patch from 4.2.6 on 4.2.7, now realizing this likely never build on 4.2.6 in the first place.

In the future, please in general consider using the upstream backport branches as per: https://wiki.documentfoundation.org/Development/Branches#libreoffice-X-Y on https://gerrit.libreoffice.org. For one, this gives you the support of upstream CI and buildbots, but more importantly also the support of upstream reviewers. Sadly, its now too late for this to succeed with that as 4.2.7.2 has been tagged and is at the end of the upstream support for this series.

For now, I will leave the patch in packaging (so that the rebase work isnt lost), but disable it. If you want this patch to still make the next trusty SRU, the patch needs to be in a good form against 4.2.7 and be thoroughly tested before the end of October 2014. That means you need to do a build and at least run the autopkgtests against it and at least do some basic manual testing in the patched functionality. Thanks.