[Libreoffice-bugs] [Bug 117731] Batch libreoffice --convert-to offers no way to wait for document completion

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=117731

Xisco Faulí <xiscofauli at libreoffice.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEEDINFO

--- Comment #14 from Xisco Faulí <xiscofauli at libreoffice.org> ---
(In reply to Richard Elkins from comment #9)
> The source code indicates that soffice.bin is the one starting a background
> process that does not finish before soffice.bin exits.
> 
> See:
> https://github.com/LibreOffice/core/blob/master/shell/source/unix/exec/
> shellexec.cxx
> Go to line 218:
> 
>     OString cmd =
> #ifdef LINUX
>         // avoid blocking (call it in background)
>         "( " + aBuffer.makeStringAndClear() + " ) &";
> #else
>         aBuffer.makeStringAndClear();
> #endif
>     FILE *pLaunch = popen(cmd.getStr(), "w");
>     if ( pLaunch != nullptr )
>     {
>         if ( 0 == pclose( pLaunch ) )
>             return;
> 
> It would be interesting to understand why execution is in background *only*
> for Linux. In my opinion, it is undesirable for command line execution in
> any O/S.
> 
> Other opinions?  It is possible for me to be perfectly content with the
> artificial `sleep N` in my shell script.

it was added in
https://cgit.freedesktop.org/libreoffice/core/commit/?id=f5fe1aa1 to fix bug
32427

@Richard, can you confirm removing the mentioned ifdef condition would fix the
issue? In that case we could submit a patch removing it. Most likely bug 32427
is obsolete nowadays...

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20200120/8f5464fe/attachment.htm>