whimsical-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ru...@apache.org
Subject [whimsy] branch master updated: File.writable? only works if file exists...
Date Wed, 07 Jun 2017 17:15:56 GMT
This is an automated email from the ASF dual-hosted git repository.

rubys pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 6599354  File.writable? only works if file exists...
6599354 is described below

commit 6599354ceab3161e0fec568b305aa09e4db290a6
Author: Sam Ruby <rubys@intertwingly.net>
AuthorDate: Wed Jun 7 13:15:40 2017 -0400

    File.writable? only works if file exists...
---
 tools/pubsub.rb | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/tools/pubsub.rb b/tools/pubsub.rb
index c43f4cf..d8a6258 100644
--- a/tools/pubsub.rb
+++ b/tools/pubsub.rb
@@ -80,8 +80,15 @@ else
   # optionally daemonize
   Process.daemon if options.daemonize
 
+  # Determine if pidfile is writable
+  if File.exist? options.pidfile
+    writable = File.writable? options.pidfile
+  else
+    writable = File.writable? File.dirname(options.pidfile)
+  end
+
   # PID file management
-  if File.writable? options.pidfile
+  if writable
     File.write options.pidfile, Process.pid.to_s
     at_exit { File.delete options.pidfile if File.exists? options.pidfile }
   else

-- 
To stop receiving notification emails like this one, please contact
['"commits@whimsical.apache.org" <commits@whimsical.apache.org>'].

Mime
View raw message