amf: Deprecate legacy AMD encoder - Githubissues

obsproject / obs-amd-encoder

AMD Advanced Media Framework Encoder Plugin for Open Broadcaster Studio

https://obsproject.com/forum/threads/amd-advanced-media-framework-encoder-plugin-for-obs-studio.52305/

GNU General Public License v2.0

457 stars 87 forks source link

amf: Deprecate legacy AMD encoder #430

Closed derrod closed 2 years ago

derrod commented 2 years ago

Description

Add deprecated flag to encoder.

Motivation and Context

Encoder has been replaced.

How Has This Been Tested?

Testing?

Types of changes

Tweak (non-breaking change to improve existing functionality)

Checklist:

[x] My code has been run through clang-format.
[x] I have read the contributing document.
[x] My code is not on the master branch.
[ ] The code has been tested.
[x] All commit messages are properly formatted and commits squashed where appropriate.
[x] I have included updates to all appropriate documentation.

dodgepong commented 2 years ago

Can we also add a "(deprecated)" or "(legacy)" label to the old encoder entry on the encoder list?

RytoEX commented 2 years ago

Superseded by https://github.com/obsproject/obs-amd-encoder/commit/5880f37307dbf36a4dc2dfc75ab13bbbfd055102 ?

derrod commented 2 years ago

Yes

jp9000 commented 2 years ago

Sorry about that, didn't see this one (darn submodules)

RytoEX commented 2 years ago

No worries. The important thing is that we get the deprecation done.