Re: Xml diff module - feedback needed

Florent André <[email protected]>
Newsgroups gmane.comp.cms.lenya.devel
Message-ID <[email protected]>
Thanks Richard, 

I understand the licence constraint and it's clearly a good think that sucj
a lib will be into trunk. 

"Problems" I have with this lib is that I don't find docs on witch kind of
diff do daisyDiff. Clearly it's work on text() modification, but my main
requirement is :
I have to distinguish new nodes and nodes that have move (from position 2
to 5 for example - idea of the xmlunit's similar function).

Have you informations about that ? If no, I will go to dress my swimsuit
and dive into daisyDiff lib ! :)

Tchuss
++

On Thu, 17 Dec 2009 22:05:03 -0600, Richard Frovarp
<[email protected]> wrote:
> On 12/17/2009 5:22 PM, florent andré wrote:
>> Hi Alexander,
>>
>> Did you had time to check it ?
>>
>> Yesterday, I see the xmlunit librairie [1]. This lib seems to have a
>> good api with myDiff.similar(), myDiff.identical(),DetailedDiff
class,...
>>
>> Did you have a look on it when you create the module ?
>> If yes, why do you choose the outerj.daisy.diff class ?
>>
>> Thanks for any answer,
>> Have a good day
>>
>> [1] : http://xmlunit.sourceforge.net
>> [2] : http://xmlunit.sourceforge.net/example.html
>>
> 
> I don't know if he saw that or not. However, taking a quick look over 
> it, it's more of a unit testing library. The similar() and identical() 
> functions simply return booleans. DetailedDiff can return a list, but I 
> don't necessarily see where it would return information as to where the 
> diffs are in relation to the document.
> 
> The biggest problem I see with the library however is its license. I 
> don't see anything identifying it as a certain type of license. He 
> appears to be using his own license that is most likely a variation of a 
> common one. Alexander did find a different diff module to use, however 
> that one was not license compatible with Apache. The one that he chose 
> is license compatible. License compatibility was something I required, 
> as I want to eventually work it into trunk.
> 
> Richard
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.