X-Git-Url: https://git.p6c8.net/devedit.git/blobdiff_plain/02321769f658fbe95c046c17f0c5b4de759d282f..95df6a6a2bb9185cceb1fcf205b4f9f07690ca27:/modules/File/Access.pm diff --git a/modules/File/Access.pm b/modules/File/Access.pm index bc1e37d..31377d9 100644 --- a/modules/File/Access.pm +++ b/modules/File/Access.pm @@ -6,45 +6,38 @@ package File::Access; # Some simple routines for doing things with files # with only one command # -# Author: Patrick Canterino -# Last modified: 2004-08-01 +# Author: Patrick Canterino +# Last modified: 2004-12-17 # use strict; -use vars qw(@EXPORT); +use vars qw(@EXPORT + $has_flock); -use Carp qw(croak); +use Fcntl qw(:DEFAULT + :flock); ### Export ### use base qw(Exporter); -@EXPORT = qw(chgrp - dir_read +@EXPORT = qw(dir_read file_create + file_lock file_read file_save - file_unlock); + file_unlock -# chgrp() -# -# Change the group of files or directories -# -# Params: 1. Group name -# 2. List of files -# -# Return: Number of files group successfully changed -# (or false) + LOCK_SH + LOCK_EX + LOCK_UN + LOCK_NB); -sub chgrp($@) -{ - my ($group,@files) = @_; - my $gid = ($group !~ /^\d+$/) ? getgrnam($group): $group; +# Check if flock() is available +# I found this piece of code somewhere in the internet - return unless($gid); - return chown(-1,$gid,@files); -} +$has_flock = eval { local $SIG{'__DIE__'}; flock(STDOUT,0); 1 }; # dir_read() # @@ -94,7 +87,11 @@ sub dir_read($) # file_create() # -# Create a file +# Create a file, but only if it doesn't already exist +# +# (I wanted to use O_EXCL for this, but `perldoc -f sysopen` +# doesn't say that it is available on every system - so I +# created this workaround using O_RDONLY and O_CREAT) # # Params: File to create # @@ -108,12 +105,31 @@ sub file_create($) return if(-e $file); - open(FILE,">$file") or return; - close(FILE) or return; + sysopen(FILE,$file,O_RDONLY | O_CREAT) or return; + close(FILE) or return; return 1; } +# file_lock() +# +# System independent wrapper function for flock() +# On systems where flock() is not available, this function +# always returns true. +# +# Params: 1. Filehandle +# 2. Locking mode +# +# Return: Status code (Boolean) + +sub file_lock(*$) +{ + my ($handle,$mode) = @_; + + return 1 unless($has_flock); + return flock($handle,$mode); +} + # file_read() # # Read out a file completely @@ -127,9 +143,13 @@ sub file_read($) my $file = shift; local *FILE; - open(FILE,"<$file") or return; + sysopen(FILE,$file,O_RDONLY) or return; + file_lock(FILE,LOCK_SH) or do { close(FILE); return }; + read(FILE, my $content, -s $file); - close(FILE) or return; + + file_lock(FILE,LOCK_UN) or do { close(FILE); return }; + close(FILE) or return; return \$content; } @@ -140,17 +160,24 @@ sub file_read($) # # Params: 1. File # 2. File content as Scalar Reference +# 3. true => open in binary mode +# false => open in normal mode (default) # # Return: Status code (Boolean) -sub file_save($$) +sub file_save($$;$) { - my ($file,$content) = @_; + my ($file,$content,$binary) = @_; local *FILE; - open(FILE,">$file") or return; - print FILE $$content or do { close(FILE); return }; - close(FILE) or return; + sysopen(FILE,$file,O_WRONLY | O_CREAT | O_TRUNC) or return; + file_lock(FILE,LOCK_EX) or do { close(FILE); return }; + binmode(FILE) if($binary); + + print FILE $$content or do { close(FILE); return }; + + file_lock(FILE,LOCK_UN) or do { close(FILE); return }; + close(FILE) or return; return 1; }