Replace Qt with TQt #27

Merged
MicheleC merged 1 commits from cmake/lower-case into master 3 months ago
Owner

As per title.

As per title.
MicheleC added this to the R14.1.2 release milestone 3 months ago
MicheleC added 1 commit 3 months ago
7b4e75c524
cmake files: change keywords to lower case
MicheleC requested review from Fat-Zer 3 months ago
MicheleC requested review from Core 3 months ago
MicheleC requested review from Owners 3 months ago
Fat-Zer requested changes 3 months ago
Fat-Zer left a comment
Collaborator

just a few minor things

just a few minor things
<para>
Due to limitations in prior releases, only version 3 of SQLite is
supported. Unfortunately, the release of the &Qt; toolkit used by &kappname;
supported. Unfortunately, the release of the &TQt; toolkit used by &kappname;
Collaborator

this is technically false as we do provide sqlite3 support.

this is technically false as we do provide sqlite3 support.
Poster
Owner

Should we remove this paragraph as a whole?

Should we remove this paragraph as a whole?
Collaborator

I suppose yes; there is no use for it anymore.

I suppose yes; there is no use for it anymore.
Poster
Owner

Ok, paragraph removed.

Ok, paragraph removed.
MicheleC marked this conversation as resolved
MyMoneyDbDrivers ();
/**
* @return a list ofsupported TQt database driver types, their qt names and useful names
* @return a list ofsupported TQt database driver types, their tqt names and useful names
Collaborator

s/ofsupported/of supported/

s/ofsupported/of supported/
MicheleC marked this conversation as resolved
/* these should be values which qt 3.3 never uses for TQTab:
* qt starts upwards from 0
/* these should be values which tqt 3.3 never uses for TQTab:
* tqt starts upwards from 0
Collaborator

probably should remove version

probably should remove version
MicheleC marked this conversation as resolved
"Non ci sono driver TQt SQL installati sul tuo sistema.\n"
"Consulta la documentazione della tua distribuzione, o visita il sito web Qt "
"Consulta la documentazione della tua distribuzione, o visita il sito web TQt "
"(www.trolltech.com) e cerca i driver SQL."
Collaborator

All strings but Italian are fuzzy anyway, so assuming we have an active Italian translator, you can go ahead and safely remove the reference to the www.trolltech.com from the source string altogether...

All strings but Italian are fuzzy anyway, so assuming we have an active Italian translator, you can go ahead and safely remove the reference to the `www.trolltech.com` from the source string altogether...
Poster
Owner

so assuming we have an active Italian translator

I guess I know the guy quite well 😉

safely remove the reference to the www.trolltech.com

Done. I have also adjusted the translations and remove fuzzy for those I am confident the translation is fine. I left fuzzy for those I am not sure about.

> so assuming we have an active Italian translator I guess I know the guy quite well 😉 > safely remove the reference to the www.trolltech.com Done. I have also adjusted the translations and remove `fuzzy` for those I am confident the translation is fine. I left `fuzzy` for those I am not sure about.
MicheleC marked this conversation as resolved
MicheleC force-pushed cmake/lower-case from 7b4e75c524 to f60c4ad9e8 3 months ago
Poster
Owner

PR updated as per feedback. Only the first point is still open for solution.

PR updated as per feedback. Only the first point is still open for solution.
MicheleC force-pushed cmake/lower-case from f60c4ad9e8 to 41c1fd1fc0 3 months ago
Poster
Owner

PR updated, should be good now.

PR updated, should be good now.
MicheleC requested review from Fat-Zer 3 months ago
Fat-Zer approved these changes 3 months ago
Fat-Zer left a comment
Collaborator

LGTM

LGTM
MicheleC merged commit 41c1fd1fc0 into master 3 months ago
MicheleC deleted branch cmake/lower-case 3 months ago

Reviewers

Fat-Zer approved these changes 3 months ago
TDE/Core was requested for review 3 months ago
TDE/Owners was requested for review 3 months ago
The pull request has been merged as 41c1fd1fc0.
Sign in to join this conversation.
No reviewers
TDE/Core
TDE/Owners
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/kmymoney#27
Loading…
There is no content yet.