Merge lp://staging/~jjesse/ubuntu/quantal/libtest-hasversion-perl/typo-fix into lp://staging/ubuntu/quantal/libtest-hasversion-perl

Proposed by Jonathan Jesse
Status: Work in progress
Proposed branch: lp://staging/~jjesse/ubuntu/quantal/libtest-hasversion-perl/typo-fix
Merge into: lp://staging/ubuntu/quantal/libtest-hasversion-perl
Diff against target: 26 lines (+7/-1)
2 files modified
debian/changelog (+6/-0)
debian/control (+1/-1)
To merge this branch: bzr merge lp://staging/~jjesse/ubuntu/quantal/libtest-hasversion-perl/typo-fix
Reviewer Review Type Date Requested Status
Michael Terry Disapprove
Ubuntu branches Pending
Review via email: mp+125545@code.staging.launchpad.net

Description of the change

changed typo in control file

To post a comment you must log in.
Revision history for this message
Michael Terry (mterry) wrote :

Thanks for the patch! This will sound familiar to you from the other merge request, but this typo doesn't seem worth a delta from Debian.

Some nits about your patch:
* You use precise in the changelog, but it should have been quantal
* Keep the changelog text on the same line as the asterisk

I'll reject the merge request for Ubuntu. But if you fix this patch up, it should be passed on to Debian.

https://wiki.ubuntu.com/Debian/ForUbuntuDevelopers#Forwarding_bug_reports

Or maybe another solution would be to fix the typo in the English translation? https://translations.launchpad.net/ddtp-ubuntu/+translations

Thanks!

review: Disapprove

Unmerged revisions

3. By Jonathan Jesse

made typo change in control file

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches