-
Notifications
You must be signed in to change notification settings - Fork 74
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
env variable SAPNWRFC_HOME read wrong on Windows 2016 #237
Comments
Where it is reported? What are the steps to reproduce? |
Node-rfc is installed via
if I do echo %SAPNWRFC_HOME% it shows C:\nwrfcsdk If I do node -p "require('node-rfc').environment" it shows "SAPNWRFC_HOME": "C:\nwrfcsdk", here the complete message: The SAP NW RFC SDK could not be loaded, check the installation: https://github.com/SAP/node-rfc/blob/master/doc/installation.md#sap-nwrfc-sdk-installation
|
sorry closed it instead of saving |
weird in copy and paste it shows only one Error: The specified module could not be found. The SAP NW RFC SDK could not be loaded, check the installation: https://github.com/SAP/node-rfc/blob/master/doc/installation.md#sap-nwrfc-sdk-installation |
Sorry for the pending response here. The '\' is standard output representation on Windows (esc character + folder separator). The reason why node-rfc is not loading is because the node-rfc and SAP NWRFC SDK are not supported on Node-RED platform. See: #161 and #148 The node-rfc README section about Node-RED will be updated, pointing to these feature requests. |
Describe the bug
there is always an additional / when the environment variable is red in Node-Red
configure as C:/nwrfcsdk
only in Node Red it is reported as C://nwrfcsdk and therefore the sdk cannot be loaded
To Reproduce
Describe how the
node-rfc
is installed, like locally or globally and usingnpm install
or build from source.Describe steps to reproduce the behavior, including the test script.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment
node -p "require('node-rfc').environment"
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: