tdf#144172: patch FB to use per-process LOCK directories
commita5bb0e66492118b79da5fef31f8d2bd686947354
authorMike Kaganski <mike.kaganski@collabora.com>
Wed, 1 Sep 2021 15:08:10 +0000 (1 17:08 +0200)
committerMike Kaganski <mike.kaganski@collabora.com>
Thu, 2 Sep 2021 06:39:54 +0000 (2 08:39 +0200)
tree72fadb7e57dbb27b843dcd03ba4293b5bbfc7e02
parent7ded01efaec8171a204a255395767432bab881b5
tdf#144172: patch FB to use per-process LOCK directories

As discussed at https://github.com/FirebirdSQL/firebird/pull/6937:

> > But as well, we may just use process id instead of session id in
> > the ConfigStorage::ConfigStorage creating filename. That would be
> > even better, fitting better into our per-process temp dir policy.
> I tend to agree with you here. It looks safe assuming LO uses
> SuperServer only.

Let's not replace the session id with process id, but instead append
process id after session id, to avoid (unlikely?) accidental overlap
of process id and session id, which would produce the same name as
already used by, say, isql tool.

Change-Id: Ibce8a8c75d405aae3e8949e7daf2476accdff22b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121457
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
external/firebird/UnpackedTarball_firebird.mk
external/firebird/wnt-per-process-trace-storage.patch.1 [new file with mode: 0644]