Re: [Mlt-devel] Producer xml regression

[Dan D. <da...@de...>]

On Sun, Sep 2, 2012 at 12:51 PM, j-b-m <j-...@us...> wrote:
> Hello Dan,
>
> You made a change to the xml producer that causes problems in Kdenlive ( for
> example http://kdenlive.org/mantis/view.php?id=2711 ).
>
> The commit is the following one:
> https://github.com/mltframework/mlt/commit/89ff4d149a0ac54deab4057178678b68bef1e92d

Do you understand the reason for the change? If you have an xml file
that contained simply a producer element under the root, then it would
overwrite the producer's resource property with the filename of the
xml file, which causes problems for the producer or for
re-serialization!

> In Kdenlive, we sometimes use the "resource" property of a clip to create a
> duplicate of the producer or for other reasons.
>
> Since the mentioned commit, the "resource" property of an xml playlist is not
> set anymore (it defaults to "<playlist>"), and it seems it is not possible
> anymore to get the location of the playlist file.
>
> What do you think about it? The "resource" property is used in several
> producers, and it seems to me that it would be good to keep it working in the
> xml producer.

Then, the resource attribute belongs on the root element and not
overwriting some other object's resource property. I will see how to
make that change.

> Let me know if you agree or if I should workaround the issue in Kdenlive.
>
> regards
> jb
>
-- 
+-DRD-+