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

Wolfgang Bumiller w.bumiller at proxmox.com
Thu Jan 31 12:02:18 CET 2019


applied, one less leftpad

On Wed, Jan 30, 2019 at 05:20:50PM +0100, Thomas Lamprecht wrote:
> 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