make update_po failed #66066

Closed
opened 2019-06-24 00:52:29 +02:00 by Hoang Duy Tran · 15 comments
Member

After performed:

svn update .

at the 'blender_docs' directory, to bringing latest changes in the documentation, if you then run

export LC_ALL=C.UTF-8; export LANG=C.UTF-8; make update_po

you will get this error:

svn: warning: W150002: '/home/htran/blender_documentations/blender_docs/locale/vi' is already under version control
svn: E200009: Could not add all targets because some targets are already versioned
svn: E200009: Illegal target for the requested operation

Error(1) on line 77, in command:
svn add $NEW_DIRS

Makefile:165: recipe for target 'update_po' failed
make: [update_po] Error 1 (ignored)

and it's down to the line above mentioned line of code 'svn add $NEW_DIRS' in the following section of code:

  • note, the Python part filters only for directories
  • there may be a cleaner way to do this in shell.
    NEW_DIRS=svn status . | grep -v -e "\.po$" | awk '/^- [x]/{print $2}' | python -c "import sys, os; sys.stdout.write('\n'.join([f for f in sys.stdin.read().split('\n') if os.path.isdir(f)]))"
    if [ "$NEW_DIRS" != "" ]; then

    Multiple args, don't quote.

    svn add $NEW_DIRS
    fi
    unset NEW_DIRS

from the file

blender_docs/tools_maintenance/update_po.sh

Printing a few debug lines out and it comes down to the

'locale/'

that was found, but this directory is already under 'svn' control for translation and it has NOTHING to do with manual repository. The following line fixed the problem:

if [ "$NEW_DIRS" != "" && "$NEW_DIRS" != "$LANG" ]; then

After performed: svn update . at the 'blender_docs' directory, to bringing latest changes in the documentation, if you then run export LC_ALL=C.UTF-8; export LANG=C.UTF-8; make update_po you will get this error: svn: warning: W150002: '/home/htran/blender_documentations/blender_docs/locale/vi' is already under version control svn: E200009: Could not add all targets because some targets are already versioned svn: E200009: Illegal target for the requested operation ``` Error(1) on line 77, in command: svn add $NEW_DIRS ``` Makefile:165: recipe for target 'update_po' failed make: [update_po] Error 1 (ignored) and it's down to the line above mentioned line of code 'svn add $NEW_DIRS' in the following section of code: - note, the Python part filters only for directories - there may be a cleaner way to do this in shell. NEW_DIRS=`svn status . | grep -v -e "\.po$" | awk '/^- [x]/{print $2}' | python -c "import sys, os; sys.stdout.write('\n'.join([f for f in sys.stdin.read().split('\n') if os.path.isdir(f)]))"` if [ "$NEW_DIRS" != "" ]; then # Multiple args, don't quote. svn add $NEW_DIRS fi unset NEW_DIRS from the file blender_docs/tools_maintenance/update_po.sh Printing a few debug lines out and it comes down to the 'locale/<language>' that was found, but this directory is already under 'svn' control for translation and it has NOTHING to do with manual repository. The following line fixed the problem: if [ "$NEW_DIRS" != "" && "$NEW_DIRS" != "$LANG" ]; then
Author
Member

Added subscriber: @hoanguk

Added subscriber: @hoanguk
Campbell Barton was assigned by Aaron Carlisle 2019-06-24 02:53:33 +02:00

Note that this is working for me, most likely a difference in my svn checkout which holds all languages. in locale/

Note that this is working for me, most likely a difference in my svn checkout which holds all languages. in `locale/`
Member

Added subscriber: @Blendify

Added subscriber: @Blendify
Member

I also have all the languages and works for me.

I also have all the languages and works for me.
Author
Member

Would that then be the location of '.svn' directory? I have this under 'locale/vi'. Yours are probably in 'locale'??? I've followed the instruction in the manual

https://docs.blender.org/manual/en/dev/about/contribute/translations/contribute.html

and clone the svn of translation as:

svn checkout https://svn.blender.org/svnroot/bf-manual-translations/trunk/blender_docs/locale/vi locale/vi

As a language translator, I don't think the existence of other languages are necessary, and thus did not download the whole set of languages to my local HDD.

Would that then be the location of '.svn' directory? I have this under 'locale/vi'. Yours are probably in 'locale'??? I've followed the instruction in the manual https://docs.blender.org/manual/en/dev/about/contribute/translations/contribute.html and clone the svn of translation as: svn checkout https://svn.blender.org/svnroot/bf-manual-translations/trunk/blender_docs/locale/vi locale/vi As a language translator, I don't think the existence of other languages are necessary, and thus did not download the whole set of languages to my local HDD.

Right, this is a problem with the script that needs to be fixed.

Right, this is a problem with the script that needs to be fixed.

This issue was referenced by 5204

This issue was referenced by 5204

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Author
Member

Thank you Campbell.

Thank you Campbell.
Author
Member

Sorry to have raised this again, tested tonight and it doesn't work, failed on the line:

svn add $NEW_FILES

Screenshot from 2019-06-29 23-54-58.png

Added for loop to perform 'svn add' on single instance of new files appeared to work:

Screenshot from 2019-06-30 00-24-31.png

Screenshot from 2019-06-30 00-24-40.png

Please correct this. Thank you.

By the way, I won't update the locale/vi repository so you can download to your local HDD to test, using:

cd blender_docs
rm -fr locale
svn checkout https://svn.blender.org/svnroot/bf-manual-translations/trunk/blender_docs/locale/vi locale/vi > /dev/null
export LC_ALL=C.UTF-8; export LANG=C.UTF-8; make update_po

Sorry to have raised this again, tested tonight and it doesn't work, failed on the line: svn add $NEW_FILES ![Screenshot from 2019-06-29 23-54-58.png](https://archive.blender.org/developer/F7556281/Screenshot_from_2019-06-29_23-54-58.png) Added for loop to perform 'svn add' on single instance of new files appeared to work: ![Screenshot from 2019-06-30 00-24-31.png](https://archive.blender.org/developer/F7556283/Screenshot_from_2019-06-30_00-24-31.png) ![Screenshot from 2019-06-30 00-24-40.png](https://archive.blender.org/developer/F7556285/Screenshot_from_2019-06-30_00-24-40.png) Please correct this. Thank you. By the way, I won't update the locale/vi repository so you can download to your local HDD to test, using: cd blender_docs rm -fr locale svn checkout https://svn.blender.org/svnroot/bf-manual-translations/trunk/blender_docs/locale/vi locale/vi > /dev/null export LC_ALL=C.UTF-8; export LANG=C.UTF-8; make update_po
Author
Member

Changed status from 'Resolved' to: 'Open'

Changed status from 'Resolved' to: 'Open'

Could you try again, this should be fixed rBM5298.

Could you try again, this should be fixed rBM5298.
Author
Member

OK, I'm waiting for some more changes on the repository to test later. I had a gut that it's down to command line buffer's limit. Look at the cut off of the last-line in the screenshot of the error.

OK, I'm waiting for some more changes on the repository to test later. I had a gut that it's down to command line buffer's limit. Look at the cut off of the last-line in the screenshot of the error.
Author
Member

Changed status from 'Open' to: 'Resolved'

Changed status from 'Open' to: 'Resolved'
Author
Member

Ran test to 'make update_po' today and it appeared working. Closing this log as resolved for the moment.

Ran test to 'make update_po' today and it appeared working. Closing this log as resolved for the moment.
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: blender/blender-manual#66066
No description provided.