shell.py issues on ArchLinux

[bmentink]

1st issue: I had to comment out the following section in shell.py to even get it running:

def reset(self, fullreset = True):
........

    #waitcr()
    #ser.write(b'\r')
    #waitcr()

2nd issue: bye does not exit shell, doesn't exist ... just get:

bye
?+

3rd issue: #flash command does not work:

flash build/nuc.hex

please wait...
Traceback (most recent call last):
File "./shell.py", line 72, in
swapforth.main(TetheredJ1a)
File "../shell/swapforth.py", line 385, in main
r.shell()
File "../shell/swapforth.py", line 339, in shell
self.shellcmd(cmd)
File "../shell/swapforth.py", line 197, in shellcmd
d = self.serialize()
File "./shell.py", line 68, in serialize
s = array.array('B', s).tostring().ljust(8192, chr(0xff))
TypeError: ljust() argument 2 must be a byte string of length 1, not str

Seems shell.py is a bit broken
.. also the documentation says to connect on /dev/ttyUSB0, but on my system it is /dev/ttyUSB1, USB0 is the programmer SPI interface ..
"make connect" does the right thing ..

EDIT:
2. My bad .. needs '#bye', not 'bye'
3. Seems shell.py only works with python2, maybe that should be documented .. ;)
4. Documentation again ... in the "Building from scratch" section it is stated:

Now run make j1a again - this compiles an FPGA image with the complete code base built-in.

This does not work, as you have to run "make -C icestorm" again before "make j1a" to pick up the added words, otherwise you just get the minimum. I guess ram.v is not updated.

[RGD2]

Just make By itself without -C icestorm should be enough to rebuild build/ram.v

But yes, it is needed after a #flash

[bmentink]

Thanks, any comment on issue #1