Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

always use full paths when starting scripts #367

Closed
wants to merge 1 commit into from

2 participants

@aMoniker

In response to this issue.

There are two downsides I can see to this.

One is that the command-line output of forever list will now be a little bulkier.

The other is that some people might currently rely on the ability to stop identically-named scripts with a single command. However, that would mean they're relying on undocumented functionality that could be classified as a bug anyway.

@indexzero indexzero closed this in 6047462
@indexzero
Owner

So not taking this, because as you said its overly verbose in the simple case. But fret not! If you really care about this you can now do:

$ forever columns add dir
$ forever list
info:    Forever processes running
data:        uid  command             script                     forever pid   logfile                          uptime       dir                                          
data:    [0] b7z5 /usr/local/bin/node examples/server.js -p 8080 13998   13999 /Users/charlie/.forever/b7z5.log 0:0:3:26.508 /Users/charlie/Git/nodejitsu/forever-monitor
@aMoniker

Excellent, thank you. Until now I've relied on differentiating processes by the names of logfiles :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 22, 2012
  1. @aMoniker
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/forever/cli.js
View
4 lib/forever/cli.js
@@ -221,7 +221,7 @@ app.cmd('cleanlogs', cli.cleanLogs = function () {
// process.
//
app.cmd(/start (.+)/, cli.startDaemon = function () {
- var file = app.argv._[1],
+ var file = process.env.PWD + '/' + app.argv._[1].replace(/^.+\//, ''),
options = getOptions(file);
forever.log.info('Forever processing file: ' + file.grey);
@@ -492,7 +492,7 @@ app.cmd('help', cli.help = function () {
// make executing other commands possible.
//
cli.run = function () {
- var file = app.argv._[0],
+ var file = process.env.PWD + '/' + app.argv._[0].replace(/^.+\//, ''),
options = getOptions(file);
tryStart(file, options, function () {
Something went wrong with that request. Please try again.