Skip to content

RDK-52204: Convert GDial as Library #114

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

yuvaramachandran-gurusamy
Copy link
Contributor

RDK-52204: Convert GDial as Library

Reason for change: Convert GDial as Library, Later it will be used from XCast OutOfProcess Thunder plugin
Test Procedure: GDial should be converted as library
Risks: HIGH
Priority: P0

Signed-off-by: yuvaramachandran_gurusamy yuvaramachandran_gurusamy@comcast.com

Reason for change: Convert GDial as Library, Later it will be used from XCast OutOfProcess Thunder plugin
Test Procedure: GDial should be converted as library
Risks: HIGH
Priority: P0

Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
@rdkcmf-jenkins
Copy link
Contributor

Copyright scan failure

Commit: ceb45e1
Report detail: https://gist.github.com/rdkcmf-jenkins/af972fe846c81ea20dbde6543aabbcfc

@rdkcmf-jenkins
Copy link
Contributor

Blackduck scan failure details

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

  • Protex Server Path: /home/blackduck/github/xdialserver/114/rdkcentral/xdialserver

  • Commit: ceb45e1

Report detail: gist

@rdkcmf-jenkins
Copy link
Contributor

build-rpi4-ipstb-oss Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/4b2c20414f3f9ae353b8722dee081256

@rdkcmf-jenkins
Copy link
Contributor

build-brcm-refboard3 Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/80f631bee08f7ffc4f94a4a41edaf981

gdialService *m_gdialService{nullptr};
gdialServiceImpl *m_gdialServiceImpl{nullptr};

static const char *dial_specification_copyright = "Copyright (c) 2017 Netflix, Inc. All rights reserved.";

Choose a reason for hiding this comment

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

Please add an extra phrase ", licensed under the BSD-2 license" to the copyright string.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok sure

Choose a reason for hiding this comment

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

Hi: I have been looking at this again, and the extra phrase is not needed. Netflix doesn't require anything extra. The license is not exactly BSD-2, and is available in NOTICE in any case. If the "Draft" status changes, please slack me to upvote.

@yuvaramachandran-gurusamy yuvaramachandran-gurusamy marked this pull request as draft August 29, 2024 12:19
@yuvaramachandran-gurusamy
Copy link
Contributor Author

Moving this PR into Draft as testing inprogress

Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
@rdkcmf-jenkins
Copy link
Contributor

b'## Copyright scan failure
Commit: 8c57e7b
Report detail: https://gist.github.com/rdkcmf-jenkins/069d62f20b0bec422eb1973023b763f6'

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

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

  • Protex Server Path: /home/blackduck/github/xdialserver/114/rdkcentral/xdialserver

  • Commit: 8c57e7b

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

build-rpi4-ipstb-oss Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/3f1d284aea96cae20088daf7cd57493a

@rdkcmf-jenkins
Copy link
Contributor

build-brcm-refboard3 Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/79275aa6c79412a72aee2dbd75c71317

@rdkcmf-jenkins
Copy link
Contributor

b'## Copyright scan failure
Commit: 0be6a94
Report detail: https://gist.github.com/rdkcmf-jenkins/4380cab58c258641ffa2c592e350c285'

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

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

  • Protex Server Path: /home/blackduck/github/xdialserver/114/rdkcentral/xdialserver

  • Commit: 0be6a94

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

build-rpi4-ipstb-oss Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/191dc9861aaf6579213b54a05efc4978

@rdkcmf-jenkins
Copy link
Contributor

build-brcm-refboard3 Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/61cda0c412429d1a6cc926484e1a3329

@rdkcmf-jenkins
Copy link
Contributor

b'## Copyright scan failure
Commit: 1164323
Report detail: https://gist.github.com/rdkcmf-jenkins/5d54700b73559d6ceb54b8b97d1510df'

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

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

  • Protex Server Path: /home/blackduck/github/xdialserver/114/rdkcentral/xdialserver

  • Commit: 1164323

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

build-rpi4-ipstb-oss Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/f930391d15c903deb1a112fe00d5c751

@rdkcmf-jenkins
Copy link
Contributor

b'## Copyright scan failure
Commit: 9454ba8
Report detail: https://gist.github.com/rdkcmf-jenkins/b11eaceee83e13473aba6f5d44339092'

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

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

  • Protex Server Path: /home/blackduck/github/xdialserver/114/rdkcentral/xdialserver

  • Commit: 9454ba8

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

build-brcm-refboard3 Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/0cb7c26923ffea51b26ced94a06ee18a

@rdkcmf-jenkins
Copy link
Contributor

build-brcm-refboard3 Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/0683fb4c73cd47a139ed9b181d8ad1d6

@rdkcmf-jenkins
Copy link
Contributor

build-rpi4-ipstb-oss Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/15656aff742b60a323e526c5cfe9ca88

Signed-off-by: yuvaramachandran_gurusamy <yuvaramachandran_gurusamy@comcast.com>
@rdkcmf-jenkins
Copy link
Contributor

b'## Copyright scan failure
Commit: 0d1151f
Report detail: https://gist.github.com/rdkcmf-jenkins/4dcf769d8d6ae479a0745ffa9da8a8b9'

@rdkcmf-jenkins
Copy link
Contributor

b'## Blackduck scan failure details

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

  • Protex Server Path: /home/blackduck/github/xdialserver/114/rdkcentral/xdialserver

  • Commit: 0d1151f

Report detail: gist'

@rdkcmf-jenkins
Copy link
Contributor

build-rpi4-ipstb-oss Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/ce7450b7ca9148e5df2f6b503c215c85

@rdkcmf-jenkins
Copy link
Contributor

build-brcm-refboard3 Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/a4117fb64053eaccbba0c4985f079bb5

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