new config version skew problem

[Greg Watson]

I found the problem that was causing the error with python 2.1. 
Please test new version.

Greg

Greg Watson  writes:
> I found the problem that was causing the error with python 2.1. Please test new
> version.

There is progress.  What I get now with python2.1 is:

Configuring TARGET hdama
Will place Makefile, crt0.S, etc. in hdama
Configuring ROMIMAGE normal
Configuring DIR /config/Config.lb
Configuring DIR /lib/Config.lb
Configuring DIR /console/Config.lb
Configuring DIR /stream/Config.lb
Configuring DIR /devices/Config.lb
Configuring DIR /pc80/Config.lb
Configuring DIR /boot/Config.lb
Configuring PART mainboard, path arima/hdama
Configuring PART arch, path i386
Adding init file: config/crt0.base
Configuring DIR lib/Config.lb
Configuring DIR boot/Config.lb
Configuring DIR smp/Config.lb
End PART arch
===> WARNING: Option CONFIG_SMP using default value 0
Traceback (most recent call last):
  File "./freebios2/util/newconfig/config.py", line 2155, in ?
    if (not parse('board', fp.read())):
  File "./freebios2/util/newconfig/config.py", line 1800, in parse
    return wrap_error_reporter(P, rule)
  File "./freebios2/util/newconfig/yappsrt.py", line 162, in wrap_error_reporter
    try: return getattr(parser, rule)()
  File "./freebios2/util/newconfig/config.py", line 1734, in board
    romstmts = self.romstmts()
  File "./freebios2/util/newconfig/config.py", line 1719, in romstmts
    romimage = self.romimage()
  File "./freebios2/util/newconfig/config.py", line 1698, in romimage
    romstmt = self.romstmt(1)
  File "./freebios2/util/newconfig/config.py", line 1689, in romstmt
    mainboard = self.mainboard(C)
  File "./freebios2/util/newconfig/config.py", line 1668, in mainboard
    if (C): mainboard(PATH)
  File "./freebios2/util/newconfig/config.py", line 1036, in mainboard
    part('mainboard', path, 'Config.lb', 0)
  File "./freebios2/util/newconfig/config.py", line 1094, in part
    doconfigfile(srcdir, partdir, file)
  File "./freebios2/util/newconfig/config.py", line 1159, in doconfigfile
    if (not parse('cfgfile', fp.read())):
  File "./freebios2/util/newconfig/config.py", line 1800, in parse
    return wrap_error_reporter(P, rule)
  File "./freebios2/util/newconfig/yappsrt.py", line 162, in wrap_error_reporter
    try: return getattr(parser, rule)()
  File "./freebios2/util/newconfig/config.py", line 1606, in cfgfile
    stmt = self.stmt(1)
  File "./freebios2/util/newconfig/config.py", line 1548, in stmt
    arch = self.arch(C)
  File "./freebios2/util/newconfig/config.py", line 1416, in arch
    partend = self.partend(C)
  File "./freebios2/util/newconfig/config.py", line 1373, in partend
    if (C): partpop()
  File "./freebios2/util/newconfig/config.py", line 1105, in partpop
    warning("Option %s using default value %s" % (i, getformated(i, curpart.image)))
  File "./freebios2/util/newconfig/config.py", line 179, in warning
    print "%s" % loc
  File "./freebios2/util/newconfig/config.py", line 137, in __str__
    s = s + '\n' + p.at()
AttributeError: 'int' object has no attribute 'at'

With python 2.2 I get:
bash-2.05a$ python2.2 ./freebios2/util/newconfig/config.py ./Config.lb ./freebios2
Configuring TARGET hdama
Will place Makefile, crt0.S, etc. in hdama
Configuring ROMIMAGE normal
Configuring DIR /config/Config.lb
Configuring DIR /lib/Config.lb
Configuring DIR /console/Config.lb
Configuring DIR /stream/Config.lb
Configuring DIR /devices/Config.lb
Configuring DIR /pc80/Config.lb
Configuring DIR /boot/Config.lb
Configuring PART mainboard, path arima/hdama
Configuring PART arch, path i386
Adding init file: config/crt0.base
Configuring DIR lib/Config.lb
Configuring DIR boot/Config.lb
Configuring DIR smp/Config.lb
End PART arch
===> WARNING: Option CONFIG_SMP using default value 0
Config.lb:0
mainboard/arima/hdama/Config.lb:0
NO FALLBACK USED!
Configuring PART northbridge, path amd/amdk8
End PART northbridge
Configuring PART southbridge, path amd/amd8111
End PART southbridge
Configuring PART southbridge, path amd/amd8131
End PART southbridge
Configuring PART superio, path NSC/pc87360
End PART superio
Configuring DIR /pc80/Config.lb
===> WARNING: object/driver mc146818rtc previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
pc80/Config.lb:0
Configuring DIR /cpu/k8/Config.lb
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
Configuring PART cpu, path k8
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
===> WARNING: object/driver cpuid previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
cpu/p5/Config.lb:0
===> WARNING: object/driver mtrr previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
===> WARNING: object/driver cpufixup previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
===> WARNING: object/driver apic_timer previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
End PART cpu
Configuring DIR /cpu/k8/Config.lb
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
===> WARNING: object/driver cpuid previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
cpu/p5/Config.lb:0
===> WARNING: object/driver mtrr previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
===> WARNING: object/driver cpufixup previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
===> WARNING: object/driver apic_timer previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
Configuring PART cpu, path k8
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
===> WARNING: object/driver cpuid previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
cpu/p5/Config.lb:0
===> WARNING: object/driver mtrr previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
===> WARNING: object/driver cpufixup previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
===> WARNING: object/driver apic_timer previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
End PART cpu
End PART mainboard
===> WARNING: Option CONFIG_IDE_STREAM using default value 0
Config.lb:0
===> WARNING: Option AMD8111_DEV using default value 0x3800
Config.lb:0
===> WARNING: Option USE_NORMAL_IMAGE using default value 1
Config.lb:0
End ROMIMAGE
Configuring ROMIMAGE fallback
Configuring DIR /config/Config.lb
Configuring DIR /lib/Config.lb
Configuring DIR /console/Config.lb
Configuring DIR /stream/Config.lb
Configuring DIR /devices/Config.lb
Configuring DIR /pc80/Config.lb
Configuring DIR /boot/Config.lb
Configuring PART mainboard, path arima/hdama
Configuring PART arch, path i386
Adding init file: config/crt0.base
Configuring DIR lib/Config.lb
Configuring DIR boot/Config.lb
Configuring DIR smp/Config.lb
End PART arch
===> WARNING: Option CONFIG_SMP using default value 0
Config.lb:0
mainboard/arima/hdama/Config.lb:0
Configuring PART northbridge, path amd/amdk8
End PART northbridge
Configuring PART southbridge, path amd/amd8111
End PART southbridge
Configuring PART southbridge, path amd/amd8131
End PART southbridge
Configuring PART superio, path NSC/pc87360
End PART superio
Configuring DIR /pc80/Config.lb
===> WARNING: object/driver mc146818rtc previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
pc80/Config.lb:0
Configuring DIR /cpu/k8/Config.lb
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
Configuring PART cpu, path k8
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
===> WARNING: object/driver cpuid previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
cpu/p5/Config.lb:0
===> WARNING: object/driver mtrr previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
===> WARNING: object/driver cpufixup previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
===> WARNING: object/driver apic_timer previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
End PART cpu
Configuring DIR /cpu/k8/Config.lb
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
===> WARNING: object/driver cpuid previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
cpu/p5/Config.lb:0
===> WARNING: object/driver mtrr previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
===> WARNING: object/driver cpufixup previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
===> WARNING: object/driver apic_timer previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
Configuring PART cpu, path k8
Configuring DIR /cpu/k7/Config.lb
Configuring DIR /cpu/p6/Config.lb
Configuring DIR /cpu/p5/Config.lb
===> WARNING: object/driver cpuid previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
cpu/p5/Config.lb:0
===> WARNING: object/driver mtrr previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
cpu/k7/Config.lb:0
cpu/p6/Config.lb:0
===> WARNING: object/driver cpufixup previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
===> WARNING: object/driver apic_timer previously defined
Config.lb:0
mainboard/arima/hdama/Config.lb:0
cpu/k8/Config.lb:0
End PART cpu
End PART mainboard
===> WARNING: Option CONFIG_IDE_STREAM using default value 0
Config.lb:0
===> WARNING: Option AMD8111_DEV using default value 0x3800
Config.lb:0
===> WARNING: Option USE_NORMAL_IMAGE using default value 0
Config.lb:0
End ROMIMAGE
Build ROM size 1048576
Creating hdama/fallback/static.c
Creating hdama/fallback/Makefile.settings
Creating hdama/fallback/crt0_includes.h
Creating hdama/fallback/Makefile
Creating hdama/fallback/ldoptions
Creating hdama/normal/static.c
Creating hdama/normal/Makefile.settings
Creating hdama/normal/crt0_includes.h
Creating hdama/normal/Makefile
Creating hdama/normal/ldoptions
Creating hdama/Makefile.settings
Creating hdama/Makefile

Eric

[ron minnich]

it still looks weird to me, but my python is 2.2.1

I'll try to do some compares tomorrow.

ron

[ron minnich]

oh yes, note the top level makefile should concatenate the images in 
fallback and normal to produce the final image, but I am not that far 
along yet.

ron