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

Generic media session Changes for AML. #31

Draft
wants to merge 17 commits into
base: generic-mediasession
Choose a base branch
from

Conversation

kvfasil
Copy link

@kvfasil kvfasil commented Dec 19, 2024

No description provided.

Reason for change:
	*removed soc dependancy.
	*Added playready packages.
Risks: High.
Signed-off-by: fasil kv <[email protected]>
Reason for change:
	*replaced code.
	*added pure virtual functions that have not been implemented.
Risks: High.
Signed-off-by: fasil kv <[email protected]>
@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 2 files pending identification.

  • Protex Server Path: /home/blackduck/github/OCDM-Playready/31/rdkcentral/OCDM-Playready

  • Commit: fe31916

Report detail: gist'

@@ -1,16 +1,34 @@
/*

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this the correct header? If the file was taken from elsewhere where it has an RDK header, then this is OK but please append a credit to top level NOTICE:

Copyright 2020 RDK Management
Licensed under the Apache License, Version 2.0

If not, then please use the Metrological header as used elsewhere in this component.

There was a spurious code match as well which I have cleared off.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mhughesacn
this is a work in progress changes.
will look into your comment make required changes.

@kvfasil kvfasil marked this pull request as draft December 19, 2024 11:08
@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/OCDM-Playready/31/rdkcentral/OCDM-Playready

  • Commit: e9676d5

Report detail: gist'

@kvfasil kvfasil force-pushed the generic-mediasession-aml branch from e9676d5 to d246dfb Compare December 19, 2024 11:51
@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/OCDM-Playready/31/rdkcentral/OCDM-Playready

  • Commit: d246dfb

Report detail: gist'

@kvfasil kvfasil force-pushed the generic-mediasession-aml branch from d246dfb to 96234ab Compare December 19, 2024 11:56
@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/OCDM-Playready/31/rdkcentral/OCDM-Playready

  • Commit: 96234ab

Report detail: gist'

@rdkcmf-jenkins
Copy link

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/OCDM-Playready/31/rdkcentral/OCDM-Playready

  • Commit: 6cbe869

Report detail: gist'

@kvfasil kvfasil force-pushed the generic-mediasession-aml branch from 25a4dd1 to 18e506f Compare December 31, 2024 09:54
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.

3 participants