-
Notifications
You must be signed in to change notification settings - Fork 602
Upgrade osgi-core to 8.0.0 #15335
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
base: 3.0-dev
Are you sure you want to change the base?
Upgrade osgi-core to 8.0.0 #15335
Conversation
|
@durgajagadeesh There are license check failures. Can you check them? ##[warning]License issues found. |
1bed6c5 to
c2026aa
Compare
Hi @sandeepkarambelkar, buddy build has been completed. Successfully. |
sandeepkarambelkar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Buddy Build Successful.
Changes LGTM
kgodara912
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As such package upgrade looks fine. Buddy build is successful. Please address minor nit.
|
Hi @sandeepkarambelkar, could you please retrigger buddy build once again with run time dependency only with osgi-annotation. |
380cdd2 to
016a3ef
Compare
|
Buddy build after latest commit. |
Hi @sandeepkarambelkar, I’ve resolved the license warning. INFO[0004][licensecheck] Search results for (/mnt/vss/_work/1/s/out/RPMS): Could you please trigger the buddy build again? Thanks! |
kgodara912
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Buddy build is successful. Version upgrade in extended repo and fixes for license warnings in both specs. LGTM.


Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
Change Log
Does this affect the toolchain?
NO
Associated issues
Links to CVEs
Test Methodology