All of lore.kernel.org
 help / color / mirror / Atom feed
From: david@lang.hm
To: saurabh gupta <saurabhgupta1403@gmail.com>
Cc: Johannes Schindelin <Johannes.Schindelin@gmx.de>, git@vger.kernel.org
Subject: Re: Google Summer of Code 2009: GIT
Date: Thu, 12 Mar 2009 11:03:47 -0700 (PDT)	[thread overview]
Message-ID: <alpine.DEB.1.10.0903121100360.16753@asgard.lang.hm> (raw)
In-Reply-To: <ab9fa62a0903120542s45b1ceebwddab932891c47cf0@mail.gmail.com>

On Thu, 12 Mar 2009, saurabh gupta wrote:

> hello,
>
> On Thu, Mar 12, 2009 at 1:51 AM,  <david@lang.hm> wrote:
>>
>>> Yes, but the thing is that the underlying codes and method will be
>>> different for GUI part and terminal part to make it readable and
>>> understandable. Like for OO Documents, if we aim to show the *diff*
>>> output in the Office tool, then we have to change the xml file
>>> accordingly. But the same xml file when used with terminal only, the
>>> *diff* output is not clear.
>>>
>>> As Johannes said in above post that for OO documents, while showing
>>> the *diff* result, no xml data should be shown.
>>
>> in part we are talking about different aspects of things, and we were all
>> wrong.
>>
>> see the e-mail a little bit ago by Junio
>>
>> there are two types of helpers that can be written
>>
>> 1. a low-level part that does the simple merges automaticaly and leaves
>> behind appropriate conflict markers when it can't
>>
>> there is no GUI involved with this.
>>
>> what 'appropriate conflict markers' are can vary from XML file to XML file
>>
>>
>> 2. after a conflict has taken place, a helper to work with the user to
>> resolve the conflict
>>
>> this can have a GUI and/or a text UI and is tied to the 'appropriate
>> conflict markers' as defined in #1, and can be _very_ tightly coupled to the
>> specific use of the XML file.
>>
>> I think it's very important to have a text UI tool that can be used for the
>> conflict resolution step as well as supporting GUI tools.
>
> All right. What I can understand from the current situation is that
> for merging and marking conflicts in xml (for example) files has
> following things to do.
>
> One, if the markers are put in the xml files like that of a text file,
> one can see the difference using a text editor or a terminal. But if
> the same xml file is to be opened in another editor which expects a
> valid xml (as clearly mentioned on the wiki ideas for GIT), then a
> merge helper is needed.
>
> But if the conflict markers are put in a way to make the xml file
> still valid which can be then opened in the appropriate editor, then
> the marking will be different. The merge driver has to produce the
> conflicted merged file in a manner which is still a valid xml file and
> user has the choice to open it in his own editor to resolve the
> conflicts.

exactly. and how you mark the conflict to have it be valid XML is going to 
depend on details of the type of file. there are probably a few basic 
methods that will work the vast majority of the time, but with some 
details needing to be configurable.

for example, if the XML document is a ODF document, it may be possible to 
add 'revision' tags around the conflict that are already understood by the 
editor.

David Lang

  reply	other threads:[~2009-03-12 18:05 UTC|newest]

Thread overview: 76+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-11  4:52 Google Summer of Code 2009: GIT Saurabh Gupta
2009-03-11  5:56 ` Daniel Barkalow
     [not found]   ` <ab9fa62a0903102317l3a7322f7w5e4d9ba0e02af37b@mail.gmail.com>
2009-03-11  9:04     ` saurabh gupta
2009-03-11  8:59 ` David Symonds
2009-03-11  9:02   ` saurabh gupta
2009-03-11 11:55   ` Johannes Schindelin
2009-03-11 11:55     ` David Symonds
2009-03-11 12:52       ` Johannes Schindelin
2009-03-11 11:58 ` Johannes Schindelin
2009-03-11 12:11   ` saurabh gupta
2009-03-11 12:58     ` Johannes Schindelin
2009-03-11 13:55       ` saurabh gupta
2009-03-11 14:02         ` Johannes Schindelin
2009-03-11 14:13           ` saurabh gupta
2009-03-11 15:27             ` Rogan Dawes
2009-03-11 16:21               ` saurabh gupta
2009-03-11 15:38             ` Johannes Schindelin
2009-03-11 16:29               ` saurabh gupta
2009-03-11 16:29             ` Daniel Barkalow
2009-03-11 16:44               ` Johannes Schindelin
2009-03-12 12:56                 ` Michael J Gruber
2009-03-12 13:07                   ` Johannes Schindelin
2009-03-12 13:15                     ` Michael J Gruber
2009-03-12 18:25                       ` david
2009-03-11 16:58               ` saurabh gupta
2009-03-12 12:47                 ` Michael J Gruber
2009-03-11 16:32           ` david
2009-03-11 17:01             ` Johannes Schindelin
2009-03-11 19:30               ` david
2009-03-11 19:55                 ` Johannes Schindelin
2009-03-11 17:07             ` saurabh gupta
2009-03-11 19:29               ` david
2009-03-11 20:02                 ` saurabh gupta
2009-03-11 20:21                   ` david
2009-03-11 20:37                     ` Johannes Schindelin
2009-03-11 21:05                       ` david
2009-03-11 21:47                         ` Junio C Hamano
2009-03-12  1:57                           ` thestar
2009-03-12  7:40                             ` Junio C Hamano
2009-03-12 12:45                           ` saurabh gupta
2009-03-12 18:00                             ` david
2009-03-12 18:43                               ` Junio C Hamano
     [not found]                               ` <ab9fa62a0903121119j6c2a1d43kd9cda99db47b5e7c@mail.gmail.com>
2009-03-12 18:53                                 ` david
2009-03-12 19:00                                   ` saurabh gupta
2009-03-12 19:29                                     ` david
2009-03-12 19:45                                       ` saurabh gupta
2009-03-12 19:59                                         ` david
2009-03-12 20:03                                           ` saurabh gupta
2009-03-12 20:45                                             ` david
2009-03-13  3:15                                               ` saurabh gupta
2009-03-18 23:16                                             ` Johannes Schindelin
2009-03-18 23:55                                               ` david
2009-03-19  0:43                                                 ` Johannes Schindelin
2009-03-19  8:37                                                   ` david
2009-03-19 10:24                                                     ` Johannes Schindelin
2009-03-19 19:12                                                       ` david
2009-03-19 19:23                                                   ` saurabh gupta
2009-03-19  6:30                                               ` Caleb Cushing
2009-03-19 10:19                                                 ` Johannes Schindelin
2009-03-21 23:53                                                   ` Caleb Cushing
2009-03-19 19:17                                               ` saurabh gupta
2009-03-19 23:42                                                 ` Johannes Schindelin
2009-03-20  0:07                                                   ` david
2009-03-20  0:30                                                     ` Johannes Schindelin
2009-03-20  3:09                                                       ` david
2009-03-20  9:35                                                         ` Johannes Schindelin
2009-03-20 20:50                                                           ` david
2009-03-21 17:38                                                             ` saurabh gupta
     [not found]                                                   ` <ab9fa62a0903211031l78c7afadg9409a544f2bda7db@mail.gmail.com>
2009-03-21 17:36                                                     ` saurabh gupta
2009-03-12 20:18                                       ` Junio C Hamano
2009-03-12 12:42                     ` saurabh gupta
2009-03-12 18:03                       ` david [this message]
2009-03-12 18:23                         ` saurabh gupta
2009-03-13  9:41                           ` Rogan Dawes
2009-03-13 20:18                             ` saurabh gupta
2009-03-11 19:36 ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alpine.DEB.1.10.0903121100360.16753@asgard.lang.hm \
    --to=david@lang.hm \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=git@vger.kernel.org \
    --cc=saurabhgupta1403@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.