diff options
author | Boris Kolpackov <boris@codesynthesis.com> | 2019-10-24 08:23:42 +0200 |
---|---|---|
committer | Boris Kolpackov <boris@codesynthesis.com> | 2019-10-24 08:23:42 +0200 |
commit | 44d5647b4b92089fc90195b41373f6b99dc5c42c (patch) | |
tree | 8b11ace1051875e9d7ffe910885ca2790f2d1ca1 /bdep/fetch.cxx | |
parent | 282361b57f66b59a50ed3d073b46c7189c9553ee (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.cxx | 14 |
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) |