RDKEMW-9440 : Fixed issue with TV Builds. - #339
Conversation
Signed-off-by: Balaji Punnuru <Balaji_Punnuru@comcast.com>
Signed-off-by: Balaji Punnuru <Balaji_Punnuru@comcast.com>
Signed-off-by: Balaji Punnuru <Balaji_Punnuru@comcast.com>
…ed settings. Signed-off-by: Balaji Punnuru <Balaji_Punnuru@comcast.com>
Signed-off-by: Balaji Punnuru <Balaji_Punnuru@comcast.com>
|
Balaji Punnuru seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
karuna2git
changed the title
| cp /opt/NetworkManager/system-connections/* /opt/secure/NetworkManager/system-connections/ | ||
| rm -rf /opt/NetworkManager/system-connections/* | ||
| fi | ||
| nmcli conn reload |
There was a problem hiding this comment.
This can be moved inside the if - fi as thats where we do the connection settings moved.
Having it outside will lead to calling reload on every reboot.
There was a problem hiding this comment.
Discussed with Balaji and confirmed that this is not affecting the boot-up and having zero impact.. so leaving it as is; we can address it in develop properly
| if [ -d /opt/NetworkManager/system-connections ]; then | ||
| rm -rf /opt/NetworkManager/system-connections/* | ||
| fi | ||
| if [ "$RDK_PROFILE" == "TV" ]; then |
There was a problem hiding this comment.
This block of script will not execute at all for TV platforms as the SSID will be ALWAYS empty.
There was a problem hiding this comment.
Discussed with Balaji and he mentioned that this is just to double-sure. Im okey with this change
karuna2git
changed the title
4 participants
No description provided.