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 636 07-05-2013, 12:49 PM
Last Post: Marcel Samek
  WP-34S assembler & constants Marcel Samek 8 1,872 06-27-2013, 12:27 PM
Last Post: Marcel Samek
  [WP34S] WP34S firmware on the AT91SAM7L-STK dev kit? jerome ibanes 1 910 10-04-2012, 04:59 PM
Last Post: Paul Dale
  HP 71b Assembler Michael Fehlhammer 2 891 09-06-2012, 08:20 AM
Last Post: Michael Fehlhammer
  Original DM-15CC batch capacitor patch Neil Hamilton (Ottawa) 2 1,071 07-19-2012, 08:26 PM
Last Post: Gerson W. Barbosa
  [WP34s] Assembler question fhub 6 1,726 05-14-2012, 03:08 PM
Last Post: Marcus von Cube, Germany
  WP 34S: Simplification to the Assembler Marcus von Cube, Germany 2 932 05-06-2012, 02:41 AM
Last Post: Marcus von Cube, Germany
  HP-41 NUT Assembler question MichaelG 6 1,377 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,227 01-26-2012, 01:34 PM
Last Post: Geir Isene
  HP-41 MCode Assembler/Linker 32-bit version MichaelG 19 3,573 01-22-2012, 12:44 PM
Last Post: MichaelG

Forum Jump: