Merge lp://qastaging/~sense/ubuntu/maverick/evolution/fix-588298 into lp://qastaging/ubuntu/maverick/evolution
Proposed by
Sense Egbert Hofstede
| Status: | Merged |
|---|---|
| Merge reported by: | Didier Roche-Tolomelli |
| Merged at revision: | not available |
| Proposed branch: | lp://qastaging/~sense/ubuntu/maverick/evolution/fix-588298 |
| Merge into: | lp://qastaging/ubuntu/maverick/evolution |
| Diff against target: |
12 lines (+1/-1) 1 file modified
mail/mail-config.ui (+1/-1) |
| To merge this branch: | bzr merge lp://qastaging/~sense/ubuntu/maverick/evolution/fix-588298 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Didier Roche-Tolomelli | Approve | ||
| Ubuntu branches | Pending | ||
|
Review via email:
|
|||
Description of the change
Fixes bug #588298
PO files will have to be generated and translations will require updates.
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Hey sense, thanks for your patch!
Some remarks:
* we tend to avoid patching inline the file, but rather using a patch system. You can find good documentation on patch system used in debian/ubuntu there:
https:/
https:/
https:/
You will find some classroom too about how to use them.
* in debian/changelog, you have sometimes a Vcs-Bzr: url. This point to where we store the packaging into ubuntu (if not present, it assumes lp:ubuntu/<package>). For desktop package, most of them contain a debian/ only directory and are located under ~ubuntu-desktop launchpad team. So, you should next time pay attention to that to propose merging against the right branch. This is only because the "packaging in bzr" is work in progress and we will soon (next cycle?) merge them to the canonical location (lp:ubuntu/<package>.
Not a very big deal, I have done that for you, if you are interested in the result, you can have a look at this commit: http://
Thanks a lot for your patch there!