- cleanup exports in CA
- remove exports from build-extra-tests CA
- put exports in build-extra-tests.sh
- based it on $GITHUB_WORKSPACE
- changed exports in run-tests CA to use $GITHUB_WORKSPACE
- in common.mk
- removed TODO comment lines where `$(sim)` had been removed
- removed commented out lines that used `$(sim)`
- removed unused debugging script1.sh
- In main workflow chipyard-run-tests.yml
- Remove debug output in `make-keys`
- remove export intializers in `build-extra-tests`
- Add prepare-chipyard-fpga to `needs` in `all_tests_passed`
- build-extra-tests composite-action comment changed to Builds extra test required for FireSim tests
- removed unneeded inputs tools-version and group-key from run-tests composite action
- Added better comment to toolchain-build composite action
- Change CI_MAKE_NPROC back to 8 in defaults.sh
- Moved all the duplicated enviroment setting stuff in workflow file to the top
- BUILDSERVER, BUILDUSER, CI_DIR, SERVER, and JVM_OPTS
- Romoved all the not needed passed parameters in the calls to run-script
- A bit of cleanup to .github/README.md
- added link to SO `if` hack
- cleaned up comments on CI_DIR
- removed bad empty final bullet line
- As similar as possible to the circle ci code.
- The `.github/README.md` file has a fair amount of documentation for this.
- Creates a worfklow file
- re-uses most of the circleci/scipts unchanged
- defines a number of *Composite Actions* which are like YML subroutines
- Removes the circle-ci code
- Points the CI badge in the top level README to use the GA test result
Switched step 2 and 3 of the partitioning of the SD card. Explained that expert mode is needed for steps 1 and 3. Added that username is root and password is fpga. Now SD card partitioning if followed shouldn't result in a failed boot (sector resets to 2048 after o command) and the user doesn't need to dig around for the password.
**Related issue**: N/A
**Type of change**: bug fix
**Impact**: other
**Release Notes**
The string of path for "-ip-vivado-tcls" could be empty ("")
For example, run "make SUB_PROJECT=arty bitstream" will get errors due the argument parsing in [prologue.tcl](d4b3878e4f/xilinx/common/tcl/prologue.tcl)
Swaping the two arguments can resolve bug.
