[pve-devel] [PATCH v2 manager] pveperf: use fsync syscall from PVE::Tools

Thomas Lamprecht t.lamprecht at proxmox.com
Wed Jan 30 17:20:50 CET 2019


allows to drop the libfile-sync-perl dependency

Signed-off-by: Thomas Lamprecht <t.lamprecht at proxmox.com>
---

v1 -> v2:
* fix borked commit

 bin/pveperf    | 6 ++++--
 debian/control | 1 -
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/bin/pveperf b/bin/pveperf
index 6f6fe2d8..d994c239 100755
--- a/bin/pveperf
+++ b/bin/pveperf
@@ -2,10 +2,12 @@
 
 use strict;
 use warnings;
-use File::Sync;
+
 use Time::HiRes qw( usleep ualarm gettimeofday tv_interval );
 use Net::DNS::Resolver;
 
+use PVE::Tools;
+
 if ($#ARGV >= 1) {
     print STDERR "usage: $0 [PATH]\n";
     exit -1;
@@ -84,7 +86,7 @@ sub test_fsync {
  
 	    print TMP $data;
 
-	    File::Sync::fsync (\*TMP);
+	    PVE::Tools::fsync(fileno(\*TMP));
 
 	    close (TMP);
 
diff --git a/debian/control b/debian/control
index 712a4398..a123f7e4 100644
--- a/debian/control
+++ b/debian/control
@@ -38,7 +38,6 @@ Depends: apt-transport-https,
          libcrypt-ssleay-perl,
          libev-perl,
          libfile-readbackwards-perl,
-         libfile-sync-perl,
          libfilesys-df-perl,
          libhttp-daemon-perl,
          libintl-perl,
-- 
2.20.1





More information about the pve-devel mailing list