Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove QLI. #6840

Merged
merged 1 commit into from
Jun 8, 2021
Merged

Remove QLI. #6840

merged 1 commit into from
Jun 8, 2021

Conversation

asfernandes
Copy link
Member

No description provided.

@asfernandes asfernandes self-assigned this Jun 2, 2021
@asfernandes
Copy link
Member Author

Anyone against merge this?

@dyemanov
Copy link
Member

dyemanov commented Jun 8, 2021

Merge it.

@asfernandes asfernandes merged commit 010c3a8 into master Jun 8, 2021
@asfernandes asfernandes deleted the work/qli-remove branch June 8, 2021 13:31
Comment on lines -265 to -268
build_dbs_: $(MSG_DB) $(HELP_DB) $(META_DB)
build_dbs_clean:
rm -f $(MSG_DB) $(HELP_DB) $(META_DB)
build_dbs_%:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I'm missing something, but shouldn't this remain for $(MSG_DB)?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure this file should even exist.

@@ -2,33 +2,33 @@ set bulk_insert INSERT INTO TRANSMSGS (ENG_TEXT, ENG_ACTION, ENG_EXPLANATION, FA
-- Notice LOCALE is hardcoded as 'de_DE' in the INSERT statement and therefore the rows below have one fields less than the table.
-- JRD
('', NULL, NULL, 0, 0, '', NULL, NULL, 'daves', '1993-07-16 12:15:54')
('arithmetic exception, numeric overflow, or string truncation', NULL, NULL, 0, 1, 'Arithmetischer Fehler, numerischer �berlauf oder Zeichenkette abgeschnitten', NULL, NULL, 'daves', '1993-07-16 12:16:07')
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes seem to have corrupted the character encoding of the file.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants