/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-11 18:24:06 UTC
  • mto: This revision was merged to the branch mainline in revision 785.
  • Revision ID: sinzui.is@verizon.net-20120311182406-f4jqpff65b30g3al
Create the mnemonic for the button label.

Show diffs side-by-side

added added

removed removed

Lines of Context:
commit.py
21
21
from gi.repository import GObject
22
22
from gi.repository import Pango
23
23
 
24
 
try:
25
 
    from bzrlib import bencode
26
 
except ImportError:
27
 
    from bzrlib.util import bencode
28
 
 
29
24
from bzrlib import (
 
25
    bencode,
30
26
    errors,
 
27
    osutils,
 
28
    revision as _mod_revision,
31
29
    trace,
 
30
    tsort,
32
31
    )
33
32
from bzrlib.plugins.gtk.dialog import question_dialog
34
33
from bzrlib.plugins.gtk.errors import show_bzr_error
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'))