Skip to content

Gstreamer 1.0: merge !49 and !54 into gstreamer_python_upgrade

Related to #54 (closed)

I merged with !49 (merged), then tdavies__catchup_to_O4_dev without realising it also included !54 (merged). But !54 (merged) is reasonably small & simple and I was halfway through the merge, so I didn't think it worth redoing and splitting out the commits.

There were a few differences with the other merge branch, but nothing major. importing from future was removed from a couple of python files, a fair bit of format rearranging (addressing some comments in !63 (closed) too).

There were a couple of exclusions from the other branhc, such as in one place a MapInfo wouldn't have been freed in the old version. If you'd like to review what I've excluded, you'd need to checkout this branch, do a git diff with tdavies__catchup_to_O4_dev, OR look at that thing's diff :|

Tests

I had to push a fix for _postcohtable.c in the latest commit. Other than that, I've had 3 runs, run 2 & 3 are identical: /fred/oz996/tdavies/spiir_project/sources/testing/gout/py3/MR78_tests

Edited by Timothy Davies

Merge request reports