Remove "minimum version of KDE needed" check of Kiosktool. #4

Open
Chris wants to merge 1 commits from branding/fix_kde_version into master
Chris commented 5 years ago
Collaborator

Signed-off-by: Chris xchrisx@uber.space

Signed-off-by: Chris <xchrisx@uber.space>
MicheleC added the PR/update-trans label 5 years ago
Owner

Marked as translation update required, since we need to remove the string from the translation files too.

Marked as translation update required, since we need to remove the string from the translation files too.
Owner

The question is whether updating of translations needs to be addressed. After the next update of the POT files, the string will automatically disappear from the template and will then be moved as "outdated" in the PO files.

The question is whether updating of translations needs to be addressed. After the next update of the POT files, the string will automatically disappear from the template and will then be moved as "outdated" in the PO files.
Owner

I think we should still need to update translation. Since the strings would disappear from the .pot file, I see no reasons to keep the now-unused strings in the .po files 😄

I think we should still need to update translation. Since the strings would disappear from the .pot file, I see no reasons to keep the now-unused strings in the .po files :smile:
Owner

Yes, it is true that it will not be useful for this string to be kept as obsolete in PO files 😸

Yes, it is true that it will not be useful for this string to be kept as obsolete in PO files :smile_cat:
Chris commented 5 years ago
Poster
Collaborator

Michele, that is a good thing to talk about. I just had the same idea to remove them. But we did not do it for other commits, so I thought it would be unneeded.

From Slavek I learned what he wrote above and so thought that there are maybe a good bunch of unused strings in the *.po files already. That might be a good thing to review at some point. I don't know.

I will remove the unused strings from the *.pot and *.po files anyway today or tomorrow. 👍

Michele, that is a good thing to talk about. I just had the same idea to remove them. But we did not do it for other commits, so I thought it would be unneeded. From Slavek I learned what he wrote above and so thought that there are maybe a good bunch of unused strings in the *.po files already. That might be a good thing to review at some point. I don't know. I will remove the unused strings from the *.pot and *.po files anyway today or tomorrow. :+1:
Owner

IMO, if a string is not in a pot file and we are aware of that, it makes no sense to keep it in the po files.
@Chris: thanks for the excellent work you are doing in polishing the brand thing.

IMO, if a string is not in a pot file and we are aware of that, it makes no sense to keep it in the po files. @Chris: thanks for the excellent work you are doing in polishing the brand thing.
Owner

With PO files, this is done as follows:

When the string disappears from the template (ie POT) file and its translation has been included in the existing PO files, such translation is pushed to the end of the file and is marked as obsolete with the #~ tag.

The idea is that older translations do not disappear immediately, but could be useful for translators if a similar string or part of it appears again and the translation or part of it can be reused. Such strings are not included in binary mo files, but can save time for translators.

Therefore, it is usually expedient to leave the original translations to be processed by this mechanism. However, in this case it seems quite certain that a similar translation will not be used here, so it can be removed.

With PO files, this is done as follows: When the string disappears from the template (ie POT) file and its translation has been included in the existing PO files, such translation is pushed to the end of the file and is marked as obsolete with the `#~` tag. The idea is that older translations do not disappear immediately, but could be useful for translators if a similar string or part of it appears again and the translation or part of it can be reused. Such strings are not included in binary `mo` files, but can save time for translators. Therefore, it is usually expedient to leave the original translations to be processed by this mechanism. However, in this case it seems quite certain that a similar translation will not be used here, so it can be removed.
Owner

@SlavekB yes, I understand your point and it makes sense. On the otherhand, I have two observations to do:

  1. would those obsolete strings be available through weblate for the translators, since the original string is no longer in the .pot file?
  2. it is more likely that a translator simply work on strings one after another insteand of looking around (weblate or source code) to compare to other strings. Just my two cents on this.
@SlavekB yes, I understand your point and it makes sense. On the otherhand, I have two observations to do: 1. would those obsolete strings be available through weblate for the translators, since the original string is no longer in the .pot file? 2. it is more likely that a translator simply work on strings one after another insteand of looking around (weblate or source code) to compare to other strings. Just my two cents on this.
Owner

Yes, you're right. These obsolete strings are not visible on the Weblate, so translators would have to search the source PO files, which they usually won't do.

Yes, you're right. These obsolete strings are not visible on the Weblate, so translators would have to search the source PO files, which they usually won't do.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/kiosktool#4
Loading…
There is no content yet.