Merge proposal diffs should use -p option

Bug #351686 reported by dobey
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

Recently, Launchpad started displaying a diff in merge proposals, and attaching that diff to the e-mails sent for the proposal. However, diff has a very useful -p option, which displays the name of the function/class/etc... in which the change occurs, next to the line offset information. It would be very useful to have the merge proposal diffs using this option as well.

Tags: lp-code
Ursula Junque (ursinha)
affects: launchpad → launchpad-bazaar
Revision history for this message
Jonathan Lange (jml) wrote :

This sounds possible and probably desirable. We'd want to look into how it makes the new diff look.

Changed in launchpad-bazaar:
importance: Undecided → Low
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.