search

nlbdev / nordic-epub3-dtbook-migrator

Tools for converting between a strict subset of DTBook and EPUB3.
http://nlbdev.github.io/nordic-epub3-dtbook-migrator/
GNU Lesser General Public License v2.1
8 stars 7 forks source link

Disable the pub identifier requirement. #485

Closed kalaspuffar closed 3 years ago

kalaspuffar commented 3 years ago

Hi @martinpub

I've commented out the requirement to have unique-identifier set to pub-identifier.

Best regards Daniel

martinpub commented 3 years ago

Thanks @kalaspuffar!

For information, the reason that this is removed is that currently, Pipeline 2 EPUB generation does not preserve the name of the connecting attribute "pub-identifier", while still preserving integrity between dc:identifier and unique-identifier. A relaxation has been suggested in https://github.com/daisy/pipeline/issues/612.

It should be noted that already EPUBCheck verifies the presence of unique-identifier on the root element of package.opf, and that it is pointing to a corresponding meta tag with an ID.

We can get back to discussing this, after summer, but this will solve a production block at MTM and given the discussion above, the removal is not considered dangerous.