big-r81 commented on PR #5106:
URL: https://github.com/apache/couchdb/pull/5106#issuecomment-2231346379

   Short summary:
   
   How do we want to bring these two solutions together?
   
   1. This PR: 
   Print relative directory and manual joining of the path parts.
   
   2. Bob's idea: 
   Generic function for getting the log dir and print it as absolute path.
   
   I tried Bob's idea on Windows an macOS:
   ```
   Windows:
   [ * ] Setup log dir: C:\Users\couchdb\Documents\couchdb-otp25\dev\logs ... ok
   [ * ] Ensure CouchDB is built ... ok
   [ * ] Ensure Erlang boot script exists ... ok
   
   macOS:
   [ * ] Setup log dir: 
/Users/big-r/Documents/Developer/CouchDB/couchdb/dev/logs ... ok
   [ * ] Ensure CouchDB is built ... ok
   [ * ] Ensure Erlang boot script exists ... ok
   ```
   
   After a few days have passed, I like Bob's (@rnewson) generic dev dir 
function a little bit more. It's cleaner, but with abs paths. What do you think 
Nick (@nickva)?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscr...@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to