Changeset 5338089 in rtems-libbsd


Ignore:
Timestamp:
05/04/16 06:24:11 (7 years ago)
Author:
Chris Johns <chrisj@…>
Branches:
5, 5-freebsd-12, 6-freebsd-12, freebsd-9.3, master
Children:
e58b898
Parents:
4b127e7
Message:

Fix coding to be CamelCase.

Files:
3 edited

Legend:

Unmodified
Added
Removed

  • builder.py

    r4b127e7 r5338089  
    7272                print(' %s' % (f))
    7373
    74 def read_file(name):
     74def readFile(name):
    7575    try:
    7676        contents = codecs.open(name, mode = 'r', encoding = 'utf-8', errors = 'ignore').read()
    … …  
    8080    return contents
    8181
    82 def write_file(name, contents):
     82def writeFile(name, contents):
    8383    path = os.path.dirname(name)
    8484    if not os.path.exists(path):
    … …  
    8686            os.makedirs(path)
    8787        except OSError as oe:
    88             print('error: cannot create directory: %s: %s' % (dst_path, oe))
     88            print('error: cannot create directory: %s: %s' % (path, oe))
    8989            sys.exit(1)
    9090    try:
    … …  
    109109# This stuff needs to move to libbsd.py.
    110110#
    111 def common_flags():
     111def commonFlags():
    112112    return ['-O2',
    113113            '-g',
    … …  
    116116            '-fno-common']
    117117
    118 def common_warnings():
     118def commonWarnings():
    119119    return ['-Wall',
    120120            '-Wno-format']
    121121
    122 def common_no_warnings():
     122def commonNoWarnings():
    123123    return ['-w']
    124124
    … …  
    143143            '-Itestsuite/include']
    144144
    145 def cpu_includes():
     145def cpuIncludes():
    146146    return ['-Irtemsbsd/@CPU@/include',
    147147            '-Ifreebsd/sys/@CPU@/include']
    … …  
    153153    return ['-std=gnu++11']
    154154
    155 def header_paths():
     155def headerPaths():
    156156    #         local path                      wildcard             dest path
    157157    return [('rtemsbsd/include',              '*.h',               ''),
    … …  
    233233class Converter(object):
    234234
    235     def convert(self, src, dst, has_source = True, source_filter = None, src_contents = None):
     235    def convert(self, src, dst, hasSource = True, sourceFilter = None, srcContents = None):
    236236
    237237        global filesProcessed, filesProcessedCount
    … …  
    239239        if verbose(verboseDebug):
    240240            print("convert: filter:%s: %s -> %s" % \
    241                   (['yes', 'no'][source_filter is None], src, dst))
     241                  (['yes', 'no'][sourceFilter is None], src, dst))
    242242
    243243        #
    … …  
    245245        # warning and do not try and convert.
    246246        #
    247         if src_contents is None:
     247        if srcContents is None:
    248248            if not os.path.exists(src):
    249                 if has_source:
     249                if hasSource:
    250250                    raise error('source not found: %s' % (src))
    251251                else:
    … …  
    256256            # Files read as a single string if not passed in.
    257257            #
    258             src_contents = read_file(src)
     258            srcContents = readFile(src)
    259259
    260260        if os.path.exists(dst):
    261             dst_contents = read_file(dst)
     261            dstContents = readFile(dst)
    262262        else:
    263263            print('warning: no destination: %s' % (dst))
    264             dst_contents = ''
     264            dstContents = ''
    265265
    266266        #
    267267        # Filter the source.
    268268        #
    269         if source_filter is not None:
    270             src_contents = source_filter(src_contents)
     269        if sourceFilter is not None:
     270            srcContents = sourceFilter(srcContents)
    271271
    272272        #
    273273        # Split into a list of lines.
    274274        #
    275         src_lines = src_contents.split(os.linesep)
    276         dst_lines = dst_contents.split(os.linesep)
     275        srcLines = srcContents.split(os.linesep)
     276        dstLines = dstContents.split(os.linesep)
    277277
    278278        if verbose(verboseDebug):
    279             print('Unified diff: %s (lines:%d)' % (src, len(src_lines)))
     279            print('Unified diff: %s (lines:%d)' % (src, len(srcLines)))
    280280
    281281        #
    … …  
    283283        # generator to list because the generator is not reusable.
    284284        #
    285         diff = list(difflib.unified_diff(dst_lines,
    286                                          src_lines,
     285        diff = list(difflib.unified_diff(dstLines,
     286                                         srcLines,
    287287                                         fromfile = src,
    288288                                         tofile = dst,
    … …  
    304304                    print("UPDATE: %s -> %s" % (src, dst))
    305305                if isDryRun == False:
    306                     write_file(dst, src_contents)
     306                    writeFile(dst, srcContents)
    307307            else:
    308308                print("diff -u %s %s" % (src, dst))
    … …  
    311311
    312312class NoConverter(Converter):
    313     def convert(self, src, dst, has_source = True, source_filter = None):
     313    def convert(self, src, dst, hasSource = True, sourceFilter = None):
    314314        return '/* EMPTY */\n'
    315315
    316316class FromFreeBSDToRTEMSHeaderConverter(Converter):
    317     def source_filter(self, data):
     317    def sourceFilter(self, data):
    318318        data = fixLocalIncludes(data)
    319319        data = fixIncludes(data)
    … …  
    322322    def convert(self, src, dst):
    323323        sconverter = super(FromFreeBSDToRTEMSHeaderConverter, self)
    324         sconverter.convert(src, dst, source_filter = self.source_filter)
     324        sconverter.convert(src, dst, sourceFilter = self.sourceFilter)
    325325
    326326class FromFreeBSDToRTEMSUserSpaceHeaderConverter(Converter):
    327     def source_filter(self, data):
     327    def sourceFilter(self, data):
    328328        data = fixIncludes(data)
    329329        return data
    … …  
    331331    def convert(self, src, dst):
    332332        sconverter = super(FromFreeBSDToRTEMSUserSpaceHeaderConverter, self)
    333         sconverter.convert(src, dst, source_filter = self.source_filter)
     333        sconverter.convert(src, dst, sourceFilter = self.sourceFilter)
    334334
    335335class FromFreeBSDToRTEMSSourceConverter(Converter):
    336     def source_filter(self, data):
     336    def sourceFilter(self, data):
    337337        data = fixLocalIncludes(data)
    338338        data = fixIncludes(data)
    … …  
    342342    def convert(self, src, dst):
    343343        sconverter = super(FromFreeBSDToRTEMSSourceConverter, self)
    344         sconverter.convert(src, dst, source_filter = self.source_filter)
     344        sconverter.convert(src, dst, sourceFilter = self.sourceFilter)
    345345
    346346class FromFreeBSDToRTEMSUserSpaceSourceConverter(Converter):
    347     def source_filter(self, data):
     347    def sourceFilter(self, data):
    348348        data = fixIncludes(data)
    349349        data = '#include <machine/rtems-bsd-user-space.h>\n\n' + data
    … …  
    352352    def convert(self, src, dst):
    353353        sconverter = super(FromFreeBSDToRTEMSUserSpaceSourceConverter, self)
    354         sconverter.convert(src, dst, source_filter = self.source_filter)
     354        sconverter.convert(src, dst, sourceFilter = self.sourceFilter)
    355355
    356356class FromRTEMSToFreeBSDHeaderConverter(Converter):
    357     def source_filter(self, data):
     357    def sourceFilter(self, data):
    358358        data = revertFixLocalIncludes(data)
    359359        data = revertFixIncludes(data)
    … …  
    362362    def convert(self, src, dst):
    363363        sconverter = super(FromRTEMSToFreeBSDHeaderConverter, self)
    364         sconverter.convert(src, dst, has_source = False,  source_filter = self.source_filter)
     364        sconverter.convert(src, dst, hasSource = False,  sourceFilter = self.sourceFilter)
    365365
    366366class FromRTEMSToFreeBSDSourceConverter(Converter):
    367     def source_filter(self, data):
     367    def sourceFilter(self, data):
    368368        data = re.sub('#include <machine/rtems-bsd-kernel-space.h>\n\n', '', data)
    369369        data = re.sub('#include <machine/rtems-bsd-user-space.h>\n\n', '', data)
    … …  
    372372    def convert(self, src, dst):
    373373        sconverter = super(FromRTEMSToFreeBSDSourceConverter, self)
    374         sconverter.convert(src, dst, has_source = False, source_filter = self.source_filter)
     374        sconverter.convert(src, dst, hasSource = False, sourceFilter = self.sourceFilter)
    375375
    376376#

  • freebsd-to-rtems.py

    r4b127e7 r5338089  
    148148
    149149try:
    150     waf_gen = waf_generator.ModuleManager()
    151     libbsd.sources(waf_gen)
     150    wafGen = waf_generator.ModuleManager()
     151    libbsd.sources(wafGen)
    152152    if not isOnlyBuildScripts:
    153         waf_gen.processSource(isForward)
    154     waf_gen.generate(libbsd.rtems_version())
     153        wafGen.processSource(isForward)
     154    wafGen.generate(libbsd.rtems_version())
    155155    builder.changedFileSummary()
    156156except IOError as ioe:

  • waf_generator.py

    r4b127e7 r5338089  
    4545data = { }
    4646
    47 def _add_files(name, files):
     47def _addFiles(name, files):
    4848    if type(files) is not list:
    4949        files = [files]
    … …  
    5252    data[name] += files
    5353
    54 def _clfags_includes(cflags, includes):
     54def _cflagsIncludes(cflags, includes):
    5555    if type(cflags) is not list:
    5656        if cflags is not None:
    … …  
    6969
    7070    def __init__(self, cflags = "default", includes = None):
    71         self.cflags, self.includes = _clfags_includes(cflags, includes)
     71        self.cflags, self.includes = _cflagsIncludes(cflags, includes)
    7272
    7373    def compose(self, path):
    … …  
    109109        self.sym = sym
    110110        self.dep = dep
    111         self.cflags, self.includes = _clfags_includes(cflags, includes)
     111        self.cflags, self.includes = _cflagsIncludes(cflags, includes)
    112112
    113113    def compose(self, path):
    … …  
    126126        self.sym = sym
    127127        self.header = header
    128         self.cflags, self.includes = _clfags_includes(cflags, includes)
     128        self.cflags, self.includes = _cflagsIncludes(cflags, includes)
    129129
    130130    def compose(self, path):
    … …  
    150150        name = os.path.join(builder.RTEMS_DIR, 'libbsd_waf.py')
    151151        converter = builder.Converter()
    152         converter.convert(name, name, src_contents = self.script)
     152        converter.convert(name, name, srcContents = self.script)
    153153
    154154    def setGenerators(self):
    … …  
    174174    def generate(self, rtems_version):
    175175
    176         def _source_list(lhs, files, append = False):
     176        def _sourceList(lhs, files, append = False):
    177177            if append:
    178178                adder = '+'
    179                 adder_space = ' '
     179                adderSpace = ' '
    180180            else:
    181181                adder = ''
    182                 adder_space = ''
     182                adderSpace = ''
    183183            ll = len(lhs)
    184184            if len(files) == 1:
    … …  
    186186            elif len(files) == 2:
    187187                self.add('%s %s= [%r,' % (lhs, adder, files[0]))
    188                 self.add('%s %s   %r]' % (' ' * ll, adder_space, files[-1]))
     188                self.add('%s %s   %r]' % (' ' * ll, adderSpace, files[-1]))
    189189            elif len(files) > 0:
    190190                self.add('%s %s= [%r,' % (lhs, adder, files[0]))
    191191                for f in files[1:-1]:
    192                     self.add('%s %s   %r,' % (' ' * ll, adder_space, f))
    193                 self.add('%s %s   %r]' % (' ' * ll, adder_space, files[-1]))
    194 
    195         def _data_insert(data, cpu, frag):
     192                    self.add('%s %s   %r,' % (' ' * ll, adderSpace, f))
     193                self.add('%s %s   %r]' % (' ' * ll, adderSpace, files[-1]))
     194
     195        def _dataInsert(data, cpu, frag):
    196196            #
    197197            # The default handler returns an empty string. Skip it.
    … …  
    234234            if m.conditionalOn == "none":
    235235                for f in m.files:
    236                     _data_insert(data, 'all', f.getFragment())
     236                    _dataInsert(data, 'all', f.getFragment())
    237237            for cpu, files in sorted(m.cpuDependentSourceFiles.items()):
    238238                for f in files:
    239                     _data_insert(data, cpu, f.getFragment())
     239                    _dataInsert(data, cpu, f.getFragment())
    240240
    241241        if trace:
    … …  
    273273        self.add('    # C/C++ flags')
    274274        self.add('    common_flags = []')
    275         for f in builder.common_flags():
     275        for f in builder.commonFlags():
    276276            self.add('    common_flags += ["%s"]' % (f))
    277277        self.add('    if bld.env.WARNINGS:')
    278         for f in builder.common_warnings():
     278        for f in builder.commonWarnings():
    279279            self.add('        common_flags += ["%s"]' % (f))
    280280        self.add('    else:')
    281         for f in builder.common_no_warnings():
     281        for f in builder.commonNoWarnings():
    282282            self.add('        common_flags += ["%s"]' % (f))
    283283        self.add('    cflags = %r + common_flags' % (builder.cflags()))
    … …  
    292292        self.add('    # Include paths')
    293293        self.add('    includes = []')
    294         self.add('    for i in %r:' % (builder.cpu_includes()))
     294        self.add('    for i in %r:' % (builder.cpuIncludes()))
    295295        self.add('        includes += ["%s" % (i[2:].replace("@CPU@", bld.get_env()["RTEMS_ARCH"]))]')
    296296        self.add('    if bld.get_env()["RTEMS_ARCH"] == "i386":')
    297         self.add('        for i in %r:' % (builder.cpu_includes()))
     297        self.add('        for i in %r:' % (builder.cpuIncludes()))
    298298        self.add('            includes += ["%s" % (i[2:].replace("@CPU@", "x86"))]')
    299299        for i in builder.includes():
    … …  
    403403                lex = lexes[l]['all']
    404404                if 'cflags' in lex:
    405                     lex_defines = [d[2:] for d in lex['cflags']]
    406                 else:
    407                     lex_defines = []
     405                    lexDefines = [d[2:] for d in lex['cflags']]
     406                else:
     407                    lexDefines = []
    408408                if 'includes' in lex:
    409                     lex_includes = lex['includes']
    410                 else:
    411                     lex_includes = []
     409                    lexIncludes = lex['includes']
     410                else:
     411                    lexIncludes = []
    412412                self.add('    if bld.env.AUTO_REGEN:')
    413413                self.add('        bld(target = "%s.c",' % (lex['file'][:-2]))
    … …  
    418418                self.add('                features = "c",')
    419419                self.add('                cflags = cflags,')
    420                 self.add('                includes = %r + includes,' % (lex_includes))
    421                 self.add('                defines = defines + %r,' % (lex_defines))
     420                self.add('                includes = %r + includes,' % (lexIncludes))
     421                self.add('                defines = defines + %r,' % (lexDefines))
    422422                self.add('                source = "%s.c")' % (lex['file'][:-2]))
    423423                self.add('    libbsd_use += ["lex_%s"]' % (lex['sym']))
    … …  
    429429            for y in sorted(yaccs.keys()):
    430430                yacc = yaccs[y]['all']
    431                 yacc_file = yacc['file']
     431                yaccFile = yacc['file']
    432432                if yacc['sym'] is not None:
    433                     yacc_sym = yacc['sym']
    434                 else:
    435                     yacc_sym = os.path.basename(yacc_file)[:-2]
    436                 yacc_header = '%s/%s' % (os.path.dirname(yacc_file), yacc['header'])
     433                    yaccSym = yacc['sym']
     434                else:
     435                    yaccSym = os.path.basename(yaccFile)[:-2]
     436                yaccHeader = '%s/%s' % (os.path.dirname(yaccFile), yacc['header'])
    437437                if 'cflags' in yacc:
    438                     yacc_defines = [d[2:] for d in yacc['cflags']]
    439                 else:
    440                     yacc_defines = []
     438                    yaccDefines = [d[2:] for d in yacc['cflags']]
     439                else:
     440                    yaccDefines = []
    441441                if 'includes' in yacc:
    442                     yacc_includes = yacc['includes']
    443                 else:
    444                     yacc_includes = []
     442                    yaccIncludes = yacc['includes']
     443                else:
     444                    yaccIncludes = []
    445445                self.add('    if bld.env.AUTO_REGEN:')
    446                 self.add('        bld(target = "%s.c",' % (yacc_file[:-2]))
    447                 self.add('            source = "%s",' % (yacc_file))
    448                 self.add('            rule = "${YACC} -b %s -d -p %s ${SRC} && ' % (yacc_sym, yacc_sym) + \
    449                          'sed -e \'/YY_BUF_SIZE/s/16384/1024/\' < %s.tab.c > ${TGT} && ' % (yacc_sym) + \
    450                          'rm -f %s.tab.c && mv %s.tab.h %s")' % (yacc_sym, yacc_sym, yacc_header))
    451                 self.add('    bld.objects(target = "yacc_%s",' % (yacc_sym))
     446                self.add('        bld(target = "%s.c",' % (yaccFile[:-2]))
     447                self.add('            source = "%s",' % (yaccFile))
     448                self.add('            rule = "${YACC} -b %s -d -p %s ${SRC} && ' % (yaccSym, yaccSym) + \
     449                         'sed -e \'/YY_BUF_SIZE/s/16384/1024/\' < %s.tab.c > ${TGT} && ' % (yaccSym) + \
     450                         'rm -f %s.tab.c && mv %s.tab.h %s")' % (yaccSym, yaccSym, yaccHeader))
     451                self.add('    bld.objects(target = "yacc_%s",' % (yaccSym))
    452452                self.add('                features = "c",')
    453453                self.add('                cflags = cflags,')
    454                 self.add('                includes = %r + includes,' % (yacc_includes))
    455                 self.add('                defines = defines + %r,' % (yacc_defines))
    456                 self.add('                source = "%s.c")' % (yacc_file[:-2]))
    457                 self.add('    libbsd_use += ["yacc_%s"]' % (yacc_sym))
     454                self.add('                includes = %r + includes,' % (yaccIncludes))
     455                self.add('                defines = defines + %r,' % (yaccDefines))
     456                self.add('                source = "%s.c")' % (yaccFile[:-2]))
     457                self.add('    libbsd_use += ["yacc_%s"]' % (yaccSym))
    458458            self.add('')
    459459
    … …  
    467467            if flags is not 'default':
    468468                objs += 1
    469                 _source_list('    objs%02d_source' % objs, sorted(data['sources'][flags]['all']))
     469                _sourceList('    objs%02d_source' % objs, sorted(data['sources'][flags]['all']))
    470470                archs = sorted(data['sources'][flags])
    471471                for arch in archs:
    472472                    if arch not in ['all', 'cflags', 'includes']:
    473473                        self.add('    if bld.get_env()["RTEMS_ARCH"] == "%s":' % arch)
    474                         _source_list('        objs%02d_source' % objs,
    475                                      sorted(data['sources'][flags][arch]),
    476                                      append = True)
     474                        _sourceList('        objs%02d_source' % objs,
     475                                    sorted(data['sources'][flags][arch]),
     476                                    append = True)
    477477                if 'cflags' in data['sources'][flags]:
    478478                    defines = [d[2:] for d in data['sources'][flags]['cflags']]
    … …  
    496496        # static library with.
    497497        #
    498         _source_list('    source', sorted(data['sources']['default']['all']))
     498        _sourceList('    source', sorted(data['sources']['default']['all']))
    499499        archs = sorted(data['sources']['default'])
    500500        for arch in archs:
    501501            if arch is not 'all':
    502502                self.add('    if bld.get_env()["RTEMS_ARCH"] == "%s":' % arch)
    503                 _source_list('        source',
    504                              sorted(data['sources']['default'][arch]),
    505                              append = True)
     503                _sourceList('        source',
     504                            sorted(data['sources']['default'][arch]),
     505                            append = True)
    506506        self.add('    bld.stlib(target = "bsd",')
    507507        self.add('              features = "c cxx",')
    … …  
    519519        self.add('    # Installs.    ')
    520520        self.add('    bld.install_files("${PREFIX}/" + rtems.arch_bsp_lib_path(bld.env.RTEMS_VERSION, bld.env.RTEMS_ARCH_BSP), ["libbsd.a"])')
    521         header_paths = builder.header_paths()
    522         self.add('    header_paths = [%s,' % (str(header_paths[0])))
    523         for hp in header_paths[1:-1]:
     521        headerPaths = builder.headerPaths()
     522        self.add('    header_paths = [%s,' % (str(headerPaths[0])))
     523        for hp in headerPaths[1:-1]:
    524524            self.add('                     %s,' % (str(hp)))
    525         self.add('                     %s]' % (str(header_paths[-1])))
     525        self.add('                     %s]' % (str(headerPaths[-1])))
    526526        self.add('    for headers in header_paths:')
    527527        self.add('        ipath = os.path.join(rtems.arch_bsp_include_path(bld.env.RTEMS_VERSION, bld.env.RTEMS_ARCH_BSP), headers[2])')
    … …  
    535535        self.add('    # Tests')
    536536        tests = data['tests']
    537         for test_name in sorted(tests):
    538             files = ['testsuite/%s/%s.c' % (test_name, f) for f in  data['tests'][test_name]['all']['files']]
    539             _source_list('    test_%s' % (test_name), sorted(files))
    540             self.add('    bld.program(target = "%s.exe",' % (test_name))
     537        for testName in sorted(tests):
     538            files = ['testsuite/%s/%s.c' % (testName, f) for f in  data['tests'][testName]['all']['files']]
     539            _sourceList('    test_%s' % (testName), sorted(files))
     540            self.add('    bld.program(target = "%s.exe",' % (testName))
    541541            self.add('                features = "cprogram",')
    542542            self.add('                cflags = cflags,')
    543543            self.add('                includes = includes,')
    544             self.add('                source = test_%s,' % (test_name))
     544            self.add('                source = test_%s,' % (testName))
    545545            self.add('                use = ["bsd"],')
    546546            self.add('                lib = ["m", "z"],')
Note: See TracChangeset for help on using the changeset viewer.