kevinpt / opbasm

Open PicoBlaze Assembler
http://kevinpt.github.io/opbasm/
MIT License
60 stars 13 forks source link

opbasm 1.1 interprets relative path as an assembler instruction #3

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I'm using opbasm 1.1 from HG repository Rev. 53 (379e9b667a51) 29.08.2014.

If I'm using opbasm with the new parser (--pyparsing is not set; see Google 
code issue 4 for reference), it seems that all INCLUDE instructions with 
relative paths are tried to translate into real assembler instructions.

examples:

; this is ok
INCLUDE "SoFPGA_Texts.psm"
;
; this throws an error
INCLUDE "..\lib\L_PicoBlaze\psm\RegisterNames.psm"

The extracted and unknown instruction is "RegisterNames ". Note the trailing 
space before the quote sign.

Original issue reported on code.google.com by Paebbels on 8 Oct 2014 at 6:07

GoogleCodeExporter commented 9 years ago
The parser regex has been updated in the repository code to handle relative 
paths correctly. The slashes and colon for Windows were not included in the 
pattern for valid include strings.

Original comment by kevin.thibedeau on 8 Oct 2014 at 3:26