Project

General

Profile

Bug #15408

Feature #10034: Translation web platform

Feature #15079: Integrate the platform with our Git and ikiwiki infrastructure

Consider forcing wrapping of po files at 79 chars per line

Added by u over 1 year ago. Updated 5 months ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
03/13/2018
Due date:
% Done:

100%

Feature Branch:
tails.git:translation_platform_hooks
Type of work:
Code
Blueprint:
Starter:
Affected tool:

Description

Whenever one opens a po file in poedit, lines may get rewrapped without any real change within the file. We should consider unifying at how many characters this should happen to avoid senseless commits.


Related issues

Related to Tails - Bug #15605: Make check_po optionally accept a list of files Resolved 05/17/2018
Related to Tails - Bug #15819: Make sure ikiwiki generated PO files satisfy the requirements we want check_po to set Resolved 08/19/2018
Related to Tails - Feature #16102: List of potential checks we might want to do on PO files Resolved 11/05/2018

History

#1 Updated by u over 1 year ago

  • Related to Feature #15364: Create .gitattributes with merge strategy for po files added

#2 Updated by u over 1 year ago

  • pre-commit git hook for people
  • main git pre-receive hook

#3 Updated by bertagaz over 1 year ago

  • Target version changed from Tails_3.7 to Tails_3.8

#4 Updated by u over 1 year ago

This should go into check_po.sh

#5 Updated by u over 1 year ago

https://github.com/translate/translate/commit/5a484fffd26b83aa1bab299583cd17cf8576687a Translate-toolkit, as used by Weblate now supports telling it a width to wrap lines. We will need to do that whenever Weblate permits it to wrap to 79 chars in Weblate.

At the same time, we will want to verify this with check_po and allow translators who use Git to rewrap their files before committing them.

Furthermore, I believe we will need to patch Ikiwiki's Pot file generation to do that too, or make it happen within our own infrastructure. --> actually we will probably not need this.

#6 Updated by u over 1 year ago

  • Related to deleted (Feature #15364: Create .gitattributes with merge strategy for po files)

#7 Updated by u over 1 year ago

  • Type of work changed from Sysadmin to Code

#8 Updated by u over 1 year ago

  • Related to Bug #15605: Make check_po optionally accept a list of files added

#9 Updated by u over 1 year ago

  • Status changed from Confirmed to In Progress

This needs to go into check_po.sh

#!/bin/sh
# Check that po files only have 79 chars per line
for FILENAME in FILELIST; do
    msgcat -w 79 $FILENAME -o "$FILENAME.new" 
    if ! diff -q $FILENAME "$FILENAME.new"; then
        echo "$FILENAME needs to be rewrapped to 79 chars." 
        exit 1
    fi
    rm "$FILENAME.new" 
done

#10 Updated by u over 1 year ago

We've implemented this differently.

#11 Updated by u over 1 year ago

  • % Done changed from 0 to 50
  • Feature Branch set to tails.git:translation_platform_hooks

#12 Updated by u over 1 year ago

I've added this to unify_po_headers and it is already in check_po in the abovementioned branch. All of this is waiting for tester feedback currently.

#13 Updated by intrigeri over 1 year ago

  • Target version changed from Tails_3.8 to Tails_3.9

#14 Updated by u about 1 year ago

  • Target version changed from Tails_3.9 to Tails_3.10.1

Won't be able to do this in the next 48 hours (release of 3.9)

#15 Updated by intrigeri about 1 year ago

  • Target version changed from Tails_3.10.1 to Tails_3.11

#16 Updated by u about 1 year ago

  • Related to Bug #15819: Make sure ikiwiki generated PO files satisfy the requirements we want check_po to set added

#17 Updated by u about 1 year ago

  • Assignee changed from u to hefee

@hefee would you be able to review this in order to move this forward?

#18 Updated by hefee about 1 year ago

  • Related to Feature #16102: List of potential checks we might want to do on PO files added

#19 Updated by hefee about 1 year ago

  • Status changed from In Progress to 11
  • % Done changed from 50 to 100

It is implemented in check_po and unify_po and are checked correctly.

#20 Updated by CyrilBrulebois 11 months ago

  • Status changed from 11 to Resolved

#21 Updated by hefee 11 months ago

  • Status changed from Resolved to In Progress
  • Target version changed from Tails_3.11 to Tails_3.12
  • QA Check set to Ready for QA

I used Fix commited as status, as it is commit on a local branch, that will together with other task review together. I use now the more general "In Progress" for the moment.

#22 Updated by intrigeri 11 months ago

  • Blocks Feature #15082: Have the Weblate Git communicate with our main Git repository added

#23 Updated by intrigeri 11 months ago

  • Blocks deleted (Feature #15082: Have the Weblate Git communicate with our main Git repository)

#24 Updated by anonym 10 months ago

  • Target version changed from Tails_3.12 to Tails_3.13

#25 Updated by u 10 months ago

  • Parent task changed from #10034 to #15079

@hefee: is this ticket still needed? if yes, please assign to enrico for review, making sure the branches are up to date with current master. thanks.

#26 Updated by hefee 10 months ago

  • Status changed from In Progress to Rejected
  • Assignee deleted (hefee)
  • QA Check deleted (Ready for QA)

We won't need to force users to wrap to 79 characters anymore, so I'll close this.

Also available in: Atom PDF