/b-gtk/fix-viz

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/b-gtk/fix-viz

« back to all changes in this revision

Viewing changes to commit.py

  • Committer: Curtis Hovey
  • Date: 2012-03-20 12:39:08 UTC
  • mfrom: (776.3.23 gpush-do-push)
  • Revision ID: sinzui.is@verizon.net-20120320123908-lclvaiasu0e50odg
Merged faster push.

Show diffs side-by-side

added added

removed removed

Lines of Context:
commit.py
22
22
from gi.repository import Pango
23
23
 
24
24
from bzrlib import (
 
25
    bencode,
25
26
    errors,
26
27
    osutils,
 
28
    revision as _mod_revision,
27
29
    trace,
 
30
    tsort,
28
31
    )
29
 
try:
30
 
    from bzrlib import bencode
31
 
except ImportError:
32
 
    from bzrlib.util import bencode
33
 
 
34
32
from bzrlib.plugins.gtk.dialog import question_dialog
35
33
from bzrlib.plugins.gtk.errors import show_bzr_error
36
34
from bzrlib.plugins.gtk.i18n import _i18n
 
35
from bzrlib.plugins.gtk.commitmsgs import SavedCommitMessagesManager
37
36
 
38
37
try:
39
38
    import dbus
43
42
    have_dbus = False
44
43
 
45
44
 
 
45
def _get_sorted_revisions(tip_revision, revision_ids, parent_map):
 
46
    """Get an iterator which will return the revisions in merge sorted order.
 
47
 
 
48
    This will build up a list of all nodes, such that only nodes in the list
 
49
    are referenced. It then uses MergeSorter to return them in 'merge-sorted'
 
50
    order.
 
51
 
 
52
    :param revision_ids: A set of revision_ids
 
53
    :param parent_map: The parent information for each node. Revisions which
 
54
        are considered ghosts should not be present in the map.
 
55
    :return: iterator from MergeSorter.iter_topo_order()
 
56
    """
 
57
    # MergeSorter requires that all nodes be present in the graph, so get rid
 
58
    # of any references pointing outside of this graph.
 
59
    parent_graph = {}
 
60
    for revision_id in revision_ids:
 
61
        if revision_id not in parent_map: # ghost
 
62
            parent_graph[revision_id] = []
 
63
        else:
 
64
            # Only include parents which are in this sub-graph
 
65
            parent_graph[revision_id] = [p for p in parent_map[revision_id]
 
66
                                            if p in revision_ids]
 
67
    sorter = tsort.MergeSorter(parent_graph, tip_revision)
 
68
    return sorter.iter_topo_order()
 
69
 
 
70
 
46
71
def pending_revisions(wt):
47
72
    """Return a list of pending merges or None if there are none of them.
48
73
 
53
78
    """
54
79
    parents = wt.get_parent_ids()
55
80
    if len(parents) < 2:
56
 
        return None
 
81
        return
57
82
 
58
83
    # The basic pending merge algorithm uses the same algorithm as
59
84
    # bzrlib.status.show_pending_merges
61
86
    branch = wt.branch
62
87
    last_revision = parents[0]
63
88
 
64
 
    if last_revision is not None:
65
 
        try:
66
 
            ignore = set(branch.repository.get_ancestry(last_revision,
67
 
                                                        topo_sorted=False))
68
 
        except errors.NoSuchRevision:
69
 
            # the last revision is a ghost : assume everything is new
70
 
            # except for it
71
 
            ignore = set([None, last_revision])
72
 
    else:
73
 
        ignore = set([None])
 
89
    graph = branch.repository.get_graph()
 
90
    other_revisions = [last_revision]
74
91
 
75
92
    pm = []
76
93
    for merge in pending:
77
 
        ignore.add(merge)
78
 
        try:
79
 
            rev = branch.repository.get_revision(merge)
80
 
            children = []
81
 
            pm.append((rev, children))
82
 
 
83
 
            # This does need to be topo sorted, so we search backwards
84
 
            inner_merges = branch.repository.get_ancestry(merge)
85
 
            assert inner_merges[0] is None
86
 
            inner_merges.pop(0)
87
 
            for mmerge in reversed(inner_merges):
88
 
                if mmerge in ignore:
89
 
                    continue
90
 
                rev = branch.repository.get_revision(mmerge)
91
 
                children.append(rev)
92
 
 
93
 
                ignore.add(mmerge)
94
 
        except errors.NoSuchRevision:
95
 
            print "DEBUG: NoSuchRevision:", merge
96
 
 
97
 
    return pm
 
94
        try:
 
95
            merge_rev = branch.repository.get_revision(merge)
 
96
        except errors.NoSuchRevision:
 
97
            # If we are missing a revision, just print out the revision id
 
98
            trace.mutter("ghost: %r", merge)
 
99
            other_revisions.append(merge)
 
100
            continue
 
101
 
 
102
        # Find all of the revisions in the merge source, which are not in the
 
103
        # last committed revision.
 
104
        merge_extra = graph.find_unique_ancestors(merge, other_revisions)
 
105
        other_revisions.append(merge)
 
106
        merge_extra.discard(_mod_revision.NULL_REVISION)
 
107
 
 
108
        # Get a handle to all of the revisions we will need
 
109
        try:
 
110
            revisions = dict((rev.revision_id, rev) for rev in
 
111
                             branch.repository.get_revisions(merge_extra))
 
112
        except errors.NoSuchRevision:
 
113
            # One of the sub nodes is a ghost, check each one
 
114
            revisions = {}
 
115
            for revision_id in merge_extra:
 
116
                try:
 
117
                    rev = branch.repository.get_revisions([revision_id])[0]
 
118
                except errors.NoSuchRevision:
 
119
                    revisions[revision_id] = None
 
120
                else:
 
121
                    revisions[revision_id] = rev
 
122
 
 
123
         # Display the revisions brought in by this merge.
 
124
        rev_id_iterator = _get_sorted_revisions(merge, merge_extra,
 
125
                            branch.repository.get_parent_map(merge_extra))
 
126
        # Skip the first node
 
127
        num, first, depth, eom = rev_id_iterator.next()
 
128
        if first != merge:
 
129
            raise AssertionError('Somehow we misunderstood how'
 
130
                ' iter_topo_order works %s != %s' % (first, merge))
 
131
        children = []
 
132
        for num, sub_merge, depth, eom in rev_id_iterator:
 
133
            rev = revisions[sub_merge]
 
134
            if rev is None:
 
135
                trace.warning("ghost: %r", sub_merge)
 
136
                continue
 
137
            children.append(rev)
 
138
        yield (merge_rev, children)
98
139
 
99
140
 
100
141
_newline_variants_re = re.compile(r'\r\n?')
101
142
def _sanitize_and_decode_message(utf8_message):
102
143
    """Turn a utf-8 message into a sanitized Unicode message."""
103
144
    fixed_newline = _newline_variants_re.sub('\n', utf8_message)
104
 
    return fixed_newline.decode('utf-8')
 
145
    return osutils.safe_unicode(fixed_newline)
105
146
 
106
147
 
107
148
class CommitDialog(Gtk.Dialog):
122
163
        self._enable_per_file_commits = True
123
164
        self._commit_all_changes = True
124
165
        self.committed_revision_id = None # Nothing has been committed yet
 
166
        self._last_selected_file = None
125
167
        self._saved_commit_messages_manager = SavedCommitMessagesManager(
126
168
            self._wt, self._wt.branch)
127
169
 
133
175
        """Setup the member variables for state."""
134
176
        self._basis_tree = self._wt.basis_tree()
135
177
        self._delta = None
136
 
        self._pending = pending_revisions(self._wt)
 
178
        self._wt.lock_read()
 
179
        try:
 
180
            self._pending = list(pending_revisions(self._wt))
 
181
        finally:
 
182
            self._wt.unlock()
137
183
 
138
184
        self._is_checkout = (self._wt.branch.get_bound_location() is not None)
139
185
 
191
237
 
192
238
        all_enabled = (self._selected is None)
193
239
        # The first entry is always the 'whole tree'
194
 
        all_iter = store.append([None, None, all_enabled, 'All Files', '', ''])
 
240
        all_iter = store.append(["", "", all_enabled, 'All Files', '', ''])
195
241
        initial_cursor = store.get_path(all_iter)
196
242
        # should we pass specific_files?
197
243
        self._wt.lock_read()
242
288
            self._check_local.hide()
243
289
            return
244
290
        if have_dbus:
245
 
            bus = dbus.SystemBus()
 
291
            try:
 
292
                bus = dbus.SystemBus()
 
293
            except dbus.DBusException:
 
294
                trace.mutter("DBus system bus not available")
 
295
                self._check_local.show()
 
296
                return
246
297
            try:
247
298
                proxy_obj = bus.get_object('org.freedesktop.NetworkManager',
248
299
                                           '/org/freedesktop/NetworkManager')
250
301
                trace.mutter("networkmanager not available.")
251
302
                self._check_local.show()
252
303
                return
253
 
            
 
304
 
254
305
            dbus_iface = dbus.Interface(proxy_obj,
255
306
                                        'org.freedesktop.NetworkManager')
256
307
            try:
282
333
        """Build up the dialog widgets."""
283
334
        # The primary pane which splits it into left and right (adjustable)
284
335
        # sections.
285
 
        self._hpane = Gtk.HPaned()
 
336
        self._hpane = Gtk.Paned.new(Gtk.Orientation.HORIZONTAL)
286
337
 
287
338
        self._construct_left_pane()
288
339
        self._construct_right_pane()
448
499
                                     self._on_treeview_files_cursor_changed)
449
500
 
450
501
    def _toggle_commit(self, cell, path, model):
451
 
        if model[path][0] is None: # No file_id means 'All Files'
 
502
        if model[path][0] == "": # No file_id means 'All Files'
452
503
            new_val = not model[path][2]
453
504
            for node in model:
454
505
                node[2] = new_val
571
622
 
572
623
    def _on_treeview_files_cursor_changed(self, treeview):
573
624
        treeselection = treeview.get_selection()
 
625
        if treeselection is None:
 
626
            # The treeview was probably destroyed as the dialog closes.
 
627
            return
574
628
        (model, selection) = treeselection.get_selected()
575
629
 
576
630
        if selection is not None:
577
631
            path, display_path = model.get(selection, 1, 3)
578
632
            self._diff_label.set_text(_i18n('Diff for ') + display_path)
579
 
            if path is None:
 
633
            if path == "":
580
634
                self._diff_view.show_diff(None)
581
635
            else:
582
 
                self._diff_view.show_diff([path.decode('UTF-8')])
 
636
                self._diff_view.show_diff([osutils.safe_unicode(path)])
583
637
            self._update_per_file_info(selection)
584
638
 
585
639
    def _on_accel_next(self, accel_group, window, keyval, modifier):
597
651
            # selected. Either way, select All Files, and jump to the global
598
652
            # commit message.
599
653
            self._treeview_files.set_cursor(
600
 
                Gtk.TreePath(path=0), None, False)
 
654
                Gtk.TreePath(path=0), "", False)
601
655
            self._global_message_text_view.grab_focus()
602
656
        else:
603
657
            # Set the cursor to this entry, and jump to the per-file commit
622
676
        self._save_current_file_message()
623
677
        text_buffer = self._file_message_text_view.get_buffer()
624
678
        file_id, display_path, message = self._files_store.get(selection, 0, 3, 5)
625
 
        if file_id is None: # Whole tree
 
679
        if file_id == "": # Whole tree
626
680
            self._file_message_expander.set_label(_i18n('File commit message'))
627
681
            self._file_message_expander.set_expanded(False)
628
682
            self._file_message_expander.set_sensitive(False)
645
699
        files = []
646
700
        records = iter(self._files_store)
647
701
        rec = records.next() # Skip the All Files record
648
 
        assert rec[0] is None, "Are we skipping the wrong record?"
 
702
        assert rec[0] == "", "Are we skipping the wrong record?"
649
703
 
650
704
        file_info = []
651
705
        for record in records:
652
706
            if self._commit_all_changes or record[2]:# [2] checkbox
653
 
                file_id = record[0] # [0] file_id
654
 
                path = record[1]    # [1] real path
 
707
                file_id = osutils.safe_utf8(record[0]) # [0] file_id
 
708
                path = osutils.safe_utf8(record[1])    # [1] real path
655
709
                # [5] commit message
656
710
                file_message = _sanitize_and_decode_message(record[5])
657
711
                files.append(path.decode('UTF-8'))
793
847
        rev_dict['revision_id'] = rev.revision_id
794
848
        return rev_dict
795
849
 
796
 
 
797
 
class SavedCommitMessagesManager:
798
 
    """Save glogal and per-file commit messages.
799
 
 
800
 
    Saves global commit message and utf-8 file_id->message dictionary
801
 
    of per-file commit messages on disk. Re-reads them later for re-using.
802
 
    """
803
 
 
804
 
    def __init__(self, tree=None, branch=None):
805
 
        """If branch is None, builds empty messages, otherwise reads them
806
 
        from branch's disk storage. 'tree' argument is for the future."""
807
 
        if branch is None:
808
 
            self.global_message = u''
809
 
            self.file_messages = {}
810
 
        else:
811
 
            config = branch.get_config()
812
 
            self.global_message = config.get_user_option(
813
 
                'gtk_global_commit_message')
814
 
            if self.global_message is None:
815
 
                self.global_message = u''
816
 
            file_messages = config.get_user_option('gtk_file_commit_messages')
817
 
            if file_messages: # unicode and B-encoded:
818
 
                self.file_messages = bencode.bdecode(
819
 
                    file_messages.encode('UTF-8'))
820
 
            else:
821
 
                self.file_messages = {}
822
 
 
823
 
    def get(self):
824
 
        return self.global_message, self.file_messages
825
 
 
826
 
    def is_not_empty(self):
827
 
        return bool(self.global_message or self.file_messages)
828
 
 
829
 
    def insert(self, global_message, file_info):
830
 
        """Formats per-file commit messages (list of dictionaries, one per file)
831
 
        into one utf-8 file_id->message dictionary and merges this with
832
 
        previously existing dictionary. Merges global commit message too."""
833
 
        file_messages = {}
834
 
        for fi in file_info:
835
 
            file_message = fi['message']
836
 
            if file_message:
837
 
                file_messages[fi['file_id']] = file_message # utf-8 strings
838
 
        for k,v in file_messages.iteritems():
839
 
            try:
840
 
                self.file_messages[k] = v + '\n******\n' + self.file_messages[k]
841
 
            except KeyError:
842
 
                self.file_messages[k] = v
843
 
        if self.global_message:
844
 
            self.global_message = global_message + '\n******\n' \
845
 
                + self.global_message
846
 
        else:
847
 
            self.global_message = global_message
848
 
 
849
 
    def save(self, tree, branch):
850
 
        # We store in branch's config, which can be a problem if two gcommit
851
 
        # are done in two checkouts of one single branch (comments overwrite
852
 
        # each other). Ideally should be in working tree. But uncommit does
853
 
        # not always have a working tree, though it always has a branch.
854
 
        # 'tree' argument is for the future
855
 
        config = branch.get_config()
856
 
        # should it be named "gtk_" or some more neutral name ("gui_" ?) to
857
 
        # be compatible with qbzr in the future?
858
 
        config.set_user_option('gtk_global_commit_message', self.global_message)
859
 
        # bencode() does not know unicode objects but set_user_option()
860
 
        # requires one:
861
 
        config.set_user_option(
862
 
            'gtk_file_commit_messages',
863
 
            bencode.bencode(self.file_messages).decode('UTF-8'))
864
 
 
865
 
 
866
 
def save_commit_messages(local, master, old_revno, old_revid,
867
 
                         new_revno, new_revid):
868
 
    b = local
869
 
    if b is None:
870
 
        b = master
871
 
    mgr = SavedCommitMessagesManager(None, b)
872
 
    revid_iterator = b.repository.iter_reverse_revision_history(old_revid)
873
 
    cur_revno = old_revno
874
 
    new_revision_id = old_revid
875
 
    graph = b.repository.get_graph()
876
 
    for rev_id in revid_iterator:
877
 
        if cur_revno == new_revno:
878
 
            break
879
 
        cur_revno -= 1
880
 
        rev = b.repository.get_revision(rev_id)
881
 
        file_info = rev.properties.get('file-info', None)
882
 
        if file_info is None:
883
 
            file_info = {}
884
 
        else:
885
 
            file_info = bencode.bdecode(file_info.encode('UTF-8'))
886
 
        global_message = osutils.safe_unicode(rev.message)
887
 
        # Concatenate comment of the uncommitted revision
888
 
        mgr.insert(global_message, file_info)
889
 
 
890
 
        parents = graph.get_parent_map([rev_id]).get(rev_id, None)
891
 
        if not parents:
892
 
            continue
893
 
    mgr.save(None, b)