Skip to content

Commit c30043e

Browse files
authored
Fix for the circular dependency issue (gpgpu-sim#116)
* Update the AccelSim test script to target the repo specified by the user. * Change the warning for a missing ACCELSIM_REPO environment variable to an error and update workflow file to point to a temperory fix for circular dependency issue * fix the repo name * Change back to use dev branch
1 parent 1c6cd9b commit c30043e

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

.github/workflows/accelsim.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ on:
1515

1616
# By default regress against accel-sim's dev branch
1717
env:
18+
ACCELSIM_REPO: https://github.com/purdue-aalp/accel-sim-framework-public.git
1819
ACCELSIM_BRANCH: dev
1920

2021
# A workflow run is made up of one or more jobs that can run sequentially or in parallel

short-tests-accelsim.sh

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,11 @@ if [ ! -n "$ACCELSIM_BRANCH" ]; then
88
exit 1;
99
fi
1010

11+
if [ ! -n "$ACCELSIM_REPO" ]; then
12+
echo "ERROR ** set the ACCELSIM_REPO env variable";
13+
exit 1;
14+
fi
15+
1116
if [ ! -n "$GPUAPPS_ROOT" ]; then
1217
echo "ERROR ** GPUAPPS_ROOT to a location where the apps have been compiled";
1318
exit 1;
@@ -19,10 +24,12 @@ export PATH=$CUDA_INSTALL_PATH/bin:$PATH
1924
source ./setup_environment
2025
make -j
2126

22-
git clone https://github.com/accel-sim/accel-sim-framework.git
27+
git clone $ACCELSIM_REPO
28+
basename=$(basename $ACCELSIM_REPO)
29+
filename=${basename%.*}
2330

2431
# Build accel-sim
25-
cd accel-sim-framework
32+
cd $filename
2633
git checkout $ACCELSIM_BRANCH
2734
source ./gpu-simulator/setup_environment.sh
2835
make -j -C ./gpu-simulator

0 commit comments

Comments
 (0)