aboutsummaryrefslogtreecommitdiff
path: root/bdep/fetch.cxx
diff options
context:
space:
mode:
authorBoris Kolpackov <boris@codesynthesis.com>2019-10-24 08:23:42 +0200
committerBoris Kolpackov <boris@codesynthesis.com>2019-10-24 08:23:42 +0200
commit44d5647b4b92089fc90195b41373f6b99dc5c42c (patch)
tree8b11ace1051875e9d7ffe910885ca2790f2d1ca1 /bdep/fetch.cxx
parent282361b57f66b59a50ed3d073b46c7189c9553ee (diff)
Close database as soon as possible
This becomes important if we trigger a recursive implicit sync that uses the same database (e.g., in a build system module project).
Diffstat (limited to 'bdep/fetch.cxx')
-rw-r--r--bdep/fetch.cxx14
1 files changed, 10 insertions, 4 deletions
diff --git a/bdep/fetch.cxx b/bdep/fetch.cxx
index 7a88106..3527e51 100644
--- a/bdep/fetch.cxx
+++ b/bdep/fetch.cxx
@@ -33,11 +33,17 @@ namespace bdep
tracer trace ("fetch");
dir_path prj (find_project (o));
- database db (open (prj, trace));
- transaction t (db.begin ());
- configurations cfgs (find_configurations (o, prj, t));
- t.commit ();
+ configurations cfgs;
+ {
+ // Don't keep the database open longer than necessary.
+ //
+ database db (open (prj, trace));
+
+ transaction t (db.begin ());
+ cfgs = find_configurations (o, prj, t);
+ t.commit ();
+ }
bool first (true);
for (const shared_ptr<configuration>& c: cfgs)