On 06/11/2012 05:47 PM, Korrawit Pruegsanusak wrote:
I'd like to propose Stephan's patch into stable branch:

* 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=55836370ffb70a34b888f81cdacdfede8fee29cf
This fixed fdo#43989, which is 3.5 MAB, but we already have Andras'
8783ead70cc2bc2a83bf473b0dfb51f3ee10b6da backported, so I'm not sure
whether we should have Stephan's patch backported too.

Also, from the commit message, Stephan committed new fix for
bnc#524250 at b5ad981518f4407243f89507b2561dd0b1ef28ee. Should this
patch backported as well?

From #libreoffice-dev backlogs:

Jun 07 11:22:35 <sberg>   timar, tml_, I'm unsure whether we want 
55836370ffb70a34b888f81cdacdfede8fee29cf (and maybe b5ad981518f4407243f89507b2561dd0b1ef28ee, 
too) in -3-6 or even -3-5; without them, we'll keep the behavior on Windows that there is no 
automatic restart (is this by now considered a feature, or rather a bug?) but also that LO only 
starts upon second attempt "sporadically" (i.e., when syncing with bundled/shared 
extensions needs to be done, after
Jun 07 11:22:35 <sberg>   those have changed in some way)
Jun 07 11:22:37 <IZBot>   no match found - might try 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=55836370ffb70a34b888f81cdacdfede8fee29cf
 in a couple of minutes
Jun 07 11:23:03 <sberg>   s/no automatic restart/no automatic restart *after 
crash*/
Jun 07 11:42:46 <timar>   sberg: and what about, if we restarted only after 
E_NORMAL_RESTART, and not after crash?
Jun 07 11:49:06 <sberg>   timar, one could fiddle something like that into -3-6 
etc., sure; but I would keep it simple -- either backport wholesale, or live with the 
current situation on -3-6 etc., which should be ~ok after your fix for fdo#43989
Jun 07 11:49:10 <IZBot>   LibreOffice-UI normal/medium RESOLVED FIXED After 
installation first launch terminates after splash screen 
https://bugs.freedesktop.org/show_bug.cgi?id=43989

My take is that b5ad981518f4407243f89507b2561dd0b1ef28ee isn't entirely trivial (there is that curious distinction between bExistsRecoveryData and bExistsSessionData in that code...), but backporting 55836370ffb70a34b888f81cdacdfede8fee29cf without b5ad981518f4407243f89507b2561dd0b1ef28ee wouldn't be ideal, either.

Stephan
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to