Merge lp:~scarneiro/ubuntu/oneiric/pppstatus/fix-for-770756 into lp:ubuntu/oneiric/pppstatus
Status: | Merged |
---|---|
Merged at revision: | 4 |
Proposed branch: | lp:~scarneiro/ubuntu/oneiric/pppstatus/fix-for-770756 |
Merge into: | lp:ubuntu/oneiric/pppstatus |
Diff against target: |
39 lines (+9/-2) 3 files modified
Makefile (+1/-1) debian/changelog (+6/-0) debian/control (+2/-1) |
To merge this branch: | bzr merge lp:~scarneiro/ubuntu/oneiric/pppstatus/fix-for-770756 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stefano Rivera | Approve on 2011-07-06 | ||
Ubuntu branches | 2011-06-18 | Pending | |
Review via email:
|
Description of the change
Changed linker parameters to work with GCC-4.6. Make other changes mainly from lintian recommendations.
Sebastien Bacher (seb128) wrote : | # |
Sebastian Carneiro (scarneiro) wrote : | # |
Sebastien, thanks for your review.
I need clarification about what the right course of action would be in a case like this:
- Do I have to revert changes leaving only the required ones, or
- Do I have to forward to Debian only the required changes, leaving the rest out of the patch ?
Again,
Thanks for your time.
Best regards.
Sebastien Bacher (seb128) wrote : | # |
What I would recommend to do is to forward your diff it is to Debian and update the merge request to drop unrequired changes, fixes are fine to keep (i.e the maintainer script tokens or watch fixes) but changing the source format in an Ubuntu specific way doesn't seem a good idea
You also don't need to list the maintainer update in the changelog that's something we do for each source we modify in Ubuntu
- 5. By Sebastian Carneiro on 2011-06-21
-
Removed unrequired changes, following advice from Sebastien Bacher.
- 6. By Sebastian Carneiro on 2011-06-21
-
Removed unrequired changes, following advice from Sebastien Bacher.
Sebastian Carneiro (scarneiro) wrote : | # |
Sebastien, did the following update to this branch:
- Removed debian/
- Restored previous values in debian/control for Standards-Version and Build-Depends.
- Removed references to previous change from changelog.
Please let me know if it's ok now, or further changes are needed.
Thanks for the patience!
Best regasrds.-
Michael Terry (mterry) wrote : | # |
Sebastian, would you mind just making this branch hold the Makefile fix? The other changes are great and would ideally be sent on to Debian. But if Ubuntu accepts the extra changes, it would make it harder for us to go back to being in sync (we'd generally wait until Debian took them too).
Sebastian Carneiro (scarneiro) wrote : | # |
Sure, no problem.
I will to that and let you know.
Thanks for your help !
Sebastian.
- 7. By Sebastian Carneiro on 2011-07-06
-
Reverted previous changes and holding only the ones on Makefile.
Sebastian Carneiro (scarneiro) wrote : | # |
Changes reverted. Only the change on Makefile remains.
Best regards.
- 8. By Sebastian Carneiro on 2011-07-06
-
Removed extra line in changelog entry.
Stefano Rivera (stefanor) wrote : | # |
Looks good, thanks.
Only issue was overly-long changelog lines. (lintian complains about it) Fixed on upload.
Thank you for the work you did there, the package is coming from Debian (https:/ /wiki.ubuntu. com/Debian) though so ideally we would be sent to their bug system (http:// www.debian. org/Bugs/ Reporting) so they can benefit from your fixes and it Ubuntu has less changes to carry and maintain which are duplicated efforts over what Debian is doing
We also try to limit the changes we do over Debian to required ones, while you standards version update and watch files are nice cleanups they are not really required in an Ubuntu specific way also