Skip to content

Conversation

Noremos
Copy link
Contributor

@Noremos Noremos commented Feb 5, 2025

This is a fix for #8430

@AlexPeshkoff AlexPeshkoff self-assigned this Feb 5, 2025
@AlexPeshkoff AlexPeshkoff merged commit 76eccad into FirebirdSQL:master Feb 6, 2025
24 checks passed
@hvlad
Copy link
Member

hvlad commented Sep 14, 2025

Unfortunately, this broke validation service (at least) ability to print formatted messages.

Current (wrong) output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors

Correct output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors
-       Number of record level warnings : 1

@Noremos
Copy link
Contributor Author

Noremos commented Sep 15, 2025

Unfortunately, this broke validation service (at least) ability to print formatted messages.

Current (wrong) output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors

Correct output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors
-       Number of record level warnings : 1

Sad to hear. I'll look into the problem.

@Ev3nt Ev3nt mentioned this pull request Sep 22, 2025
@Noremos
Copy link
Contributor Author

Noremos commented Sep 23, 2025

Unfortunately, this broke validation service (at least) ability to print formatted messages.

Current (wrong) output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors

Correct output:

fbsvcmgr.exe inet://service_mgr action_repair dbname XXX.FDB rpr_validate_db rpr_full
Summary of validation errors
-       Number of record level warnings : 1

Due to some real-life commitments, I have little free time, so my friend covered for me and made the fix: #8752

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.

4 participants