Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Transifex bot does not work #22796

Closed
tobiasKaminsky opened this issue Sep 11, 2020 · 6 comments · Fixed by #22858 or #22860
Closed

Transifex bot does not work #22796

tobiasKaminsky opened this issue Sep 11, 2020 · 6 comments · Fixed by #22858 or #22860
Assignees
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug high

Comments

@tobiasKaminsky
Copy link
Member

Our internal transifex bot is not working correctly on server repo:

+ msgcat --use-first stable-templates/master.updatenotification.pot stable-templates/stable17.updatenotification.pot stable-templates/stable18.updatenotification.pot stable-templates/stable19.updatenotification.pot\n+ cut -b 25-\n+ echo stable-templates/master.user_ldap.pot\n+ name=user_ldap.pot\n+ msgcat --use-first stable-templates/master.user_ldap.pot stable-templates/stable17.user_ldap.pot stable-templates/stable18.user_ldap.pot stable-templates/stable19.user_ldap.pot\n+ cut -b 25-\n+ echo stable-templates/master.workflowengine.pot\n+ name=workflowengine.pot\n+ msgcat --use-first stable-templates/master.workflowengine.pot stable-templates/stable17.workflowengine.pot stable-templates/stable18.workflowengine.pot stable-templates/stable19.workflowengine.pot\n+ rm -rf stable-templates\n+ tx push -s\ntx INFO: Pushing resource nextcloud.core\ntx INFO: Pushing source file (translationfiles/templates/core.pot)\ntx INFO: Pushing resource nextcloud.files\ntx INFO: Pushing source file (translationfiles/templates/files.pot)\ntx INFO: Pushing resource nextcloud.settings-1\ntx INFO: Pushing source file (translationfiles/templates/settings.pot)\ntx ERROR: Error received from server: Found unexpected msgstr[*] on the non pluralized entry with msgid `aria-hidden`.\ntx ERROR: Could not upload source file. You can use --skip to ignore this error and continue the execution.\ntx ERROR: Error received from server: Found unexpected msgstr[*] on the non pluralized entry with msgid `aria-hidden`.')"

@nextcloud/server-triage

@tobiasKaminsky tobiasKaminsky added bug 0. Needs triage Pending check for reproducibility or if it fits our roadmap high labels Sep 11, 2020
@tobiasKaminsky
Copy link
Member Author

Seems to still be the case:
(I re-triggered transifex bot):

+ msgcat --use-first stable-templates/master.updatenotification.pot stable-templates/stable17.updatenotification.pot stable-templates/stable18.updatenotification.pot stable-templates/stable19.updatenotification.pot
+ cut -b 25-
+ echo stable-templates/master.user_ldap.pot
+ name=user_ldap.pot
+ msgcat --use-first stable-templates/master.user_ldap.pot stable-templates/stable17.user_ldap.pot stable-templates/stable18.user_ldap.pot stable-templates/stable19.user_ldap.pot
+ cut -b 25-
+ echo stable-templates/master.workflowengine.pot
+ name=workflowengine.pot
+ msgcat --use-first stable-templates/master.workflowengine.pot stable-templates/stable17.workflowengine.pot stable-templates/stable18.workflowengine.pot stable-templates/stable19.workflowengine.pot
+ rm -rf stable-templates
+ tx push -s
tx INFO: Pushing resource nextcloud.core
tx INFO: Pushing source file (translationfiles/templates/core.pot)
tx INFO: Pushing resource nextcloud.files
tx INFO: Pushing source file (translationfiles/templates/files.pot)
tx INFO: Pushing resource nextcloud.settings-1
tx INFO: Pushing source file (translationfiles/templates/settings.pot)
tx ERROR: Error received from server: Found unexpected msgstr[*] on the non pluralized entry with msgid `aria-hidden`.
tx ERROR: Could not upload source file. You can use --skip to ignore this error and continue the execution.
tx ERROR: Error received from server: Found unexpected msgstr[*] on the non pluralized entry with msgid `aria-hidden`.

@MorrisJobke
Copy link
Member

Let me have a look.

@MorrisJobke
Copy link
Member

Ah ... there are .l10nignore files in the apps folders. But for the server repo only the one in the root is used:

apps/accessibility/.l10nignore
apps/comments/.l10nignore
apps/files/.l10nignore
apps/files_external/.l10nignore
apps/files_sharing/.l10nignore
apps/files_trashbin/.l10nignore
apps/files_versions/.l10nignore
apps/oauth2/.l10nignore
apps/settings/.l10nignore
apps/systemtags/.l10nignore
apps/twofactor_backupcodes/.l10nignore
apps/updatenotification/.l10nignore
apps/workflowengine/.l10nignore

@MorrisJobke
Copy link
Member

Just checked the code and it should work, but it needs to hold all files and not wildcards. Let me add the wildcard for subfolders.

@MorrisJobke
Copy link
Member

See #22860

@MorrisJobke
Copy link
Member

It took this time 17 minutes instead of the typical ~5 minutes, but then it worked: 869f3f5 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending check for reproducibility or if it fits our roadmap bug high
Projects
None yet
3 participants