[Libreoffice] Patch review

Petr Mladek pmladek at suse.cz
Thu Jan 13 10:01:24 PST 2011


Kohei Yoshida píše v Čt 13. 01. 2011 v 11:38 -0500:
> Hi there,
> 
> The attached patches revert the recent two commits which consist of the
> merging of OOo330-m19 changes.
> 
> It fixed i#115906 and i#116164, but the fix for i#116164 unfortunately
> was pretty large and invasive, plus it introduced a regression as
> reported in i#116439 of OOo bug tracker.
> 
> It is my opinion that, since these fixes don't fit *our* blocker
> criteria, plus the change is too large to be in RC, reverting them will
> be our best option.
> 
> The master branch already contains these fixes, and we'll work on fixing
> any remaining issues in that branch.
> 
> Anyway, review and sign-off appreciated.

Looks fine. I checked:

	+ the patches are complete and exact revert of the
          problematic committs
	+ build succeeded
	+ runtime test confirmed that it fixed i#116439 (was able
          to reproduce it with the plain rc3 build)


Best Regards,
Petr



More information about the LibreOffice mailing list