Merge lp:~kroq-gar78/ubuntu/precise/lincity-ng/fix-674519 into lp:ubuntu/precise/lincity-ng

Proposed by Aditya V
Status: Rejected
Rejected by: Martin Pitt
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
Reviewer Review Type Date Requested Status
Daniel Holbach (community) Disapprove
Ubuntu branches Pending
Review via email: mp+99151@code.launchpad.net

Description of the change

I fixed the typo "electricityand" to "electricity and" in lincity-ng.desktop.

To post a comment you must log in.
Revision history for this message
Daniel Holbach (dholbach) wrote :

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.)

review: Disapprove

Unmerged revisions

8. By Aditya V

lincity-ng.desktop: fix typo "electricityand" (LP: #674519)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2010-09-25 23:52:22 +0000
3+++ debian/changelog 2012-03-24 04:03:17 +0000
4@@ -1,3 +1,9 @@
5+lincity-ng (2.0-2build2) precise; urgency=low
6+
7+ * lincity-ng.desktop: fix typo "electricityand" (LP: #674519)
8+
9+ -- Aditya Vaidya <kroq.gar78@gmail.com> Fri, 23 Mar 2012 22:56:17 -0500
10+
11 lincity-ng (2.0-2build1) maverick; urgency=low
12
13 * Rebuild against new libphysfs (LP: #614067)
14
15=== modified file 'lincity-ng.desktop'
16--- lincity-ng.desktop 2007-12-05 05:13:23 +0000
17+++ lincity-ng.desktop 2012-03-24 04:03:17 +0000
18@@ -5,7 +5,7 @@
19 Name=Lincity-NG
20 GenericName=City Simulation Game
21 GenericName[de]=Stadtsimulationsspiel
22-Comment=Plan and Build a city and take care of transport, economics, electricityand others.
23+Comment=Plan and Build a city and take care of transport, economics, electricity and others.
24 Icon=lincity-ng.png
25 Exec=lincity-ng
26 Terminal=false

Subscribers

People subscribed via source and target branches

to all changes: