add --execute option to run-fremake

[rem1776]

Describe your changes

Changes run-fremake to work more like bronx's fremake and require a -e/--execute flag to run build scripts.

If the --execute option is given, it'll work as it does currently without the flag and run the compile script/docker build command after it is created. If its not given, it'll print out a compile script path for the user to run.

Since container builds don't have a build script currently this also adds code to write out a short script with the container commands to build and transfer the format.

Issue ticket number and link (if applicable)

Checklist before requesting a review

• I ran my code
• I tried to make my code readable
• I tried to comment my code
• I wrote a new test, if applicable
• I wrote new instructions/documentation, if applicable
• I ran pytest and inspected it's output
• I ran pylint and attempted to implement some of it's feedback

[singhd789]

Great idea, thanks for this! Takes a little bit more of the magic of how to build the actual container away. 🎉 Maybe we should actually make the creation of this script a default behavior in def build. (basically replacing def build with def createBuildScript)
Then in runFremake.py, instead of

if execute:
    dockerBuild.build(containerBuild,containerRun)
else:
    dockerBuild.createBuildScript(containerBuild, containerRun)
    print("Container build script created at "+dockerBuild.userScriptPath+"\n\n")

It would be:

#create the container build script
dockerBuild.createBuildScript(containerBuild, containerRun)

#print statement 
(ADD print statement somewhere about where the script is located)

if execute:
    subprocess.run(args=[dockerBuild.userScriptPath], check=True)
else:
    print("Container build script created at "+dockerBuild.userScriptPath+"\n\n")

What do you think? Or do you think it might be better to have one less script being made if it doesn't need to be (i.e. if execute is passed, no need to make the build script)

[rem1776]

Yeah I think always making the script is a good idea just cause it'll be a little more transparent and easier to debug build issues. I'll add those changes in!

[singhd789]

This is looking good! Do you think we can also add print statements for where the checkout and Makefile files are too? I think for checkout, we could put this on line 91 of runfremake.py:

print("\nCheckout script created in "+ srcDir + "/checkout.sh \n")

Then for the makefile, output around line 131 of runfremake.py:

print("\nMakefile created at " + bldDir + "/Makefile" + "\n")