-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
How run a script with arguments? #7
Comments
Well, it looks like that's not actually supported yet (the above errors is from trying to run it as an executable with MPW which doesn't work). |
I'm curious, would adding this be complicated, or is it something I could do, like adding a command-line argument that calls through to a particular function? |
The main hangup is environment variable handling. I did some testing with MPW in Basilisk and it looks like with user scripts, they get their own copy of the local variables which are isolated from the main shell's variables (but scripts can also export them out). Currently, there's one big table of variables and some are flagged as exported so that would need to change. For the most part, it's probably not a big deal if local environment variables are shared but if a script calls another script, the argument variables (0, 1, 2, ...) will be clobbered. |
scripts should work now. You may need to mark them as text ( |
I added regular expression support to evaluate ... but there are a couple other deficiencies that are preventing that script from running. |
Thanks for all the work you've put into this so far. If there's anything I can help with, let me know. |
I'm trying to get this to run in my (non-MPW) makefile:
Installed copied the
Scripts
folder from Apple's MPW to~/mpw/Scripts/
. TriedAnd as long as I add
{MPW}Scripts
to$Commands
that seems to run, but it then fails withI tried adding
-v
before-c
to get some additional information, but that doesn't seem to have any effect. Also tried:build:obj:$(EXECUTABLE).tmp
as the path. I'm assuming something about the file paths is wrong, but no idea how to confirm that or what format this stuff should really be. Do I somehow need to tellmpw-shell
about the current directory fromzsh
? Or should it just pick that up?The text was updated successfully, but these errors were encountered: