July 13: Didn't work much today. Just a little cleaning up of the the return code. Now, control flow statements like if, else, while, etc. do not modify the rc variable. * [[http://git.etherboot.org/?p=people/lynusvaz/gpxe.git;a=commit;h=3f2c5a501a622b271f5cf0db9cb7fb9f495b37ac|if, else, etc. do not affect rc]] July 18: Added comments. I've started a new branch, [[http://git.etherboot.org/?p=people/lynusvaz/gpxe.git;a=shortlog;h=refs/heads/scripting_test|scripting_test]] for integration into gPXE. Pushed the quoting changes to this branch, as a first commit. Hopefully it will stand up to the scrutiny. Also made a bash script to automate testing of the scripting features. The steps are: * Pick up each .gpxe file in turn from the src/tests/ directory * Boot gPXE in qemu and pass the script using the -bootp switch * Capture the output in a temporary file * Kill qemu after a timeout * Remove the beginning lines of the file, upto 'Booting from filename...' * Compare this to a pre-made output file Qemu is run as: qemu -tftp . -cdrom bin/gpxe.iso -bootp tftp://10.0.0.2//$GSCRIPT -serial stdio > test_out 2>/dev/null & Today's commits: * [[http://git.etherboot.org/?p=people/lynusvaz/gpxe.git;a=commit;h=e01f237ef8b6786c1cbcfcc12e846aeec3189bcc|Added quoting]] * [[http://git.etherboot.org/?p=people/lynusvaz/gpxe.git;a=commit;h=e34825284cd3e8c620bf687d160d26570c4b1197|Automatic testing of gPXE scripts]] July 19: Still cleaning up the code. One of the things to be done is to reduce the number of global variables, and move the necessary ones to header files, to reduce the coupling between files. So, I added a new header file to deal with the branch and loop variables. Also went through the scripting_test commit, and made a few small changes: increased the max number of arguments to 68, and a few cosmetic changes. Today's commits: * [[http://git.etherboot.org/?p=people/lynusvaz/gpxe.git;a=commit;h=16a5dfef7b65e4bb2b7d673a47c179bb3e647688|Fix global variables]] * [[http://git.etherboot.org/?p=people/lynusvaz/gpxe.git;a=commit;h=ab44e95989966a9ed5153dd02842d7a248e7fd49|Added quoting (modified)]]