[whimsy] branch master updated: File.writable? only works if file exists...

[ru...@apache.org]

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 <ru...@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" <co...@whimsical.apache.org>'].