From cfa612e5b2ea0d9938e4e8779628ac6a84edc711 Mon Sep 17 00:00:00 2001 From: Peter Palfrader Date: Mon, 12 Mar 2012 15:52:04 +0100 Subject: [PATCH] Use eatmydata! --- debian/changelog | 4 +++- ud-generate | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/debian/changelog b/debian/changelog index 33974b4..57f4daf 100644 --- a/debian/changelog +++ b/debian/changelog @@ -44,6 +44,8 @@ userdir-ldap (0.3.80) UNRELEASED; urgency=low - get rid of global state variable CurrentHost. This will enable upcoming changes. - UDLdap.py: make a cache for __getitem__() decisions. + - wrap cdbmake calls in eatmydata. Nothing else does any fsync stuff, + so doing it here just costs a lot. [ Stephen Gran ] * Fix deprecation warnings for sha module by using hashlib module instead @@ -56,7 +58,7 @@ userdir-ldap (0.3.80) UNRELEASED; urgency=low * ud-replicate: set correct permissions for web-passwords * add freecdb to depends - -- Peter Palfrader Mon, 12 Mar 2012 15:47:13 +0100 + -- Peter Palfrader Mon, 12 Mar 2012 15:51:06 +0100 userdir-ldap (0.3.79) unstable; urgency=low diff --git a/ud-generate b/ud-generate index d78dcaa..badef2f 100755 --- a/ud-generate +++ b/ud-generate @@ -570,7 +570,9 @@ def GenCDB(accounts, File, key): Fdb = None try: OldMask = os.umask(0022) - Fdb = os.popen("cdbmake %s %s.tmp"%(File, File), "w") + # nothing else does the fsync stuff, so why do it here? + prefix = "/usr/bin/eatmydata " if os.path.exists('/usr/bin/eatmydata') else '' + Fdb = os.popen("%scdbmake %s %s.tmp"%(prefix, File, File), "w") os.umask(OldMask) # Write out the email address for each user -- 2.20.1