incubator-odf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Svante Schubert <>
Subject Re: Code freeze at the end of this week.
Date Fri, 07 Dec 2012 14:30:38 GMT
Hi Dali,

Great work! I am very thankful for your help!
I have reviewed the patch, after applying it via my Netbeans IDE.
Fixed some typos (of mine) in the JavaDoc, only the binary ODF test file
did not work with the DIFF patch, it had 0 bytes, so I used the original
In the test file (test_rdfmeta.odt) the manifest file is not the first
in the test document - common problem of hand edited and zipped files -
but when reopening with any OpenOffice descendant the embedded file is
gone, therefore the validation error remains, but does not break a test.

Most annoying one RDF test (RDFMetadataTest) had different test results
(with JDK7 & 6), although from my believe the new results are the
correct one.
The problem is that RDF triples with different subjects (XML files) had
been regarded as duplicates, but are no duplicates to me. Therefore I
simply commented the problematic part of the test out, but kept the
comments in case I have overseen something.
During this debugging/research, I have as well updated the RDF libraries
in the pom.xml to the latest version.

After that, I realized that my odf2rdf.xsl had no copyright header and
added it - although there is no license test beyond the resource
directory anyway.

Finally, after some weeks after my last review, I realized that the GSoC
student had used my XSLT file not only for testing as I had desired it,
but as well from the extraction of incontent metadata.
I limited the damage, by moving the previous XSLT helper class
OdfXMLHelper from dom to pkg layer and making it only package visible.
Moving some functionality of OdfSchemaDocument to OdfPackageDocument,
see new function
    public Model getXMLFileMetadata(String internalPath) {
which is now being called from OdfSchemaDocument.
And reused the parsed entity of the RDF XSLT file (Java Template) class,
to reduce the overhead.

The repository now builds fine with "mvn clean install -Ppedeantic",
which will include the Apache license header test (rat).
I made the above test under Linux and Windows7 and pushed the changes,
before some other changes might again result into merge problems ;)

PS: BTW avoided any default Java indentation given by Netbeans, perhaps
we can agree on one indentation given by XML configuration in the project?
PPS: I have removed the *.original~ files..

Thanks again for your help, Dali.

On 07.12.2012 07:41, Dali Liu wrote:
> Hi Svante,
> I merger all the codes in it include the indent of codes.
> The test case run successfully but I am not sure whether the RDF metadata
> function works fine. Please have a verify on this patch, if it works, I can
> commit it to trunk.
> 2012/12/7 Rob Weir <>
>> On Thu, Dec 6, 2012 at 3:17 AM, Dali Liu <> wrote:
>>> Hi Svante,
>>> Don't worry about this, I would like to provide some help on this. If the
>>> time is not enough, I think that we can delay the code freeze date to
>> next
>>> week.
>> Is Devin able to look at the password encrypted file integration?  He
>> had an old patch, but I was not able to merge it successfully.    But
>> the longer we go without merging, the harder it will be.
>> -Rob
>>> 2012/12/5 Svante Schubert <>
>>>> Hi,
>>>> I got problems to integrate the RDF support for ODF Toolkit, perhaps
>>>> someone could give a helping hand as I ran out of time.
>>>> In theory all patches of the issue can be found at
>>>> but they have already been accumulated on github for a better review
>>>> handling to the following patch
>>>> You may oversee the overall history
>>>> and compare it with the main trunk
>>>> Doing so you may notice that on the last day of the patch of the RDF
>>>> repository, the encryption patch was revoked and some other had to be
>>>> reapplied, which seem to cause trouble for merging.
>>>> In addition there are problems with line endings, as we switched all
>>>> line endings to Unix as we worked with Windows and Linux VM on the same
>>>> source tree on a shared directory making code comparison more difficult.
>>>> (Can we do this in general for ODFDOM in the future?)
>>>> And finally, sometimes the student seem to have a different indent as
>>>> the usual, showing up changes, where no semantic was changed.
>>>>  (Arn't there tools in Maven to check for a uniform indent?)
>>>> All together, we could name it a real nightmare before Christmas!
>>>> On the other hand the RDF support for the ODF Toolkit is a huge feature
>>>> for this project and would provide additional marketing effect to the
>>>> next version.
>>>> Therefore I still hope that someone could push this over the line.
>>>> Thanks in advance,
>>>> Svante
>>>> On 05.12.2012 04:10, Dali Liu wrote:
>>>>> Dear all,
>>>>> We are planning to release a new version of our ODF toolkit.
>>>>> From previous discuss we choose this weekend(12/07) as the codes
>> freeze
>>>>> date.
>>>>> Please try to commit your patches to SVN before the date.
>>>>> If you are working on something and want to deliver it in this new
>>>> version
>>>>> please speak out, Let's choose another codes freeze date.

View raw message