Review: Needs Fixing
- I like the signatures and the documentation
- The two modules should both live in the same branch process (subdirectories
process/v1 and process/v2) for the versioning to work the v2 one needs to be
added first and then the v1 one. If we don't want to change all of our
infrastructure immediately, we need to have both versions installed.
- update license header to 2013
- create_result_node(...) in process.cpp is not used anymore
- such as "sh" on Linux or "cmd.exe" on Windows. => such as "sh" on Unix or
"cmd.exe" on Windows.
- import module namespace process =
process:exec("export", "", "foo=bar")
"exit-code" : 139,
"stdout" : "",
"stderr" : "execl: No such file or directory\nID: 80 Referenced URI:
http://www.zorba-xquery.com/modules/process\nID: 94 Referenced URI:
- the referenced uri message seems to be related to a memory leak
Your team Zorba Coders is subscribed to branch lp:zorba/process-module.
Mailing list: https://launchpad.net/~zorba-coders
Post to : email@example.com
Unsubscribe : https://launchpad.net/~zorba-coders
More help : https://help.launchpad.net/ListHelp