Merge lp:~smartboyhw/wubi/bug-1080090 into lp:wubi
| Status: | Merged |
|---|---|
| Approved by: | Colin Watson on 2012-11-19 |
| Approved revision: | 276 |
| Merged at revision: | 276 |
| Proposed branch: | lp:~smartboyhw/wubi/bug-1080090 |
| Merge into: | lp:wubi |
| Diff against target: |
35 lines (+7/-2) 2 files modified
debian/changelog (+5/-1) src/wubi/backends/common/distro.py (+2/-1) |
| To merge this branch: | bzr merge lp:~smartboyhw/wubi/bug-1080090 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Colin Watson | 2012-11-17 | Approve on 2012-11-19 | |
|
Review via email:
|
|||
Commit Message
Changed code in distro.py to not replace hyphens with spaces (LP: #1080090)
Description of the Change
== Background ==
bcbc reported that when he tries out the code to install Ubuntu Studio into Wubi, he received a dialog telling him that it still didn't match the data in /.disk/info. However I checked the name in isolist.ini and it is correct. So I decided to report a bug (LP: #1080090) to track the bug/problem.
== Issues ==
It turns out in /src/wubi/
name = info.group(
that it replaces hyphens with spaces.
The original correct name "Ubuntu-Studio" turns into "Ubuntu Studio", which then does not fit the data in .disk/info
== Changed code ==
I went to delete the .replace("-", " ") and also added the example of Ubuntu-Studio ISO .disk/info into it so that people know why I changed the code to not replace hyphens with spaces.
BTW this code doesn't affect other distros, since that only our distro has two words in our distro name.


OK, but on the understanding that this is a temporary hack and that it should be reverted as soon as the underlying problems are fixed. I'll add a comment to that effect.