[Pkg-pascal-devel] Castle Game Engine 7.0-alpha.2, view3dscene 4.2.0 releases

[Abou Al Montacir]

Hi All,

On Wed, 2022-10-26 at 02:52 +0200, Michalis Kamburelis wrote:
> Forgot to attach: error log from FreeWRL when reading
> CADGeometryPROTO.wrl.gz (before fixing with backslashes).
> 
> śr., 26 paź 2022 o 01:50 Michalis Kamburelis
> <michalis.kambi at gmail.com> napisał(a):
> > 
> > In this case, the VRML file
> > /usr/share/doc/whitedune-docs/docs/x3d/CADGeometryPROTO.wrl is really
> > invalid.
OK, makes sens to not support buggy files as Paul said also.
I'll drop this file from tests.
> > While I can imagine how to "tolerate" such files, but it would be
> > quite a hack in CGE (tovrmlx3d just uses Castle Game Engine to read
> > X3D). We would effectively guess too much "what was the intention of
> > the author of this VRML file". I would need to encode that, where a
> > single string is expected in VRML, we allow a sequence of "string +
> > (identifier + string)*" and we just glue them all together, assuming
> > that it's actually one long string but someone forgot backslashes.
> > That's too much guessing, and too big hack, just to support a single
> > invalid file :)
> > 
> > I wanted to look at what others do:
> > 
> > - FreeWRL: https://freewrl.sourceforge.io/ (4.0) also fails to read
> > it. Adding backslashes, as I suggested above, makes FreeWRL also read
> > it OK. So basically FreeWRL behaves the same as
> > view3dscene/tovrmlx3d/CGE in this case. And it's a completely
> > independent codebase to read VRML/X3D files.
> > 
> > - I wanted to test whitedune itself, alas it fails for me with a
> > SEGFAULT now (just running it without any parameters makes a
> > SEGFAULT). Tested on Debian (whitedune 0.30.10-2.2) and another
> > machine with Ubuntu (whitedune 0.30.10-2.2build1).
> > 
> > The part of the CADGeometryPROTO.wrl file that misses backslashes
> > contains a shader code inside a VRML string. While this is a comment
> > for shader (GLSL), but first the VRML browser must read the string
> > with shader code, and the "Script.url" must contain something that can
> > be parsed.
> > 
> > All in all, we cannot easily ignore this problem on tovrmlx3d side --
> > it would be a too big hack, just for this file. And I have yet to see
> > any VRML browser that actually opens this file :)
> > 
> > My suggestion would be to:
> > 
> > - Report this to whitedune-docs, as error in
> > /usr/share/doc/whitedune-docs/docs/x3d/CADGeometryPROTO.wrl.gz
> > 
> > - While you're at this, check can you run whitedune at all. If it
> > SEGFAULTs for others, then this is likely even more critical bug we
> > should submit to whitedune package.
> > 
> > - In view3dscene: either stop testing on whitedune-docs, or at least
> > omit this particular faulty file CADGeometryPROTO.wrl.gz .
> > 
> > - If you're looking for files to test, we maintain a big repository of
> > test models on https://github.com/castle-engine/demo-models . All on
> > open-source licenses. Though I understand you would want something
> > that is already part of Debian.
I can't download files from the internet, so I should package these and I don't
think it make sense.
So will try to filter out buggy files from whitedune.

-- 
Cheers,
Abou Al Montacir
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-pascal-devel/attachments/20221029/7a212c08/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: This is a digitally signed message part
URL: <http://alioth-lists.debian.net/pipermail/pkg-pascal-devel/attachments/20221029/7a212c08/attachment.sig>