OpenEuphoria

1. bug in the intinit.e file

• Posted by bugfinder Oct 08, 2010
• 1124 views

There is a bug in the intinit.e file at line 60 when Argv = {} the slice will  
 
cause an error condition. 
 
I am running the eu.ex files in a directory other than the source directory. 
 
NOTE: I should get display of the options not a slice error. 
 
 
This other directory contains the following : 
 
global.e 
keylist.e 
symtab.e 
scanner.e 
parser.e 
emit.e 
pathopen.e 
eu.ex 
reswords.e 
error.e 
main.e 
opnames.e 
execute.e 
 
Enviroment: 
 
E:\WORKCDE4>set eudir 
EUDIR=E:\EU4 
 
E:\WORKCDE4>set euinc 
EUINC=E:\EU4\include;E:\EU4\source;E:\EU4\MTRINC 
 
   

 
This will work OK running from the source directory. 
 
 
   55:          Argv = expand_config_options(Argv) 
   56:          m:map opts = cmd_parse( get_options(), 
   57:                  { NO_VALIDATION_AFTER_FIRST_EXTRA, PAUSE_MSG, pause_msg 
   58:  trace(1) 
   59:          sequence tmp_Argv = Argv 
   60==>        Argv = Argv[1..2] & GetDefaultArgs() 
   61:          Argc = length(Argv) 
   62: 
   63:          m:map default_opts = cmd_parse( get_options(), , Argv ) 
   64:          merge_maps( opts, default_opts ) 
   65: 
   66:          Argv = tmp_Argv 
   67:          Argc = length( Argv ) 
   68: 
variable name? Argv 
Argv={{101e,58:,92\,101e,117u,524,92\,98b,105i,110n,92\,101e,117u,105i,46.,... 
 

 
This will FAIL running eu.ex from the different directory with 
 
source directory set in the path.  
 
 
 
   55:          Argv = expand_config_options(Argv) 
   56:          m:map opts = cmd_parse( get_options(), 
   57:                  { NO_VALIDATION_AFTER_FIRST_EXTRA, PAUSE_MSG, pause_msg 
   58:  trace(1) 
   59:          sequence tmp_Argv = Argv 
   60==>        Argv = Argv[1..2] & GetDefaultArgs() 
   61:          Argc = length(Argv) 
   62: 
   63:          m:map default_opts = cmd_parse( get_options(), , Argv ) 
   64:          merge_maps( opts, default_opts ) 
   65: 
   66:          Argv = tmp_Argv 
   67:          Argc = length( Argv ) 
   68: 
variable name? Argv 
Argv={} 
 

3. Re: bug in the intinit.e file

• Posted by jimcbrown (admin) Oct 08, 2010
• 1095 views

On Linux, but I can not reproduce these steps with eui r3414.

cp -a /home/eu/source/global.e /home/eu/source/keylist.e /home/eu/source/scanner.e /home/eu/source/parser.e /home/eu/source/emit.e /home/eu/source/pathopen.e /home/eu/source/eu.ex /home/eu/source/reswords.e /home/eu/source/error.e /home/eu/source/main.e /home/eu/source/opnames.e /home/eu/source/execute.e /tmp/y/ 
cd /tmp/y 
EUDIR=/home/eu EUINC=/home/eu/include:/home/eu/source eui eu.ex 

This gives me the help screen - or the display of the list of options.

There is a bug in the intinit.e file at line 60 when Argv = {} the slice will  
 
cause an error condition. 
 
I am running the eu.ex files in a directory other than the source directory. 
 
NOTE: I should get display of the options not a slice error. 
 
 
This other directory contains the following : 
 
global.e 
keylist.e 
symtab.e 
scanner.e 
parser.e 
emit.e 
pathopen.e 
eu.ex 
reswords.e 
error.e 
main.e 
opnames.e 
execute.e 
 
Enviroment: 
 
E:\WORKCDE4>set eudir 
EUDIR=E:\EU4 
 
E:\WORKCDE4>set euinc 
EUINC=E:\EU4\include;E:\EU4\source;E:\EU4\MTRINC 
 
   

 
This will work OK running from the source directory. 
 
 
   55:          Argv = expand_config_options(Argv) 
   56:          m:map opts = cmd_parse( get_options(), 
   57:                  { NO_VALIDATION_AFTER_FIRST_EXTRA, PAUSE_MSG, pause_msg 
   58:  trace(1) 
   59:          sequence tmp_Argv = Argv 
   60==>        Argv = Argv[1..2] & GetDefaultArgs() 
   61:          Argc = length(Argv) 
   62: 
   63:          m:map default_opts = cmd_parse( get_options(), , Argv ) 
   64:          merge_maps( opts, default_opts ) 
   65: 
   66:          Argv = tmp_Argv 
   67:          Argc = length( Argv ) 
   68: 
variable name? Argv 
Argv={{101e,58:,92\,101e,117u,524,92\,98b,105i,110n,92\,101e,117u,105i,46.,... 
 

 
This will FAIL running eu.ex from the different directory with 
 
source directory set in the path.  
 
 
 
   55:          Argv = expand_config_options(Argv) 
   56:          m:map opts = cmd_parse( get_options(), 
   57:                  { NO_VALIDATION_AFTER_FIRST_EXTRA, PAUSE_MSG, pause_msg 
   58:  trace(1) 
   59:          sequence tmp_Argv = Argv 
   60==>        Argv = Argv[1..2] & GetDefaultArgs() 
   61:          Argc = length(Argv) 
   62: 
   63:          m:map default_opts = cmd_parse( get_options(), , Argv ) 
   64:          merge_maps( opts, default_opts ) 
   65: 
   66:          Argv = tmp_Argv 
   67:          Argc = length( Argv ) 
   68: 
variable name? Argv 
Argv={} 
 

5. Re: bug in the intinit.e file

• Posted by mattlewis (admin) Oct 08, 2010
• 1072 views

Sorry, reading more closely, I see that it is. Originally, I copied the eu.ex source to a different directory and it ran fine. I can't see why or where the contents of Argv are disappearing. What was it before being passed to expand_config_options()?

Matt

7. Re: bug in the intinit.e file

• Posted by jimcbrown (admin) Oct 08, 2010
• 1048 views

 
Matt: 
 
If you add this line it will fix the problem. 
 
          Argv = expand_config_options(Argv) 
          m:map opts = cmd_parse( get_options(), 
                { NO_VALIDATION_AFTER_FIRST_EXTRA, PAUSE_MSG, pause_msg }, Arg 
 
add ==>>  if length(Argv) = 0 then Argv = getenv("eudir") end if 
 
          sequence tmp_Argv = Argv 
          Argv = Argv[1..2] & GetDefaultArgs() 
        Argc = length(Argv) 
 

This is wrong for a couple of different reasons:

1) EUDIR is not guaranteed to be set for 4.0 (we're moving to use of eu.cfg)

2) if EUDIR is not set, then getenv() returns -1, which could cause a crash as Argv is a sequence

3) even if EUDIR is set, Argv is meant to be a list of strings. Setting it equal to the contents of EUDIR will make Argv into a single string (or a list of characters)

4) Most importantly, the real problem seems to be in getting the list of config options. That is the reason why Argv is empty. It's not clear to any of us why that is failing for you...

9. Re: bug in the intinit.e file

• Posted by bugfinder Oct 08, 2010
• 1034 views

I am using SVN 3474 with no modifications. That fix only got me the option list so it did not solve anything anyways.

I looked at the option list displayed and there is no comment about how to pass a file to run on the command line ( example: eui.exe eu.ex test.ex ) The eui.exe file and euiw.exe should give a simple example with an option to display help; not just scroll off the screen with all the options.

The options used by developers should be a separate help display from the user options display.

11. Re: bug in the intinit.e file

• Posted by mattlewis (admin) Oct 08, 2010
• 1031 views

Yes, but if you run eui by itself, or with -?, it should probably have something like:

Usage: eui [options] <program> [program arguments] 

Matt

13. Re: bug in the intinit.e file

• Posted by jimcbrown (admin) Oct 08, 2010
• 1068 views

eu.ex has been available for users in any recent version 2.4 2.5 3.1 etc.

[/quote]

So has, e.g., dis.ex - but not very many users use it.

Aside from that, eu.ex does not have any special options of its own. It has the same options as eui.

With the exception of the new coverage options, very few of eui's options are intended for developers only. Maybe -test and -strict. Most of what you see is intended for use by the end-user.