All of lore.kernel.org
 help / color / mirror / Atom feed
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
 fuzz-pack-headers.c => oss-fuzz/fuzz-pack-headers.c | 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

  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.