- 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.
**Related issue**: #932 and #880
**Type of change**: other enhancement
**Impact**: other (change a repo initialization script)
**Release Notes**
change a repo initialization script [init-submodules-no-riscv-tools-nolog.sh](https://github.com/zslwyuan/chipyard/blob/master/scripts/init-submodules-no-riscv-tools-nolog.sh) so it can generate the .sbtopt file in the Chipyard directory during repo initialization and use the absolute path for the SBT workspace variables in .sbtopts.
Hope it can adapt to more compilation scenarios. For example, now users should be able to open the shipyard project directly using Intellij.
