Pdf files content included on merge proposal diff.

Bug #540400 reported by Ronaldo Maia
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Bazaar
New
Undecided
Unassigned

Bug Description

The following merge proposal shows a diff with the content of a whole pdf file that was changed.

https://code.edge.launchpad.net/~georgeyk/stoqlib/production-order-report/+merge/21166

It would be interesting to only indicate that a binary file was changed (I am not sure if this happens to other types of binary files)

affects: launchpad → launchpad-code
Revision history for this message
Aaron Bentley (abentley) wrote :

We generate our diffs with bzr

affects: launchpad-code → bzr
Revision history for this message
Tim Penhey (thumper) wrote :

The way that bzr determines a file is binary is a bit crude. In that it checks for a null byte in the first 1024 characters of the file.

Your pdf file probably doesn't have one.

Perhaps bzrlib could also use the guessed mimetype of the file to help determine if the file is text or not.

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.