Merge lp:~kroq-gar78/ubuntu/precise/lincity-ng/fix-674519 into lp:ubuntu/precise/lincity-ng
Proposed by
kroq-gar78
on 2012-03-24
| Status: | Rejected |
|---|---|
| Rejected by: | Martin Pitt on 2012-03-27 |
| Proposed branch: | lp:~kroq-gar78/ubuntu/precise/lincity-ng/fix-674519 |
| Merge into: | lp:ubuntu/precise/lincity-ng |
| Diff against target: |
26 lines (+7/-1) 2 files modified
debian/changelog (+6/-0) lincity-ng.desktop (+1/-1) |
| To merge this branch: | bzr merge lp:~kroq-gar78/ubuntu/precise/lincity-ng/fix-674519 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Daniel Holbach | Disapprove on 2012-03-27 | ||
| Ubuntu branches | 2012-03-24 | Pending | |
|
Review via email:
|
|||
Description of the Change
I fixed the typo "electricityand" to "electricity and" in lincity-ng.desktop.
To post a comment you must log in.
Unmerged revisions
- 8. By kroq-gar78 on 2012-03-24
-
lincity-ng.desktop: fix typo "electricityand" (LP: #674519)


Thanks a lot for your work on this.
A few small bits I noticed:
- the package uses the quilt patch system, so it would be nice if you
could use edit-patch to make modifications to the actual source code
- as your change modifies source code (it's not a mere rebuild), it
would make sense to change the revision in debian/changelog to
2.0-2ubuntu1 instead of 2.0-2build2.
The change is pretty small and maybe not worth creating a delta between Debian and Ubuntu, so I would suggest you forward the change to lincity-ng.desktop to Upstream. (As Upstream is moving from berlios to fedorahosted, it might be enough to just mail it to this mailing list.)
Thanks again for your work on this. I hope you agree with the suggested course of action. (I'll mark this as 'Disapprove' just to indicate that we don't merge this as-is, but instead forward to Upstream and let the change trickle back from there.)