Add SDL version info in BasicCommunication.GetSystemInfo Response

[henry-fan]

Head unit wants to show SDL version in system diagnosis. Suggest we add SDL version in HMI protocol GetSystemInfo.

[justinjdickow]

What type do you think system version should be?

[henry-fan]

Something like "SDL_RB_B3.6_{build number}", I can see a CCPU version left blank in system info response. Is it intended to indicate the SDL version?

[justinjdickow]

Oops, I didn't realize this was already an issue. I would suggest we also add a new field alongside syncMsgVersion so we can eventually deprecate that field which will indicate the version in the register app interface response.

[justinjdickow]

Some feedback on this issue I got via email

There is a constant string(kApplicationVersion) to define the SDL Core version but isn’t exposed to HMI protocol.

Considering there are variants HMI implementations near future, that should be an enhancement

[justinjdickow]

We added sdl version and system software version to register app interface response to communicate this information. It sounds like the proposal is to also add this information to be sent to the HMI.

[justinjdickow]

@henry-fan Can you let us know if that information in the register app interface being communicated to the application is enough for your system diagnosis purposes or whether it needs to be on the hmi?

[henry-fan]

@justinjdickow
we need SDL version string to be exposed to HMI. Thanks.

[justinjdickow]

@henry-fan Do you know if this was ever resolved?

[BrandonHe]

@justinjdickow
As far as we know, neither sdl_version nor kApplicationVersion exposed to HMI. Thanks.

[Jack-Byrne]

@henry-fan I think this is partly covered by https://github.com/smartdevicelink/sdl_evolution/blob/master/proposals/0096-deliver-build-configuration.md

If this solution does not satisfy your needs please consider submitting a proposal to the sdl evolution repository.

https://github.com/smartdevicelink/sdl_evolution