crayon icon indicating copy to clipboard operation
crayon copied to clipboard

`Tests/runner.py` doesn't handle alias `crayon`

Open TimJSwan89 opened this issue 5 years ago • 1 comments

This may only relate to my other issue #265 for myself: I did figure out that I don't actually have a 'crayon' command, but rather an alias set up to run the .exe with mono.

The issue is that it doesn't seem like the python script is set up to actually run the tests when it is set up that way. It says sh: crayon: command not found when I run python runner.py

In my terminal, I have to use type instead of which to see the alias.


The fix should be one of the following:

  1. Documentation on how to run tests when someone uses an alias to run crayon.
  2. runner.py to be updated to handle the alias.

There may be another fix which is better, but I'm not sure, yet. Basically, since it is saying sh in the error, that may point to the core of the issue. I tend to prefer bash instead of sh, but I don't explicitly see sh used in runner.py. I'm guessing it somehow indirectly uses it via os.popen(cmd)

I'll just add this link and trust that you will come up with a wise solution to the issue. https://stackoverflow.com/questions/21822054/how-to-force-os-system-to-use-bash-instead-of-shell

TimJSwan89 avatar Sep 24 '19 23:09 TimJSwan89

Best solution is for me to build a true executable for Mac/*nix platforms and not require an alias or mono :)

In the mean time I'll take a look at the quick fix once I have my MacBook with me.

Thanks!

blakeohare avatar Sep 25 '19 03:09 blakeohare