qdiff should have revert options

Bug #269458 reported by Gary van der Merwe
6
Affects Status Importance Assigned to Milestone
QBzr
Confirmed
Medium
Unassigned

Bug Description

It would be nice if:

qdiff had the following options:

* Revert all files in the diff.
* Revert a file in the diff.
* Revert a chunk.

These options will only be available if one of the trees is a working tree.
The first two will be easy. Just run bzr revert in a SubProcessWindow.
The 3rd is scary because we will have to write to the file our selfs. Then we have to make sure we handle encoding and line endings correctly. Is there maybe some code in bzrlib that will help us do this?

Tags: feature qdiff
Changed in qbzr:
assignee: nobody → garyvdm
importance: Undecided → Wishlist
Revision history for this message
Alexander Belchenko (bialix) wrote :

it seems with recent support of external diff viewers this bug is less actual now?

Changed in qbzr:
status: New → Confirmed
Revision history for this message
Craig Hewetson (craighewetson-deactivatedaccount) wrote :

Maybe we can consider this as part of the "qdiff improvements"? The "revert chunk" feature would be especially useful.

tags: added: qdiff
Revision history for this message
Greg (gregspecialsource) wrote :

Just a comment...

This is very much how I work.
1) Diff to verify changes
2) While verifying changes, need to revert some of them.

External diff viewers (editors) do not currently work well because of the way bzr launches them. They effectively have read but not write access to the working tree.

Changed in qbzr:
importance: Wishlist → High
importance: High → Medium
tags: added: feature-request
tags: added: feature
removed: feature-request
Revision history for this message
Gary van der Merwe (garyvdm) wrote :

Similar to bug 148026

Changed in qbzr:
assignee: Gary van der Merwe (garyvdm) → nobody
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.