Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Feature: --bind server to IP address #285

Merged
merged 1 commit into from Dec 20, 2012

Conversation

Projects
None yet
3 participants
Contributor

whitepaperclip commented Dec 16, 2012

Note: it looks like --option isn't working, but -o switches are. Also seems to affect --port.

Contributor

larzconwell commented Dec 16, 2012

I'll let @mde decide if we should merge this, since we currently don't have an hostname cli option either, but if we decide to merge it you should also make bind a full option, adding it to lib/base_config.js.

Also note that long options must use the "--option=value" type, which traditionally is the correct way to set them, only the short options allow a space instead of = sign.

Contributor

whitepaperclip commented Dec 16, 2012

Ah! mde and I are herping and derping in the IRC channel. --option=value works!

@mde mde added a commit that referenced this pull request Dec 20, 2012

@mde mde Merge pull request #285 from whitepaperclip/master
Feature: --bind server to IP address
dd86e30

@mde mde merged commit dd86e30 into geddy:master Dec 20, 2012

Contributor

mde commented Dec 20, 2012

Merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment