-
Notifications
You must be signed in to change notification settings - Fork 602
perl-Test-Simple removal from 3.0-dev. #15380
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
perl-Test-Simple removal from 3.0-dev. #15380
Conversation
c077a8b to
309950f
Compare
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.
Changes LGTM.
Triggered full build for verifying removal.
|
Full build to verify tensorflow package build failure - https://dev.azure.com/mariner-org/mariner/_build/results?buildId=1015445&view=results |
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.
Please address the merge conflict.
ea8d55b to
b87d489
Compare
|
I have resolved the merge conflict. |
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.
Removal of extended repo package as the package is already available in core repo. Buddy build is successful for one arch and timeout out for another which is unrelated to this PR. 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
What does the PR accomplish, why was it needed?
Removal of perl-Test-Simple package from extended since it causes conflicts sometimes during install tests for packages and it is maintained in core specs.
Change Log
Does this affect the toolchain?
NO
Test Methodology