readium / readium-sdk

A C++ ePub renderer SDK
BSD 3-Clause "New" or "Revised" License
389 stars 164 forks source link

ePubs with XML 1.1 fail to load for Windows #167

Open bdares opened 9 years ago

bdares commented 9 years ago

The Windows XML parser can't handle anything other than XML 1.0. see https://social.msdn.microsoft.com/Forums/en-US/ebf3df45-c934-46d7-adc9-61d2d1ebead9/which-msxml-verion-support-parse-xml-11?forum=xmlandnetfx

XML 1.1 support was dropped with ePub3 but this is still causing issues with legacy ePub2 books.

danielweck commented 9 years ago

I understand that MSXML is used across the board, but just out of interest: are you referring to "modern" Windows RT (managed framework) or "classic" Windows (native C++ code). In other words, did you reproduce this bug using the MFC Readium launcher: https://github.com/readium/SDKLauncher-Windows/tree/feature/feature_revised_launcher https://github.com/readium/readium-sdk/tree/feature/feature_revised_launcher ...or using a modern app based on one of these: https://github.com/readium/readium-sdk/tree/develop/Platform/WinRT https://github.com/readium/readium-sdk/tree/feature/WinRT https://github.com/readium/readium-sdk/tree/feature/WindowsRT_v2 https://github.com/readium/readium-sdk/tree/WinPhone8 ?

rkwright commented 9 years ago

I believe this is the folks who built their own Windows RT-framework launcher so probably the latter.