Patch for wp34s assembler



#2

There is a minor bug in the wp34s assember. If you are running the assembler from a path that has a space in it, it will locate the preprocessor OK, but when it tries executing the preprocessor, the space in the path will cause the command line to fail.

The patch below puts the preprocessor path into quotes.


Index: wp34s_asm.pl
===================================================================
--- wp34s_asm.pl (revision 3424)
+++ wp34s_asm.pl (working copy)
@@ -1810,7 +1810,7 @@
}
$pp_location =~ s:\\:/:g;
$pp_options = $no_skip_override if $no_skip_override;
- $cmd = "${pp_location} $pp_options -cat $tmp_file";
+ $cmd = "\"${pp_location}\" $pp_options -cat $tmp_file";

if ($v3_mode) {
$cmd .= " -v3";


Possibly Related Threads…
Thread Author Replies Views Last Post
  WP_34S: Patch for wp34s_lib.pl Marcel Samek 0 841 07-05-2013, 12:49 PM
Last Post: Marcel Samek
  WP-34S assembler & constants Marcel Samek 8 2,441 06-27-2013, 12:27 PM
Last Post: Marcel Samek
  [WP34S] WP34S firmware on the AT91SAM7L-STK dev kit? jerome ibanes 1 1,226 10-04-2012, 04:59 PM
Last Post: Paul Dale
  HP 71b Assembler Michael Fehlhammer 2 1,192 09-06-2012, 08:20 AM
Last Post: Michael Fehlhammer
  Original DM-15CC batch capacitor patch Neil Hamilton (Ottawa) 2 1,407 07-19-2012, 08:26 PM
Last Post: Gerson W. Barbosa
  [WP34s] Assembler question fhub 6 2,205 05-14-2012, 03:08 PM
Last Post: Marcus von Cube, Germany
  WP 34S: Simplification to the Assembler Marcus von Cube, Germany 2 1,241 05-06-2012, 02:41 AM
Last Post: Marcus von Cube, Germany
  HP-41 NUT Assembler question MichaelG 6 1,899 02-14-2012, 03:35 AM
Last Post: MichaelG
  Latest update for Clonix & NoV's modules (includes DISASM patch for 41CL) Diego Diaz 9 2,832 01-26-2012, 01:34 PM
Last Post: Geir Isene
  HP-41 MCode Assembler/Linker 32-bit version MichaelG 19 4,951 01-22-2012, 12:44 PM
Last Post: MichaelG

Forum Jump: