Improve handling of image subtitle IDs in XML (DoM bug #1965)

When reading/writing the XML for image subtitles, we assumed that
the content of the <Image> tag is just the ID of the PNG in the MXF,
without any prefix.

DoM bug #1965 mentions a DCP where this is not the case, and SMPTE
429-5-2009 has an example where there is urn:uuid: in the XML.

This change makes DoM write this urn:uuid: prefix, and accept it if
it's present (but not complain if it's not).

If the urn:uuid: _is_ required in the field, it's a bit surprising
that nobody has complained up to this point.  Maybe nobody noticed,
or nobody reported it.

Cherry-picked from 098007a1ee6a46b6ff11398f94faff5c85951da4 in master.

Ignore parameters in PKL type strings when checking them.

Cherry-picked from 1ffc787c6318815592e8d81c4878488eb6183ca4 in master.

Rename write_subtitle_test -> shared_subtitle_test

Cherry-picked from 66566512275d363f68781b9a47ade6967d09e676 in master.

Test reference updates.

Move some tests from write_subtitle_test into interop_subtitle_test.

Cherry-picked from 39e010ee51749198528f754c89e98b6d8fb123c4 in master.

Rename read_interop_subtitle_test -> interop_subtitle_test.

Cherry-picked from 72d9cb462c03b1b41380e50ab9cccf2950207c18 in master.

Move some tests from write_subtitle_test to smpte_subtitle_test.

Cherry-picked from 9e3f8bfe9e274865590d49ad1e59e82f7894a12a in master.

Merge read_smpte_subtitle_test with smpte_subtitle_test.

Cherry-picked from 4e514f7dcc12a72bb4c78fd5067ff872ef54025b in master.

Fix erroneous reports of unresolved assets when checking OV/VF pairs.

Test churn after adding a new one.

Fix crash when loading DCPs whose ASSETMAP contains files that are
not mentioned in any PKL (#1855).

Back-ported from 4c8cc9e99678a4095772f283ffaaa1d6bd8aaf67 in master.

Be more cautious in CertificateChain::private_key_valid.

Add fix_empty_font_ids() to replace empty Font ids with a dummy string.

Back-ported from abf4f7577877254ecd7a1dc7955e0d7ca49ad07c in master.

Just emit EmptyAssetPathError from DCP::read, not a MissingAssetError as well.

Bump asdcplib for Python 3 compatibility in the build system.

Allow DCP reading to continue even with empty <Path> nodes in ASSETMAP.

Fix incorrect LocalTime strings when timezone is behind UTC and not an integer number of hours.

Fix enum confusion.

Fix unit tests after CPL root namespace fix.

Reject certificate chains where the adjacent certificates have the same subject.

This is a backport of 31f31800705880da6ec185f9b31c7e125ae18c38 from master.

In 8c852cb the CPL root attribute "dsig" was removed (to prevent
'XML root element can contain only one namespace' errors)

In a266fc2 [Sony digest validation fix] this was (for some
unknown reason) re-added, which makes the root element bug recur.

This commit removes that namespace again.

It's a shame there was no unit test to pick this regression up.

Increase header size when writing SMPTE subtitle MXFs; fixes problems when writing lots of subs (DCP-o-matic bug #1561).

This is a backport of b7c7e14463d91608b080a5a63fcfb4d330ced863 from master.

Throw a different exception when an unrecognised LabelSetType is found.

Add EPISODE and PROMO content types.

Fix incorrect dcpinfo output for sound assets.

Check that all image data is present after loading a SMPTE subtitle asset.

If we load SMPTE subtitles from a XML file try to load PNG data
from the same directory; this feels like a hack.

Tweak link flags.

And corresponding test updates.

More insulation against libdcp version updates.

Try to fix libdcp static build.

Update tests for previous.

Stop test output depending on libdcp version.

Fix gcov shlib build; this fixes tests to run against the locally-built libdcp rather than the installed one.

Fix string decoding; bump libcxml version.

Bump wscript required version of libcxml.

Bump libcxml to a release version.

Update for changes to video MXF headers.

Bump asdcplib for fix to DoM bug #1325.

Check return value of fread in File::File.

Fix test references now that we indent things.

Slightly hacky but hopefully functional fix for KDMs after the changes
to signer.  The KDM stuff uses add_signature_value() but not sign()
since it has to allow pass-through of a KDM (so it handles <Signature> etc.
itself).  This means we have to make the indentation-adding optional.

It might have been nicer to make indent() not add indentation if it's
not already there.

Remove sign test tool.

Fully indent PKL/CPL.

Attempt to fix Sony digest validation by indenting the <Signer>
and <Signature> before signing.  This is in the belief that, perhaps,
the Sony software "reformats" the XML before checking that the signature
is correct (or something).

Reveal path in MissingAssetError.

Missing include for arch.

Update waf and bump libcxml for similar updates.

Bump openjpeg2-cdist for debugging code.

Merge branch 'master' of ssh://git.carlh.net/home/carl/git/libdcp

Tidy repeated call of make_uuid() and don't put urn:uuid: in front of default content version label text as it's a bit confusing.

Speculative removal of call to xmlSecKeySetName when signing, on
the basis that I can't see what it's for and opendcp doesn't
do it.  Chasing Sony no-validate bug.

Move verify API away from strings towards error codes.

Fix crash with insufficient parameters to libdcp.

Fix incorrect XML namespace in MXF header; I was using the prefix
instead of the namespace URI for what asdcplib calls
TimedTextDescriptor::NamespaceName.

Thanks to clairmeta; see https://github.com/Ymagis/ClairMeta/issues/72

ClairMeta points out that SMPTE subtitle MXF's ResourceID should
be the same as the internal XML ID.  This in turn must be different
from the MXF's UUID.

I *think* that we have:

AssetUUID: MXF's UUID, which must be different; written from
the WriterInfo (set up by fill_writer_info, which is passed _id)

AssetID: MXF's ResourceID, which must be the _xml_id; written from
the TimedTextDescriptor.

I believe that this patch fixes the ResourceID to be the XML ID (as
required) without undoing the Doremi fix whereby the MXF UUID must
be different to the XML ID.

This patch quells ClairMeta's error.

More DCP verification detail and testing.

Tidy up a bit.

Tidy up and slightly extend verify tests.

Fix tests following additional verify check.

Check CPL hash in the PKL on verify.

Merge branch 'master' of ssh://git.carlh.net/home/carl/git/libdcp

Very simple dcpverify.

Fix some warnings.

Add duration as H:M:S:F.

Simpole DCP recovery utility (dcprecover) added.

Take thumbprint rather than a full dcp::Certificate for trusted devices.

Slightly nicer error handling.

Rename dcpsignerthumb -> dcpthumb.

More raw/locale_convert.

Merge branch 'master' of ssh://main.carlh.net/home/carl/git/libdcp

More _convert functions.

Move some methods to where they make more sense.

Cope with multiple PKLs in a DCP.

ID churn.

Merge branch 'master' of ssh://git.carlh.net/home/carl/git/libdcp

Add missing constructor call.

Fix build with newer boost.

Throw a better error on bad content kind.

Add operator< for LocalTime.

Bump waf.

Bump libcxml for waf upgrade.

Bump libcxml.

Fix missing version string when Popen communicate returns byte strings.

Silence warnings from asdcplib when testing a MXF file for validity.

Hopefully-correct PKL and AssetMap when using Interop PNG subtitles.

Merge branch 'master' of ssh://main.carlh.net/home/carl/git/libdcp

Allow CCAP reel nodes to have language tags.

Missing initialisattion of DCPReadError::_message.

Support more than one closed caption asset per reel.

Missing test files.

Missing header include.

Throw an exception on an unknown asset type.

Unexplained ID churn.

Adjust for Interop <PackingList> not needing content.

Remove debug.

Use PKL types rather than file extensions.

Removed unused variables.

Use PKL when verifying DCPs.

Read PKL when reading DCP.

Merge branch 'master' of ssh://git.carlh.net/home/carl/git/libdcp