the "Ubuntu Help" should be "Ubuntu Kylin Help"

IRC discussion from #ubuntu-desktop:

[09:16:50] <charles> hmm, I wonder if there's a better way to do things like this so that we don't wind up with #ifdef-level code all over the place
[09:16:55] <charles> https://code.launchpad.net/~445865575-b/indicator-session/indicatorForKylin/+merge/223664
[09:20:31] <Laney> maybe they should divert /etc/os-release or something
[09:29:15] <seb128> charles, what laney said, use NAME from /etc/os-release and get Kylin to change that
[09:29:36] <seb128> like "%s Help", NAME
[09:30:19] <seb128> happyaron, FJKong: do you if Kylin change its /etc/os-release?
[09:30:50] <happyaron> seb128: Kylin does not change it atm
[09:32:37] <seb128> happyaron, do you know if they discussed change it?
[09:34:43] <happyaron> seb128: they didn't
[09:35:26] <happyaron> seb128: they added a file, /etc/ubuntukylin-release, for identifying Kylin systems.
[09:35:52] <Laney> it means that programs have to special case kylin though
[09:35:55] <seb128> happyaron, well, that forces code to special case rather than just reading os-release
[09:37:42] <happyaron> seb128: they want to keep compatibility with Ubuntu as much as possible, so that all program that do OS checking can identify it as Ubuntu, except those ones can recognize the additional indication of Kylin flavor.
[09:37:58] <seb128> shrug
[09:38:09] <seb128> they want to be Ubuntu but not be flagged as Ubuntu at the same time
[09:38:23] <happyaron> that's it
[09:39:02] <Laney> http://www.freedesktop.org/software/systemd/man/os-release.html
[09:39:06] <Laney> I think that NAME is for presentation

Anthony, is there harm in changing the presentation name in /etc/os-release? What breaks if we set NAME=Kylin Ubuntu there?

Charles, changing NAME should be fine.
I will ask Jack's team, who is subscribed to this bug, to verify if there are any side-effects.

Charles & Anthony, I think it's fine to use NAME /etc/os-release, instead of adding /etc/ubuntukylin-release. We just want this system could be recongnized as 'Ubuntu Kylin'. We will check the effects asap and update here.

We will divert it in ubuntukylin-default-settings package and update the code, Thanks!

How about bug: https://bugs.launchpad.net/ubuntukylin/+bug/1329584, use the same way like: "%s Desktop", Name?

Yep, sounds good to me.

Handsome, will the update be pushed to your indicatorForKylin branch so we can keep the same MR?

yes , i have updated my branch and pushed it up ,thank you !

However you name the menu label, it takes you to Ubuntu Desktop Guide. Or does Ubuntu Kylin ship its own user help package?

we have made our own user help package.

Aha, thanks, that explains it. ;)

Thank you! :)

This bug was fixed in the package indicator-session - 12.10.5+14.10.20140717-0ubuntu1

---------------
indicator-session (12.10.5+14.10.20140717-0ubuntu1) utopic; urgency=low

  [ Charles Kerr ]
  * Add support for getting the distro name from /etc/os-release. (LP:
    #1331873)
 -- Ubuntu daily release <email address hidden> Thu, 17 Jul 2014 08:33:50 +0000

ubuntukylin-default-settings/1.2.0 is updated in utopic to set NAME to Ubuntu Kylin in /etc/os-release, please verify if this works in an up-to-date system and close the UbuntuKylin task if appropriate.

I tested it in ubuntukylin 14.10 a short while ago ,it works well ,thank you !

hi, Charles, does the translation of "%s Help" or "Help" update in indicator-session?

hi, The help label has not been translated yet.

https://translations.launchpad.net/ubuntu/vivid/+source/indicator-session/+pots/indicator-session/zh_CN/4/+translate

Verified fixed.

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
http://iso.qa.ubuntu.com/qatracker/reports/bugs/1331873