Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow to set IIPs for exported ports on commandline #17

Open
jonnor opened this issue Mar 26, 2015 · 2 comments
Open

Allow to set IIPs for exported ports on commandline #17

jonnor opened this issue Mar 26, 2015 · 2 comments

Comments

@jonnor
Copy link
Member

jonnor commented Mar 26, 2015

I think exported ports on top-level graph should be instantly accessible as command-line options, including generated --help docs

Embedding api should exist for this also, so once could have a "myexec" js/coffee file which just specifies graph and rest is handled by this "mounter".
This would make it very easy to make standard commandline tools using NoFlo

@ensonic
Copy link
Contributor

ensonic commented Mar 30, 2015

We can add a --args key=value options that can be specified multiple times. If no graph is given on the commandline we ignore them. Otherwise:

  • we create a on-the-fly component to connect tot he exports
  • when it is getting loaded, we listen for "addExport" signals, for each signal emission we create ports of the opposite direction, set the IIP and connect them to the export. This might handle input parameters. If we have special values for STDIN, STDERR and STDOUT we can also stream data. If there are multiple ouput exports, only one can be selected (or we need a way to mux them).

Generating --help output is tricky is one would need to do: noflo-nodejs --help --graph abc.fbp

@jonnor
Copy link
Member Author

jonnor commented Mar 30, 2015

Sounds good. That way of doing help would be acceptable for the noflo-nodejs exectuable I think

jonnor added a commit to flowhub/bigiot-bridge that referenced this issue Nov 14, 2017
Needed to pass the hostname data

If could pass IIPs on commandline, would not be needed:
noflo/noflo-nodejs#17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants