Fix ptest for fontconfig#16133
Merged
kgodara912 merged 3 commits intomicrosoft:3.0-devfrom Mar 10, 2026
Merged
Conversation
Contributor
|
/azurepipelines run |
|
Azure Pipelines successfully started running 1 pipeline(s). |
Contributor
Contributor
Contributor
Author
|
Buddy Build is successful |
kgodara912
approved these changes
Mar 10, 2026
Contributor
kgodara912
left a comment
There was a problem hiding this comment.
ptest fix for fontconfig. Buddy build is successful after fix. run-test.sh test is passing after excluding the specific sub test as excluded in upstream reference. VERBOSE=1 is not generating many logs and test logs are a little more than normal logs. LGTM.
BinduSri-6522866
pushed a commit
to BinduSri-6522866/azurelinux
that referenced
this pull request
Mar 31, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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?
Fix ptest for fontconfig
The test was failing at run-test.sh script which tests like MT-safe global config which requires curl/unzip to fetch Noto fonts.
As per the AI analysis, in a constrained CI environment, missing bubblewrap (bwrap) results in those sections being skipped; however other environment dependencies like curl/unzip or access to external network may cause aborts within the "MT-safe global config" block. Also, several checks compare directory mtimes and contents with SOURCE_DATE_EPOCH; differing filesystem semantics can produce mismatches. Without test-suite.log, the most probable subtest failures are those requiring network access and external tools (curl, unzip) or sandboxing via bwrap. The script traps EXIT and will print "Aborting from '' with the exit code" in failure, but this message is not captured in the provided log, suggesting make harness only records the FAIL
Suggested Fix: Ensure CI environment provides required tools and conditions: install or make available curl and unzip; provide internet access or mock Noto font download; if external network not allowed, skip MT-safe global config by disabling that subtest via environment flag or patch.
Hence we have skipped the test using patch provided by https://src.fedoraproject.org/rpms/fontconfig/c/1bacf0fb4436ce5fea7e5db2de0ebb823c1f2471
Change Log
Does this affect the toolchain?
NO
Associated issues
Test Methodology