On 23 May 2010 05:35, Philip Olson <phi...@roshambo.org> wrote:
>
> On May 21, 2010, at 1:12 PM, David Tajchreber wrote:
>
>> On Fri, 21 May 2010 11:23:06 +0100, Richard Quadling wrote:
>>
>>> On 21 May 2010 11:15, Richard Quadling <rquadl...@googlemail.com> wrote:
>>>> On 21 May 2010 10:08, Philip Olson <phi...@roshambo.org> wrote:
>>>>>>
>>>>>> The problem is not the tags per se. I can and have successfully
>>>>>> compiled all the manuals I've altered (en, es, fa, fr, ja, tr) from
>>>>>> the XML through to a Windows CHM (I'm on Windows after all).
>>>>>>
>>>>>> I'm not getting the warnings that you are seeing.
>>>>>
>>>>> For example, add <para>FOO</para> to strlen.xml outside of a refsect1
>>>>> and you'll see the datetime xcinlude errors along with the real error.
>>>>> It's confusing, and annoying, but I'm unsure how to solve said
>>>>> annoyance.
>>>>>
>>>>> Regards,
>>>>> Philip
>>>>>
>>>> configure.php already suppresses xinclude errors when the errors are
>>>> being reported ( see print_xml_errors() ).
>>>>
>>>> It would seem that we also need to include the ability to suppress
>>>> xpointer errors.
>>>>
>>>> But sometimes the errors may be needed, so I think the option
>>>> --enable-xml-details should show the xinclude and the xpointer errors.
>>>>
>>>> Using ...
>>>>
>>>> if ($details || (!strpos($err->message, 'xi:include') &&
>>>> !strpos($err->message, 'element include') && False ===
>>>> strpos($err->message, 'XPointer evaluation failed:'))) {
>>>>
>>>> will now also suppress the XPointer errors, but show all the errors if
>>>> the --enable-xml-details option is enabled.
>>>>
>>>> Enough do you think?
>>>>
>>>>
>>> Nope. Seems that there are a LOT of errors which we suppress. Maybe
>>> something to work on there.
>>>
>>> So, going to just suppress the XPointer evaluation errors along with the
>>> other errors.
>>
>> The problem is definitely the xi:include tags. They reference sections
>> that don't exist, notably destructorsynopsis for the datetime classes.
>> Your patch to standardize the xi:include tags in the datetime files,
>> r299458, introduced the bug. configure.php hides the warnings by default
>> and that's why you're able to build all the manuals without seeing any of
>> the warnings (the xi:fallback tag is a recovery for the missing resources
>> but a warning is still generated). When a non-recoverable error occurs,
>> configure.php iterates over all the warnings/errors, printing them out as
>> it goes, which explains why the datetime xi:include warnings only show up
>> when there is another error in the documentation tree. If you want to
>> generate them, something as simple as what's below will do it.
>>
>> $doc = new DOMDocument();
>> $doc->load('doc-base/manual.xml', LIBXML_NOENT);
>> $doc->xinclude();
>> var_dump($doc->validate());
>
> The datetime documentation has essentially become (or is becoming) our model 
> for future (or ideal) documentation, at least when it comes to writing 
> DocBook XML for an extension that has both an OOP and Procedural API.
>
> That said, the introduced xi:include tags sound a bit broken, so should be 
> fixed or reverted. I'm hopeful some DocBook geeks here will find the best 
> answer, but in the meantime it sounds like reverting this is the correct 
> solution.
>
> Regards,
> Philip
>
>
>

I can't get the errors locally, Considering we already throw away a
LOT of errors regarding xinclude when things go wrong, I can only
suggest excluding the warnings from the xinclude() call in
configure.php.

If someone can replicate the warnings locally (i.e
DOMDocument::xinclude() warning on line 615), then try ...

$dom->xinclude(LIBXML_NOWARNING);

as line 615 ($Id: configure.php 299572 2010-05-21 11:04:34Z rquadling $)

The <xinclude> tags are not "broken". They correctly include what is
required/available. The issue is the warning which I am hoping will be
suppressed by the use of the above code.



-- 
-----
Richard Quadling
"Standing on the shoulders of some very clever giants!"
EE : http://www.experts-exchange.com/M_248814.html
EE4Free : http://www.experts-exchange.com/becomeAnExpert.jsp
Zend Certified Engineer : http://zend.com/zce.php?c=ZEND002498&r=213474731
ZOPA : http://uk.zopa.com/member/RQuadling

Reply via email to