Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to SDK version 12.1 and microhttpd >= 0.9.71 #12

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

derpeter
Copy link

Tested on ubuntu 21.04 and debian bullseye

derchrisuk and others added 6 commits November 28, 2019 02:41
Changes:

- Deprecated Interfaces
    - IDeckLinkAttributes
      - Replaced by IDeckLinkProfileAttributes interface

- Updated Types
    - BMDDeckLinkAttributeID
      - deprecated BMDDeckLinkSupportsDuplexModeConfiguration
          - Support of duplex mode can be determined using IDeckLinkProfileIterator

IDeckLinkAttributes has already been converted to IDeckLinkProfileAttributes

BMDDeckLinkSupportsDuplexModeConfiguration has been commented out,
currently compiles.
But needs to be changed to new IDeckLinkProfileIterator
@derpeter derpeter requested a review from MaZderMind June 30, 2021 21:56
derpeter added 2 commits July 1, 2021 09:46
This reverts commit 052c426.
This reverts commit 2017838.
@MaZderMind
Copy link
Contributor

Aw, sweet! Will test & review later today!

@derpeter
Copy link
Author

derpeter commented Jul 1, 2021

Just checked, buster / 20.04 have an to old microhttp versions, we need a switch in the code. trying to find a buster system to test this on

@derpeter
Copy link
Author

derpeter commented Jul 1, 2021

now it builds on ubuntu 20.04 / debian buster as well as on bullseye/21.04

@derpeter
Copy link
Author

derpeter commented Jul 1, 2021

i was pointed to #9 so we need to test sie on a duplex card i guess.

@derpeter
Copy link
Author

derpeter commented Jul 5, 2021

With 0c16b85 i guess we can now merge it an make a new release. Thanks @derchrisuk !!

@derpeter
Copy link
Author

derpeter commented Jul 9, 2021

@MaZderMind i guess now we can merge this and continue in master / new feature branches.

@MaZderMind
Copy link
Contributor

@derpeter I'm so sorry. I'm do deep into family things that ended up taking up way more time and energy. I wanted to grab zwo Cameras and test out the former "Duplex"-Feature of my Duo2 but didn't come around getting the cameras or even compiling the code. I'm deeply sorry, but I don't think I'll find the time or enegery to it in the next weeks either. Sad things to organize…

@derpeter
Copy link
Author

don't worry and thanks for the update.

@derchrisuk
Copy link
Member

I did a test with Duplex enabled on a Duo2.
But it will crash the debugger, claiming that it failed to enable video input.
Here is a debug log: https://p.rrbone.net/?45c68f6d5279273f#75HPTqLEAgW11BJJrDrfZ2GTUUh3WvQ7bkk5dSNYbTiY

@frankforpresident
Copy link

I just used this branch to creata a docker image with the Blackmagic DeckLink SDK 12.8 and it works fine for me. From my limited testing I could not reproduce @derchrisuk his issue

@derpeter Thanks for sharing this branch!

@derpeter
Copy link
Author

@frankforpresident your welcome. If someone with a one of the duplex cards at hand is could look into the remaining issue we can merge this into master i guess

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants