Changeset 2718

Show
Ignore:
Timestamp:
11/08/17 02:45:42 (1 month ago)
Author:
jwoithe
Message:

Additional scons3 compatibility fixes (add implicit bytes/string conversions).

Within the scons scripts, some of the env[VAR] calls return bytes under
scons3. Implicit conversion of bytes to strings has been removed in python3
which necessitates the use of decode() in some instances which expect
strings to prevent type mismatch errors. This change does not affect
compatibility with scons2.

Thanks to Orcan Ogetbil for his scons3 tests and the patch (via the
ffado-devel mailing list).

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved
  • trunk/libffado/SConstruct

    r2717 r2718  
    229229    sys.stdout.write('Checking jackd version...') 
    230230    sys.stdout.flush() 
    231     ret = Popen("which jackd >/dev/null 2>&1 && jackd --version | tail -n 1 | cut -d ' ' -f 3", shell=True, stdout=PIPE).stdout.read()[:-1] 
     231    ret = Popen("which jackd >/dev/null 2>&1 && jackd --version | tail -n 1 | cut -d ' ' -f 3", shell=True, stdout=PIPE).stdout.read()[:-1].decode() 
    232232    if (ret == ""): 
    233233        print("not installed") 
     
    714714        # /bin/mount:     file format elf32-powerpc 
    715715        for line in x.split(b'\n'): 
    716             line = line.strip() 
     716            line = line.strip().decode() 
    717717            if line.startswith(real_exe): 
    718718                x, fmt = line.rsplit(None, 1) 
  • trunk/libffado/src/SConscript

    r2650 r2718  
    285285if not env.GetOption( "clean" ): 
    286286        libenv.MergeFlags( "-lrt -lpthread" ) 
    287         libenv.MergeFlags( env['LIBRAW1394_FLAGS']
    288         libenv.MergeFlags( env['LIBIEC61883_FLAGS']
    289         libenv.MergeFlags( env['LIBCONFIG_FLAGS']
     287        libenv.MergeFlags( env['LIBRAW1394_FLAGS'].decode()
     288        libenv.MergeFlags( env['LIBIEC61883_FLAGS'].decode()
     289        libenv.MergeFlags( env['LIBCONFIG_FLAGS'].decode()
    290290        if not env['SERIALIZE_USE_EXPAT']: 
    291291            if 'LIBXML30_FLAGS' in env : 
    292                 libenv.MergeFlags( env['LIBXML30_FLAGS']
     292                libenv.MergeFlags( env['LIBXML30_FLAGS'].decode()
    293293            if not('LIBXML30_FLAGS' in env) : 
    294                 libenv.MergeFlags( env['LIBXML26_FLAGS']
     294                libenv.MergeFlags( env['LIBXML26_FLAGS'].decode()
    295295        else: 
    296296                libenv.PrependUnique( LIBS=["expat"] ) 
    297297                libenv.MergeFlags( "-DSERIALIZE_USE_EXPAT" ) 
    298298        if env['REQUIRE_LIBAVC']: 
    299                 libenv.MergeFlags( env['LIBAVC1394_FLAGS']
     299                libenv.MergeFlags( env['LIBAVC1394_FLAGS'].decode()
    300300 
    301301libname_versioned = "libffado.so.%s" % libenv['VERSION'] 
  • trunk/libffado/support/alsa/SConscript

    r2714 r2718  
    3838 
    3939if 'ALSA_FLAGS' in env and env['ALSA_FLAGS']: 
    40     env.MergeFlags( env["ALSA_FLAGS"]
     40    env.MergeFlags( env["ALSA_FLAGS"].decode()
    4141    env.MergeFlags( "-DPIC" ) 
    4242    alsaplugin = env.SharedLibrary( "asound_module_pcm_ffado", sources ) 
  • trunk/libffado/support/dbus/SConscript

    r2711 r2718  
    3838 
    3939if not env.GetOption( "clean" ): 
    40     env.MergeFlags( env["DBUS1_FLAGS"]
    41     env.MergeFlags( env["DBUSC1_FLAGS"]
    42     env.MergeFlags( env['LIBRAW1394_FLAGS']
     40    env.MergeFlags( env["DBUS1_FLAGS"].decode()
     41    env.MergeFlags( env["DBUSC1_FLAGS"].decode()
     42    env.MergeFlags( env['LIBRAW1394_FLAGS'].decode()
    4343    if not env['SERIALIZE_USE_EXPAT']: 
    4444        if 'LIBXML30_FLAGS' in env : 
    45             env.MergeFlags( env['LIBXML30_FLAGS']
     45            env.MergeFlags( env['LIBXML30_FLAGS'].decode()
    4646        if not('LIBXML30_FLAGS' in env) : 
    47             env.MergeFlags( env['LIBXML26_FLAGS']
     47            env.MergeFlags( env['LIBXML26_FLAGS'].decode()
    4848    else: 
    4949        env.PrependUnique( LIBS=["expat"] ) 
     
    8484if env['dbus_service_dir'] and ( env.destdir or os.access( env['dbus_service_dir'], os.W_OK ) ): 
    8585    print("Will install the service-file") 
    86     targetdir = env.destdir + env['dbus_service_dir'] 
    87     env.Alias( "install", env.Install( env.destdir + env['dbus_service_dir'], servicefile ) ) 
     86    targetdir = env.destdir + env['dbus_service_dir'].decode() 
     87    env.Alias( "install", env.Install( env.destdir + env['dbus_service_dir'].decode(), servicefile ) ) 
    8888else: 
    8989    if not env['dbus_service_dir']: 
  • trunk/libffado/support/firmware/SConscript

    r2650 r2718  
    3131if not env.GetOption( "clean" ): 
    3232    env.MergeFlags( "-lrt -lpthread" ) 
    33     env.MergeFlags( env['LIBRAW1394_FLAGS']
    34     env.MergeFlags( env['LIBIEC61883_FLAGS']
     33    env.MergeFlags( env['LIBRAW1394_FLAGS'].decode()
     34    env.MergeFlags( env['LIBIEC61883_FLAGS'].decode()
    3535    if not env['SERIALIZE_USE_EXPAT']: 
    3636        if 'LIBXML30_FLAGS' in env : 
    37             env.MergeFlags( env['LIBXML30_FLAGS']
     37            env.MergeFlags( env['LIBXML30_FLAGS'].decode()
    3838        if not('LIBXML30_FLAGS' in env) : 
    39             env.MergeFlags( env['LIBXML26_FLAGS']
     39            env.MergeFlags( env['LIBXML26_FLAGS'].decode()
    4040    else: 
    4141        env.PrependUnique( LIBS=["expat"] ) 
  • trunk/libffado/support/tools/SConscript

    r2650 r2718  
    3535        if not env['SERIALIZE_USE_EXPAT']: 
    3636            if 'LIBXML30_FLAGS' in env : 
    37                 e.MergeFlags( env['LIBXML30_FLAGS']
     37                e.MergeFlags( env['LIBXML30_FLAGS'].decode()
    3838            if not('LIBXML30_FLAGS' in env) : 
    39                 e.MergeFlags( env['LIBXML26_FLAGS']
     39                e.MergeFlags( env['LIBXML26_FLAGS'].decode()
    4040        else: 
    4141                e.PrependUnique( LIBS=["expat"] ) 
  • trunk/libffado/tests/SConscript

    r2714 r2718  
    3030if not env.GetOption( "clean" ): 
    3131        env.MergeFlags( "-lpthread" ) 
    32         env.MergeFlags( env['LIBIEC61883_FLAGS']
    33         env.MergeFlags( env['LIBRAW1394_FLAGS']
     32        env.MergeFlags( env['LIBIEC61883_FLAGS'].decode()
     33        env.MergeFlags( env['LIBRAW1394_FLAGS'].decode()
    3434        if not env['SERIALIZE_USE_EXPAT']: 
    3535            if 'LIBXML30_FLAGS' in env : 
    36                 env.MergeFlags( env['LIBXML30_FLAGS']
     36                env.MergeFlags( env['LIBXML30_FLAGS'].decode()
    3737            if not('LIBXML30_FLAGS' in env) : 
    38                 env.MergeFlags( env['LIBXML26_FLAGS']
     38                env.MergeFlags( env['LIBXML26_FLAGS'].decode()
    3939        else: 
    4040                env.PrependUnique( LIBS=["expat"] ) 
     
    6969if env['ENABLE_GENERICAVC']: 
    7070        if 'ALSA_FLAGS' in env and env["ALSA_FLAGS"]: 
    71                 env.MergeFlags( env["ALSA_FLAGS"]
     71                env.MergeFlags( env["ALSA_FLAGS"].decode()
    7272                apps.update( { "test-scs" : "test-scs.cpp" } ) 
    7373        apps.update( { "test-volume" : "test-volume.cpp" } ) 
  • trunk/libffado/tests/systemtests/SConscript

    r1777 r2718  
    3131 
    3232if not env.GetOption( "clean" ): 
    33         env.MergeFlags( env['LIBRAW1394_FLAGS']
     33        env.MergeFlags( env['LIBRAW1394_FLAGS'].decode()
    3434        env.MergeFlags( "-lrt -lpthread" ) 
    3535