Skip to content

[6.36] [xml] prevent files parsing from parent folder#21855

Merged
linev merged 3 commits intoroot-project:v6-36-00-patchesfrom
linev:xml_636
Apr 9, 2026
Merged

[6.36] [xml] prevent files parsing from parent folder#21855
linev merged 3 commits intoroot-project:v6-36-00-patchesfrom
linev:xml_636

Conversation

@linev
Copy link
Copy Markdown
Member

@linev linev commented Apr 9, 2026

Backport fix without roottest part from #21812

linev added 2 commits April 9, 2026 07:46
Check using `std::filesystem::proximate` functionality
When file cannot be opened - do not try to parse it, just print failure
And print correctly error when string is parsing
@linev linev self-assigned this Apr 9, 2026
@linev linev requested a review from pcanal as a code owner April 9, 2026 05:48
@linev
Copy link
Copy Markdown
Member Author

linev commented Apr 9, 2026

/backport to 6.32, 6.30, 6.28, 6.26

@dpiparo
Copy link
Copy Markdown
Member

dpiparo commented Apr 9, 2026

The alma8, i.e. gcc 8.5, build is likely to be fixed if the stdc++fs library is linked. The switch should be -lstdc++fs and should be conditionally added on linux and if gcc version <=8.

@linev linev requested a review from bellenot as a code owner April 9, 2026 09:08
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 9, 2026

Test Results

    18 files      18 suites   2d 14h 44m 35s ⏱️
 2 753 tests  2 752 ✅  1 💤 0 ❌
47 849 runs  47 837 ✅ 12 💤 0 ❌

Results for commit 032722f.

@linev linev merged commit 9b0c14b into root-project:v6-36-00-patches Apr 9, 2026
21 of 22 checks passed
@linev linev deleted the xml_636 branch April 9, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants