From ef45fa81f1cfb263fd4d0b9835681ac424ea2dac Mon Sep 17 00:00:00 2001 From: Cody Boone Ferguson <53008573+xexyl@users.noreply.github.com> Date: Tue, 19 Nov 2024 08:28:08 -0800 Subject: [PATCH] Remove makefile.local functionality Remove makefile.local functionality in the Makefiles as this can cause compilation problems and the purpose of the file is no longer needed. Should one really want to override things they can pass it at the command line manually. This change means that the jparse_bug_report.sh script has been updated as well. This problem actually caused the mess of commits bcb793e7d996da45fa9d47ae70084c9294eb1f2c and the commit that rolled it back, bcb793e7d996da45fa9d47ae70084c9294eb1f2c, as when trying to fix a mess in another repo that used it, by accident it was pushed to this repo, undoing fixes from yesterday. --- CHANGES.md | 12 +++++++++ Makefile | 17 ------------- jparse_bug_report.sh | 59 +++----------------------------------------- test_jparse/Makefile | 18 -------------- version.h | 2 +- 5 files changed, 17 insertions(+), 91 deletions(-) diff --git a/CHANGES.md b/CHANGES.md index cf30c85..a4e5429 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -1,6 +1,18 @@ # Significant changes in the JSON parser repo +## Release 2.1.4 2024-11-19 + +Remove `makefile.local` functionality in the Makefiles as this can cause +compilation problems and the purpose of the file is no longer needed. Should one +really want to override things they can pass it at the command line manually. +This change means that the `jparse_bug_report.sh` script has been updated as +well. This problem actually caused the mess of commits +bcb793e7d996da45fa9d47ae70084c9294eb1f2c and the commit that rolled it back, +bcb793e7d996da45fa9d47ae70084c9294eb1f2c, as when trying to fix a mess in +another repo that used it, by accident it was pushed to this repo, undoing fixes +from yesterday. + ## Release 2.1.3 2024-11-18 Improve function `parse_json_str()` and `parse_json()` by allowing for empty or diff --git a/Makefile b/Makefile index cf51a34..ffab0ac 100644 --- a/Makefile +++ b/Makefile @@ -424,23 +424,6 @@ TMP_BUILD_LOG= ".build.log.$$$$" BUILD_LOG= build.log -############################################################ -# User specific configurations - override Makefile values # -############################################################ - -# The directive below retrieves any user specific configurations from makefile.local. -# -# The - before include means it's not an error if the file does not exist. -# -# We put this directive just before the first all rule so that you may override -# or modify any of the above Makefile variables. To override a value, use := symbols. -# For example: -# -# CC:= gcc -# --include makefile.local - - ########################################### # all rule - default rule - must be first # ########################################### diff --git a/jparse_bug_report.sh b/jparse_bug_report.sh index dd46328..da815f2 100755 --- a/jparse_bug_report.sh +++ b/jparse_bug_report.sh @@ -62,9 +62,8 @@ export TOOLS=" verge " -# we need this to find overriding makefile.local in all directories to see if -# the user is overriding any Makefile. As well, we check if the directory even -# is searchable and has a Makefile. +# We need to verify that certain subdirectories, in particular +# test_jparse/, are searchable and have a readable Makefile. export SUBDIRS=" test_jparse " @@ -80,7 +79,7 @@ if [[ -z "$MAKE" ]]; then fi export MAKE export MAKE_FLAGS="V=@ S=@ Q= E=@ I= Q_V_OPTION=1 INSTALL_V='-v' MAKE_CD_Q=" -export BUG_REPORT_VERSION="2.0.1 2024-11-16" +export BUG_REPORT_VERSION="2.0.2 2024-11-19" export FAILURE_SUMMARY= export NOTICE_SUMMARY= export DBG_LEVEL="0" @@ -1468,7 +1467,7 @@ if [[ -z "$T_FLAG" ]]; then # ...and yes, this actually happened by chance, not deliberately, at least # in the mkiocccentry repo. Later on, after this script in jparse was # updated to be correct, the numbers were kept the same even after the tools - # that were unneeded were removed. + # that were unneeded were removed, due to the 'significance' of 42. run_check 42 "$MAKE $MAKE_FLAGS all" # make test: run the jparse test suite @@ -1638,56 +1637,6 @@ for d in $SUBDIRS; do done write_echo "" -# check for makefile.local files to see if user is overriding any rules or variables. -# -# NOTE: we don't use run_check for this because it's not an actual error whether -# or not the user has a makefile.local file. What matters is the contents of it -# if they do have one. -# -write_echo "## CHECKING IF \"makefile.local\" EXISTS" -if [[ -e "./makefile.local" ]]; then - if [[ -r "./makefile.local" ]]; then - write_echo "### Warning: found \"Makefile\" overriding file \"makefile.local\":" - write_echo "RUNNING: \"cat ./makefile.local\"" - write_echo "--" - if [[ -z "$L_FLAG" ]]; then - # tee -a -- "$LOGFILE" < makefile.local - < makefile.local tee -a -- "$LOGFILE" - else - cat makefile.local >> "$LOGFILE" - fi - write_echo "--" - else - write_echo "### Warning: found unreadable \"makefile.local\"" - fi -else - write_echo "# \"Makefile\" has no overriding \"makefile.local\"" -fi -write_echo "" - -# now do the same for subdirectories -for d in $SUBDIRS; do - if [[ -e "$d/makefile.local" ]]; then - if [[ -r "$d/makefile.local" ]]; then - write_echo "### Warning: found \"$d/Makefile\" overriding file \"$d/makefile.local\":" - write_echo "--" - if [[ -z "$L_FLAG" ]]; then - # tee -a -- "$LOGFILE" < makefile.local - < "$d/makefile.local" tee -a -- "$LOGFILE" - else - cat "$d/makefile.local" >> "$LOGFILE" - fi - write_echo "--" - else - write_echo "### Warning: found unreadable \"$d/makefile.local\"" - fi - else - write_echo "# \"$d/Makefile\" has no overriding \"$d/makefile.local\"" - fi -done -write_echo "" - - # check if there are any local modifications to the code # # NOTE: We don't use run_check for this because if git does not exist for some diff --git a/test_jparse/Makefile b/test_jparse/Makefile index ca8f6a6..318ac89 100644 --- a/test_jparse/Makefile +++ b/test_jparse/Makefile @@ -303,24 +303,6 @@ ALL_OTHER_TARGETS= ${SH_TARGETS} extern_everything # TARGETS= ${PROG_TARGETS} - -############################################################ -# User specific configurations - override Makefile values # -############################################################ - -# The directive below retrieves any user specific configurations from makefile.local. -# -# The - before include means it's not an error if the file does not exist. -# -# We put this directive just before the first all rule so that you may override -# or modify any of the above Makefile variables. To override a value, use := symbols. -# For example: -# -# CC:= gcc -# --include makefile.local - - ###################################### # all - default rule - must be first # ###################################### diff --git a/version.h b/version.h index 8ece38b..6c699ee 100644 --- a/version.h +++ b/version.h @@ -30,7 +30,7 @@ * * NOTE: this should match the latest Release string in CHANGES.md */ -#define JPARSE_REPO_VERSION "2.1.3 2024-11-18" /* format: major.minor YYYY-MM-DD */ +#define JPARSE_REPO_VERSION "2.1.4 2024-11-19" /* format: major.minor YYYY-MM-DD */ /* * official jparse version