search

foglamp / foglamp-south-opcua

FogLAMP OPC/UA South plugin: an asynchronous OPC UA plugin that registers for change events on OPC UA objects and injects data from the OPC/UA server into FogLAMP
Apache License 2.0
0 stars 0 forks source link

FOGL-2828 Update requirements.sh #22

Closed MarkRiddoch closed 5 years ago

MarkRiddoch commented 5 years ago

Not sure why this did not merge properly to develop previously. My local copy of develop has this it and claims to be up to date with GitHub, but the GitHub content is different. Let's try another merge.

MarkRiddoch commented 5 years ago

make is still failing. Seems like we need same fix as we made for make_deb. Apply same fix for make_plugin_rpm script if plugin is opcua set below code? OR may be other better solution which fits into it

DIR=/tmp
FREEOPCUA=freeopcua
rm -rf ${DIR}/${FREEOPCUA}
./requirements.sh ${DIR}
export FREEOPCUA=${DIR}/${FREEOPCUA}```

No, this is not correct. ~the problem is with the change that was made to make_plugin_rpm to make the requirements.sh run under sudo. This should not have been done, rather only this individual commands in the required,ents.sh that require root privileges should be run under sudo.