Skip to content

DPL: boost::property_tree::json_parser::write_json no longer adds EoL#15333

Merged
ktf merged 1 commit intoAliceO2Group:devfrom
aalkin:fix-dpl-config-json-missing-brace
Apr 27, 2026
Merged

DPL: boost::property_tree::json_parser::write_json no longer adds EoL#15333
ktf merged 1 commit intoAliceO2Group:devfrom
aalkin:fix-dpl-config-json-missing-brace

Conversation

@aalkin
Copy link
Copy Markdown
Member

@aalkin aalkin commented Apr 27, 2026

Fixes malformed dpl-config.json at the end of processing, as the old code was removing the closing brace.

@aalkin aalkin requested a review from a team as a code owner April 27, 2026 16:12
@ktf
Copy link
Copy Markdown
Member

ktf commented Apr 27, 2026

https://github.com/boostorg/property_tree/pull/23/changes is the upstream change. How come this does not break everything?

@aalkin
Copy link
Copy Markdown
Member Author

aalkin commented Apr 27, 2026

We only use this route to collect the configuration at the end of processing, otherwise we would have noticed it when bumping boost.

@ktf ktf merged commit 5dfe60e into AliceO2Group:dev Apr 27, 2026
9 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants