Skip to content
This repository has been archived by the owner on Feb 14, 2018. It is now read-only.

rename express server to sfwd #99

Open
pmuellr opened this issue Jan 16, 2012 · 1 comment
Open

rename express server to sfwd #99

pmuellr opened this issue Jan 16, 2012 · 1 comment

Comments

@pmuellr
Copy link
Contributor

pmuellr commented Jan 16, 2012

In the comments for pull request 98, Ward mentioned:

Nick is looking to rename "server/express/bin/server". Is there a problem with that name? The name "sfw-express" seems to encode information already present in the path to the file.

Seems to me, following typical web server naming idioms, the server should be named "#{thing-it-serves}d". So sfwd would be the name.

Or perhaps since there are two servers, it should be called one of:

  • sfwd-express
  • sfw-expressd
  • ???

I think I like that first one. When the day comes when there is a single reference server, we can rename whichever that is to sfwd

Also, we should look at adding a bin entry to the package.json, so that when folks do an npm -g install, it will install the binary in a nice place. See, for example, https://github.com/jashkenas/coffee-script/blob/master/package.json

@nrn
Copy link
Collaborator

nrn commented Jan 16, 2012

I'm starting to think that just adding the .js back to server/express/bin/server.js may be the way to go. The name of the file doesn't have to be same as the global command when installed with npm -g, see https://github.com/substack/node-browserify/blob/master/package.json#L7

For global commands I do like the "d" at the end to denote a daemon, it helps to make sense of the commands.

I've been waiting on finishing up the package.json with a bin entry and real package versions ranges, etc. until the server/express code stops changing so much. I'm hoping it will have calmed down some in 2 weeks or so.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants