Am 17.02.2020 um 11:29 schrieb Enrico Forestieri <for...@lyx.org>: > > On Mon, Feb 17, 2020 at 10:05:01AM +0100, Stephan Witt wrote: >> Hi all, >> >> while trying to understand how temp directory management works >> I’ve stumbled over the „lyx_tmpdir“ string constant use. >> >> 1. This magic name should be coded in filetools only. >> For this I didn’t prepare a patch. >> >> BTW: is the code in lyxSockets() of lyxclient ok? >> There is a check for "lyx_tmpdir" + pid but I cannot see >> how this match should happen. > > Normally, lyxclient looks for all temporary directories and returns > all the ones in which it finds a lyxsocket. One can restrict the > search to a particular one by specifying -p pid on the command line, > otherwise pid is empty. > > The documentation has to be updated for specifying that pid is > actually the stem of "lyx_tmpdir" and not the pid of the process.
Ah, now I see. Thanks. >> 2. The attached patch simplifies LyX::prepareExit() >> IMO it’s safe and an improvement. Isn’t it? > > I don't think you need a nod to apply this patch ;-) It’s in with commit hash 0f1f00db27. Thanks again. Stephan -- lyx-devel mailing list lyx-devel@lists.lyx.org http://lists.lyx.org/mailman/listinfo/lyx-devel
