Hi Johann
On 01/08/12 15:26, Johann Messner wrote:
hello Noel,
[Cc to Philipp],
I tried to prepare a more "verbose" edition of the
patch set (and especially of the minor modifications
by myself) ... please see
https://linux.jku.at/messner/LibreOffice/
(-> patchInfo-31-Jul-2012__for_libreoffice.3.5.3-verbose.txt )
This file should be acceptable to the 'patch' command on a
box with the very same src tree (libreoffice 3.5.3). However,
since you're a developer you use a more advanced version of the libreoffice
source code. You will have to examine (and eventually adopt in a
cleaned up fashion) some of the code pieces very carefully..
we try to make the process for submitting patches really easy ( for both
submitters and developers/reviewers ) we don't require much. But really
a patch ( ontop of another patch ) and the whole lot based on top of an
old source version doesn't make things easy to review. Ok, granted this
is abit of a special case since some of the patches involved are not
committed, so... I have committed the remaining xml and preview related
fixes to master. Can you please checkout the current master, make your
changes and create your patches against that. That avoids messy time
consuming and error prone steps of trying to trace through a large patch
trying to filter special custom markup to detect the relevant changes.
It would really help us if you could do that, if you have problems
either creating the patches and/or getting/building the master branch
there is plenty of help at hand and we are ready and happy to help you.
regards,
Noel
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice