search

danpovey / pocolm

Small language toolkit for creation, interpolation and pruning of ARPA language models
Other
90 stars 48 forks source link

make poco compatible with python3 #69

Closed chris920820 closed 8 years ago

chris920820 commented 8 years ago

Thanks for your comments. I will fix them tonight

best regards, Zhouyang

On Sep 20, 2016, at 6:43 PM, Daniel Povey notifications@github.com wrote:

@danpovey commented on this pull request.

In scripts/check_dependencies.sh:

@@ -40,16 +40,17 @@ if ! which awk >&/dev/null; then fi

if which python >&/dev/null ; then

  • version=python 2>&1 --version | awk '{print $2}'
  • if [[ $version != "2."* ]] ; then
  • if which python2.7 >&/dev/null || which python2 >&/dev/null ; then
  • echo "$0: python 2.x is not the default python. You should either make it"
  • echo "$0: default or create an bash alias for kaldi scripts to run correctly"
  • version=/usr/bin/env python 2>&1 --version | awk '{print $2}'
  • if [[ $version != "2.7"* && $version != "3."* ]] ; then
  • if which python2.7 >&/dev/null ; then
  • echo "$0: python 2.7 is not the default python (lower version python does not "
  • echo "$0: have packages that are required by pocolm). You should make it default" you should set error status on this branch. you may have to move 'status=0' higher in the script.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.