[Laszlo-dev] Trunk doc build broken [Was: FAILURE for 14837-openlaszlo-trunk on linux-builder.corp.laszlosystems.com]

[Max Carlson]

For the record: http://jira.openlaszlo.org/jira/browse/LPP-8518

P T Withington wrote:
> Sure, but I don't know who's going to fix it.  But I think it is a bug 
> in the xsl processor -- it issues a warning and then generates nothing.  
> It really ought to be a fatal error, I would think.
> 
> On 2009-09-25, at 23:33, Max Carlson wrote:
> 
>> Should we file a bug against the doc toolchain?
>>
>> P T Withington wrote:
>>> Ah.  I've run into things like this before.  I suspect lossage on the 
>>> part of the tool, but you can work around it with the appropriate 
>>> character entity (I think °).
>>> On 2009-09-25, at 10:33, André Bargull wrote:
>>>> The XSL processor doesn't like °
>>>>
>>>>
>>>> On 9/25/2009 4:29 PM, André Bargull wrote:
>>>>> This seems to be the root cause for the failure:
>>>>>
>>>>>> lzx2js2doc.transform:
>>>>>>   [style] Processing 
>>>>>> \src\svn\openlaszlo\trunk\lps\components\doctest-library.lzx to 
>>>>>> \src\svn\openlaszlo\trunk\docs\src\build\reference\components-raw.xml
>>>>>>   [style] Loading stylesheet 
>>>>>> \src\svn\openlaszlo\trunk\docs\src\xsl\lzx2js2doc.xsl
>>>>>>   [style] 
>>>>>> /src/svn/openlaszlo/trunk/docs/src/xsl/lzx2js2doc.xsl:77:60: 
>>>>>> Warning! Angeforderte Do
>>>>>> kumentation kann nicht geladen werden: Invalid byte 1 of 1-byte 
>>>>>> UTF-8 sequence.
>>>>>
>>>>>
>>>>> To shorten the reference build, I've changed 
>>>>> "components/doctest-library.lzx" to only include drawview and then 
>>>>> called 'ant -DQUICK=true components.js2doc.generate'.
>>>>>
>>>>>
>>>>> On 9/25/2009 3:20 PM, P T Withington wrote:
>>>>>> Henry has an option where in docs/src you can say `ant 
>>>>>> -DQUICK=true reference` which makes the doc build only look at 
>>>>>> lfc/doctestLaszloLibrary and components/doctest-library.  Make 
>>>>>> sure the files you are trying to debug are listed in those test 
>>>>>> libraries.  It makes the doc build take less than 3 hours 
>>>>>> (hopefully on the order of 10 minutes or so).
>>>>>>
>>>>>> Beyond that, I have no hints other than digging in.
>>>>>>
>>>>>> The doc system is simpler than it used to be, but still not too 
>>>>>> simple.  <sigh />
>>>>>>
>>>>>> There is usually a hint in the output of the ant file, the problem 
>>>>>> is, it is not always a fatal error.  Sometimes it will output 
>>>>>> something that seems innocuous, but that in actuality is screwing 
>>>>>> the whole thing.  Try inspecting the output of the above command 
>>>>>> and look for anything at all suspicious.
>>>>>>
>>>>>> Oh, and, I guess we should have a rule:  If you are adding new 
>>>>>> code, or editing the doc, please make sure the doc build will pass 
>>>>>> before you check in!  (It should be sufficient to check that your 
>>>>>> stuff passes the QUICK build, if you want to take a shortcut; but 
>>>>>> you are on the hook to fix it if it breaks the build...)
>>>>>>
>>>>>> On 2009-09-25, at 09:03, Max Carlson wrote:
>>>>>>
>>>>>>> Hmm - I've fiddled with this quite a bit and the only thing that 
>>>>>>> makes the doc build successfully is removing the two new 
>>>>>>> functions __roundrectFactors() and __getRoundRectFactors().  I 
>>>>>>> tried adding them t the interface, tried removing their docs and 
>>>>>>> nothing works.
>>>>>>>
>>>>>>> How are we supposed to debug doc build issues like this?  I don't 
>>>>>>> see any errors, but lz.drawview.html fails to build - it just 
>>>>>>> isn't there.
>>>>>>>
>>>>>>> I'm about to go to bed, so I'm going to hand this off to André...
>>>>>>>
>>>>>>> Max Carlson wrote:
>>>>>>>> r14823 worked fine.  I'm 95% sure this was caused by r14824 - 
>>>>>>>> André's change to drawview.  I'm guessing this this line is 
>>>>>>>> confusing the doc system:
>>>>>>>> +        // @devnote mixins can't have static members (at least 
>>>>>>>> in AS3),
>>>>>>>> Testing now...
>>>>>>>> P T Withington wrote:
>>>>>>>>> The trunk nightly is broken:
>>>>>>>>>
>>>>>>>>>   [exec] Error: broken link in doc navigation: lz.drawview.html 
>>>>>>>>> referenced from toc.xml does not exist
>>>>>>>>> ...
>>>>>>>>>   [exec] Error: docs/src/nav/toc.xml is out of date, see broken 
>>>>>>>>> link messages above
>>>>>>>>>
>>>>>>>>> I suspect due to Max's recent changes to drawview.  Unless 
>>>>>>>>> someone else has been fiddling with documentation lately?
>>>>>>>>>
>>>>>>>>> On 2009-09-24, at 05:07, svnbuild at laszlosystems.com wrote:
>>>>>>>>>
>>>>>>>>>> 2009-09-24 02:07:14 PDT: FAILURE for 14837-openlaszlo-trunk 
>>>>>>>>>> with build return code 1 on 
>>>>>>>>>> linux-builder.corp.laszlosystems.com (Deploy return code -1; 
>>>>>>>>>> Megatest , -1) attaching 
>>>>>>>>>> /home/svnbuild/logs/tail-14837-trunk-nightly.log
>>>>>>>>>> <tail-14837-trunk-nightly.log>
>>>>>>>>>> _______________________________________________
>>>>>>>>>> Laszlo-builds mailing list
>>>>>>>>>> Laszlo-builds at openlaszlo.org
>>>>>>>>>> http://www.openlaszlo.org/mailman/listinfo/laszlo-builds
>>>>>>>>>
>>>>>>>
>>>>>>> -- 
>>>>>>> Regards,
>>>>>>> Max Carlson
>>>>>>> OpenLaszlo.org
>>>>>>
>>>>>>
>>>>>
>>
>> -- 
>> Regards,
>> Max Carlson
>> OpenLaszlo.org
> 

-- 
Regards,
Max Carlson
OpenLaszlo.org