From: "Arthur Chan via GitGitGadget" <gitgitgadget@gmail.com>
To: git@vger.kernel.org
Cc: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>,
"Arthur Chan" <arthur.chan@adalogics.com>,
"Arthur Chan" <arthur.chan@adalogics.com>
Subject: [PATCH v4] fuzz: reorganise the path for existing oss-fuzz fuzzers
Date: Mon, 19 Sep 2022 14:36:54 +0000 [thread overview]
Message-ID: <pull.1353.v4.git.1663598215154.gitgitgadget@gmail.com> (raw)
In-Reply-To: <pull.1353.v3.git.1663542495094.gitgitgadget@gmail.com>
From: Arthur Chan <arthur.chan@adalogics.com>
This patch is aimed to provide a better organisation for oss-fuzz
fuzzers and to avoid top-level clustters in the git repository when
more fuzzers are introduced. In addition, grouping the fuzzers into
their own directory separate their application on fuzz-testing from
the core functionalities of the git code. It also provide better
and tidier structure the oss-fuzz fuzzing library to manage, locate,
build and execute those fuzzers for fuzz-testing purposes in future
development.
A new direrctory oss-fuzz has been created and existing fuzzers are
moved into the new directory. Makefile and .gitignore have been
fixed accordingly.
CC: Josh Steadmon <steadmon@google.com>
CC: David Korczynski <david@adalogics.com>
Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>
---
fuzz: reorganise the path for existing oss-fuzz fuzzers
This patch is aimed to provide a better organisation for oss-fuzz
fuzzers and to avoid top-level clustters in the git repository when more
fuzzers are introduced. In addition, grouping the fuzzers into their own
directory separate their application on fuzz-testing from the core
functionalities of the git code. It also provide better and tidier
structure the oss-fuzz fuzzing library to manage, locate, build and
execute those fuzzers for fuzz-testing purposes in future development.
A new direrctory oss-fuzz has been created and existing fuzzers are
moved into the new directory. Makefile and .gitignore have been fixed
accordingly.
Create new .gitignore file for the new directory in v4 and fix the
commit message.
CC: Josh Steadmon steadmon@google.com CC: David Korczynski
david@adalogics.com Signed-off-by: Arthur Chan arthur.chan@adalogics.com
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1353%2Farthurscchan%2Frelocate-fuzzer-v4
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1353/arthurscchan/relocate-fuzzer-v4
Pull-Request: https://github.com/gitgitgadget/git/pull/1353
Range-diff vs v3:
1: 21714e583aa ! 1: 82d9568ea65 fuzz: reorganise the path for existing oss-fuzz fuzzers
@@ Commit message
fuzz: reorganise the path for existing oss-fuzz fuzzers
This patch is aimed to provide a better organisation for oss-fuzz
- fuzzers, allowing more fuzzers for the git project to be added
- in later development.
+ fuzzers and to avoid top-level clustters in the git repository when
+ more fuzzers are introduced. In addition, grouping the fuzzers into
+ their own directory separate their application on fuzz-testing from
+ the core functionalities of the git code. It also provide better
+ and tidier structure the oss-fuzz fuzzing library to manage, locate,
+ build and execute those fuzzers for fuzz-testing purposes in future
+ development.
A new direrctory oss-fuzz has been created and existing fuzzers are
moved into the new directory. Makefile and .gitignore have been
@@ Commit message
## .gitignore ##
@@
-/fuzz-commit-graph
-+/oss-fuzz/fuzz-commit-graph
/fuzz_corpora
-/fuzz-pack-headers
-/fuzz-pack-idx
-+/oss-fuzz/fuzz-pack-headers
-+/oss-fuzz/fuzz-pack-idx
/GIT-BUILD-OPTIONS
/GIT-CFLAGS
/GIT-LDFLAGS
@@ Makefile: SCRIPTS = $(SCRIPT_SH_GEN) \
fuzz-objs: $(FUZZ_OBJS)
+ ## oss-fuzz/.gitignore (new) ##
+@@
++fuzz-commit-graph
++fuzz-pack-headers
++fuzz-pack-idx
+
## fuzz-commit-graph.c => oss-fuzz/fuzz-commit-graph.c ##
## fuzz-pack-headers.c => oss-fuzz/fuzz-pack-headers.c ##
.gitignore | 3 ---
Makefile | 6 +++---
oss-fuzz/.gitignore | 3 +++
fuzz-commit-graph.c => oss-fuzz/fuzz-commit-graph.c | 0
| 0
fuzz-pack-idx.c => oss-fuzz/fuzz-pack-idx.c | 0
6 files changed, 6 insertions(+), 6 deletions(-)
create mode 100644 oss-fuzz/.gitignore
rename fuzz-commit-graph.c => oss-fuzz/fuzz-commit-graph.c (100%)
rename fuzz-pack-headers.c => oss-fuzz/fuzz-pack-headers.c (100%)
rename fuzz-pack-idx.c => oss-fuzz/fuzz-pack-idx.c (100%)
diff --git a/.gitignore b/.gitignore
index 80b530bbed2..ed8b9f04e5c 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,7 +1,4 @@
-/fuzz-commit-graph
/fuzz_corpora
-/fuzz-pack-headers
-/fuzz-pack-idx
/GIT-BUILD-OPTIONS
/GIT-CFLAGS
/GIT-LDFLAGS
diff --git a/Makefile b/Makefile
index d9247ead45b..2d56aae7a1d 100644
--- a/Makefile
+++ b/Makefile
@@ -686,9 +686,9 @@ SCRIPTS = $(SCRIPT_SH_GEN) \
ETAGS_TARGET = TAGS
-FUZZ_OBJS += fuzz-commit-graph.o
-FUZZ_OBJS += fuzz-pack-headers.o
-FUZZ_OBJS += fuzz-pack-idx.o
+FUZZ_OBJS += oss-fuzz/fuzz-commit-graph.o
+FUZZ_OBJS += oss-fuzz/fuzz-pack-headers.o
+FUZZ_OBJS += oss-fuzz/fuzz-pack-idx.o
.PHONY: fuzz-objs
fuzz-objs: $(FUZZ_OBJS)
diff --git a/oss-fuzz/.gitignore b/oss-fuzz/.gitignore
new file mode 100644
index 00000000000..9acb74412ef
--- /dev/null
+++ b/oss-fuzz/.gitignore
@@ -0,0 +1,3 @@
+fuzz-commit-graph
+fuzz-pack-headers
+fuzz-pack-idx
diff --git a/fuzz-commit-graph.c b/oss-fuzz/fuzz-commit-graph.c
similarity index 100%
rename from fuzz-commit-graph.c
rename to oss-fuzz/fuzz-commit-graph.c
diff --git a/fuzz-pack-headers.c b/oss-fuzz/fuzz-pack-headers.c
similarity index 100%
rename from fuzz-pack-headers.c
rename to oss-fuzz/fuzz-pack-headers.c
diff --git a/fuzz-pack-idx.c b/oss-fuzz/fuzz-pack-idx.c
similarity index 100%
rename from fuzz-pack-idx.c
rename to oss-fuzz/fuzz-pack-idx.c
base-commit: d3fa443f97e3a8d75b51341e2d5bac380b7422df
--
gitgitgadget
next prev parent reply other threads:[~2022-09-19 14:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-09-16 19:03 [PATCH] fuzz: reorganise the path for existing oss-fuzz fuzzers Arthur Chan via GitGitGadget
2022-09-16 21:55 ` Junio C Hamano
2022-09-17 0:45 ` Arthur Chan
2022-09-17 23:28 ` [PATCH v2] " Arthur Chan via GitGitGadget
2022-09-18 23:08 ` [PATCH v3] " Arthur Chan via GitGitGadget
2022-09-19 11:10 ` Ævar Arnfjörð Bjarmason
2022-09-19 14:36 ` Arthur Chan via GitGitGadget [this message]
2022-09-19 16:38 ` [PATCH v4] " Junio C Hamano
2022-09-19 16:40 ` Junio C Hamano
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=pull.1353.v4.git.1663598215154.gitgitgadget@gmail.com \
--to=gitgitgadget@gmail.com \
--cc=arthur.chan@adalogics.com \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.