Skip to content

Commit a84cc1e

Browse files
committed
corpus: remove submodule
A Git submodule was introduced in commit 341ac1e ("corpus: initial integration"). The main idea was to keep in sync source code of tests and their seed corpus. However, this approach is not convenient: - seed corpus updates often and someone need to regularly bump submodule version, otherwise outdated seed corpus is used - seed corpus takes about 5Gb and git clone of repository with tests takes significant time
1 parent 4fc4f85 commit a84cc1e

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

.clusterfuzzlite/Dockerfile

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,5 @@ RUN apt-get update && apt-get install -y \
1212

1313
COPY . $SRC/lua-c-api-tests
1414
WORKDIR $SRC/lua-c-api-tests
15-
RUN rm -rf corpus
1615
RUN git clone --depth 1 --branch cfl https://github.com/ligurio/lua-c-api-corpus corpus
1716
COPY .clusterfuzzlite/build.sh $SRC/

.github/workflows/test.yaml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,9 +37,15 @@ jobs:
3737
runs-on: ubuntu-22.04
3838
steps:
3939
- uses: actions/checkout@v4
40+
41+
- name: Download seed corpus
42+
uses: actions/checkout@v4
4043
with:
44+
repository: ligurio/lua-c-api-corpus
45+
# The tests expect a corpus directory in a repository
46+
# root directory. Otherwise corpus will not be used.
47+
path: corpus
4148
fetch-depth: 0
42-
submodules: recursive
4349

4450
- name: Setup common packages
4551
run: |

corpus

Submodule corpus deleted from a823138

0 commit comments

Comments
 (0)