summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix up tarball import.baserock/build-essentialSam Thursfield2013-01-14122-246/+12933
| | | | ALWAYS REMOVE ALL INSTANCES OF .gitignore.
* Remove GNUlib git module to avoid involving GNULib in a tarball buildbaserock/samthursfield/build-essentialSam Thursfield2013-01-142-3/+0
|
* Update morphology for tarball build and remove insanitySam Thursfield2013-01-111-8/+2
|
* Commit files from tarballSam Thursfield2013-01-11243-612/+103642
| | | | | patch requires GNUlib which requires GNU patch to work correctly from git, so we need to break this circular dependency with a tarball version.
* Update .gitmodules to point at upstream:gnulibbaserock/morphDaniel Silverstone2012-09-111-1/+1
|
* patch.morph: remove rm .git*Richard Maw2012-02-221-1/+0
|
* patch.morph: enable submodule supportRichard Maw2012-02-171-2/+3
|
* this is not tarRichard Maw2012-01-171-1/+1
|
* add morphRichard Maw2012-01-161-0/+14
|
* maint: enable the sc_space_tab syntax-check ruleJim Meyering2012-01-012-6/+5
| | | | | | * cfg.mk (local-checks-to-skip): Remove sc_space_tab, thus enabling this syntax-check rule. * tests/quoted-filenames: Use TAB-space, not space-TAB.
* maint: enable the useless_cpp_parens syntax-check ruleJim Meyering2012-01-012-3/+2
| | | | | | * cfg.mk (local-checks-to-skip): Remove sc_useless_cpp_parens, thus enabling this syntax-check rule. * src/patch.c: Remove unneeded parentheses.
* maint: update all copyright year number rangesJim Meyering2012-01-0157-63/+62
| | | | Run "make update-copyright".
* build: update to latest gnulib and adaptJim Meyering2011-12-149-24/+37
| | | | | | | | | | | | | | | Mark functions as pure of const, per recommendations enabled by new gcc -W options. Apply _GL_ATTRIBUTE_PURE and _GL_ATTRIBUTE_CONST. * src/common.h: Apply new function attributes. * src/pch.c: Likewise. * src/pch.h: Likewise. * src/util.c: Likewise. * src/util.h: Likewise. * configure.ac: Use -Wno-format-nonliteral. * m4/.gitignore: Update. * gnulib: Update to latest. * cfg.mk: Exempt src/util.c from two tests, to avoid new "make syntax-check" failures.
* Timestamp not set when creating files with --set-time or --set-utcAndreas Gruenbacher2011-12-094-9/+34
| | | | | | | | | | | * src/util.h (enum file_attributes): Add FA_XATTRS flag for extended attributes. * src/patch.c (main): Use set_file_attributes() even when the infile doesn't exist: it may still set the file time (FA_TIMES). Omit all other FA_ flags if infile doesn't exist. Otherwise, add FA_XATTRS as well. * src/util.c (set_file_attributes): Only copy extended attributes if FA_XATTRS is set. Avoid using st where it may be undefined. * tests/preserve-mode-and-timestamp: Add file create test.
* tests: specify test runner in LOG_COMPILER, not in TEST_ENVIRONMENTStefano Lattarini2011-10-121-1/+3
| | | | | | * tests/Makefile.am (TESTS_ENVIRONMENT): Don't use $(SHELL) here to ensure the test scripts are run through it; instead, ... (LOG_COMPILER): ... use it here.
* give a diagnostic rather than a failed assertion for a mangled patchJim Meyering2011-10-114-1/+47
| | | | | | | | | | * src/pch.c (another_hunk): Rather than asserting(C), issue the "replacement text or line numbers mangled ..." diagnostic when !C. * tests/mangled-numbers-abort: New test for the above. * tests/Makefile.am (TESTS): Add it. * NEWS: Mention it. Reported by Gabriel Vlasiu via Tim Waugh. See also http://bugzilla.redhat.com/738959
* build: use largefile module and update to latest gnulibJim Meyering2011-08-114-1/+4
| | | | | | | | * configure.ac: Remove AC_SYS_LARGEFILE, subsumed by ... * bootstrap.conf (gnulib_modules): ...this. Use largefile module. * gnulib: Update to latest. This is useful to Mac OS X 10.5 users if/when configure is generated using autoconf prior to v2.68-80-gdb2f2e0.
* build: include .version in tarball to avoid distcheck failureJim Meyering2011-08-111-0/+1
| | | | | | * Makefile.am (EXTRA_DIST): Append .version, to avoid "make distcheck" failure when run from an unpacked tarball. Reported by Iain Nicol.
* README: Describe where to go from a "git clone"Andreas Gruenbacher2011-08-111-3/+12
| | | | * README: Refer users to README-hacking after a "git clone".
* don't call fdopen with a negative FD upon dup failureJim Meyering2011-05-251-2/+5
| | | | | * src/patch.c (open_outfile): If dup fails, don't clobber its errno value by calling fdopen with -1.
* plug a leak in inp.c's plan_aJim Meyering2011-05-251-1/+4
| | | | * src/inp.c (plan_a): Don't leak "buffer" upon early return.
* emit one more diagnostic with the required "program_name: " prefixJim Meyering2011-05-251-2/+2
| | | | | | * src/util.c: Include "error.h". (ask): Use error, not perror. The latter would not have included the usual "program name: " prefix.
* remove side effect from assertJim Meyering2011-05-251-1/+2
| | | | * src/util.c (parse_c_string): Don't increment "s" in assert.
* explicitly ignore close return value to placate static analyzersJim Meyering2011-05-252-1/+3
| | | | | | | * src/util.c: Include "ignore-value.h". (ask): Use ignore_value to tell tools that yes, we really do mean to ignore any close failure on this error path. * bootstrap.conf (gnulib_modules): Add ignore-value.
* plug a leak in fetchnameJim Meyering2011-05-251-0/+1
| | | | * src/util.c (fetchname): Don't leak "timestr" when returning early.
* avoid a used-uninitialized error in fetchnameAndreas Gruenbacher2011-05-251-3/+2
| | | | | | | | * src/util.c (fetchname): Avoid a used-uninitialized error. Before, when "*t == '\n'", stamp.tv_nsec would have been used undefined. The fix is to set that member rather than stamp.tv_sec, which is already set to the desired value. This was reported by coverity.
* plug a leak in bestmatchJim Meyering2011-05-251-2/+7
| | | | * src/bestmatch.h (bestmatch): Don't leak V when returning early.
* maint: ignore more build artifactsJim Meyering2011-03-272-3/+7
|
* build: don't turn off -Wmissing-declarationsJim Meyering2011-03-271-2/+1
| | | | | | * configure.ac (WERROR_CFLAGS): Don't turn off -Wmissing-declarations and admit that it's not worth fixing the few warnings triggered by -Wmissing-format-attribute.
* build: don't turn off -Wmissing-prototypesJim Meyering2011-03-273-3/+2
| | | | | | * configure.ac (WERROR_CFLAGS): Don't turn off -Wmissing-prototypes. * src/pch.c (skip_hex_digits): Declare static. * src/bestmatch.h (bestmatch): Likewise.
* maint: remove now-unneeded macro definitionsJim Meyering2011-03-274-13/+1
| | | | | | | | | | * bootstrap.conf (gnulib_modules): Include gnulib's signal module, so that signal.h guarantees definition of certain macros. * src/util.c (SIG_BLOCK, SIG_UNBLOCK, SIG_SETMASK): Remove definition. Now, gnulib guarantees that these are defined. * src/common.h (SIZE_MAX): Likewise. * cfg.mk (local-checks-to-skip): Enable the sc_prohibit_always-defined_macros check, now that it passes.
* maint: use gnulib's progname moduleJim Meyering2011-03-274-4/+3
| | | | | | | | | | * src/patch.c (main): Call set_program_name rather than initializing program_name explicitly. * src/common.h: Include progname.h rather than declaring the extern, program_name. * bootstrap.conf (gnulib_modules): Add progname. * cfg.mk (local-checks-to-skip): Remove sc_program_name, thus enabling this test.
* maint: update bootstrap and gnulib submoduleJim Meyering2011-03-274-66/+270
| | | | | | * bootstrap: Update from gnulib. * cfg.mk (local-checks-to-skip): Remove (thus, enable) sc_copyright_check, now that the gnulib submodule is up to date.
* maint: update copyright date year ranges to include 2011Jim Meyering2011-03-2750-61/+61
| | | | | * bootstrap.conf (gnulib_modules): Add update-copyright. Run "make update-copyright".
* build: avoid three gcc warningsJim Meyering2011-03-273-4/+2
| | | | | | * src/patch.c (mangled_patch): Add "noreturn" attribute. * src/pch.h (pch_timestamp): Remove ignored "const" attribute. * src/version.c (XTERN): Remove unused #undef and #define.
* tests: temporarily disable failing syntax-check rulesJim Meyering2011-03-271-0/+17
| | | | | | * cfg.mk (local-checks-to-skip): Define, to skip all of the currently-failing syntax-check rules. We'll reenable them one by one, as problems are addressed.
* build: accept new configure-time option --enable-gcc-warningsJim Meyering2011-03-264-0/+96
| | | | | | * configure.ac: Enable many options. * bootstrap.conf (gnulib_modules): Add manywarnings. * src/Makefile.am (AM_CFLAGS): Use $(WARN_CFLAGS) and $(WERROR_CFLAGS).
* maint: avoid non-portable use of test -aJim Meyering2011-03-264-5/+5
| | | | | | | | | | With these changes, "make sc_prohibit_test_minus_ao" almost passes. Uses of "test -o" remain. Note: unchecked uses of test -ot/-nt also remain. * tests/empty-files: Use "test C1 && test C2", not "test C1 -a C2" * tests/merge: Likewise. * tests/symlinks: Likewise. * tests/test-lib.sh: Likewise.
* maint: allow the sc_prohibit_empty_lines_at_EOF test to passJim Meyering2011-03-261-1/+0
| | | | * tests/test-lib.sh: Remove empty line at end of file.
* maint: remove trailing blanksJim Meyering2011-03-266-15/+22
| | | | | | | | | | * bootstrap.conf: Remove trailing blanks. * tests/reject-format: Define a dummy, empty variable, and use it in here-doc to protect required trailing blanks from accidental removal. * tests/no-newline-triggers-assert: Likewise. * tests/preserve-c-function-names: Likewise. * tests/create-delete: Likewise. * tests/global-reject-files: Complete a sentence that ended in a space.
* maint: add some m4 quotingJim Meyering2011-03-261-2/+2
| | | | * m4/setmode.m4 (AC_FUNC_SETMODE_DOS): Use proper M4 quoting.
* maint: arrange for the sc_require_config_h_first test to passJim Meyering2011-03-263-0/+23
| | | | | | | | * cfg.mk: New file, to configure maint.mk. * Makefile.am (EXTRA_DIST): Add, so the new file is distributed. (config_h_header): Define, to make the sc_require_config_h_first syntax-check test pass. * pc/chdirsaf.c: Include <config.h>.
* maint: use gnulib's maintainer-makefile moduleJim Meyering2011-03-261-0/+1
| | | | * bootstrap.conf (gnulib_modules): Add maintainer-makefile.
* doc: update README-hackingJim Meyering2011-03-211-13/+51
| | | | | | * README-hacking: Update from coreutils, including mention of how to use vc-dwim to git-commit efficiently and safely using a non-VC'd ChangeLog file.
* build: update gnulib submodule to latestJim Meyering2011-03-212-0/+2
|
* do not version-control ChangeLog; instead, generate it from git logJim Meyering2011-03-174-1/+23
| | | | | | | | | | | | | | With this change, all ChangeLog entries going forward are generated into a file named ChangeLog in each distribution tarball. ChangeLog entries prior to today's date are in ChangeLog-2011. * Makefile.am (gen-ChangeLog): New rule. (dist-hook): Depend on it. (EXTRA_DIST): Add ChangeLog-2011. * ChangeLog-2011: Renamed from ChangeLog * ChangeLog: Remove file. * .gitignore: Ignore ChangeLog. * bootstrap.conf: Ensure that ChangeLog exists. (gnulib_modules): Add gitlog-to-changelog.
* maint: remove unnecessary tests before freeJim Meyering2011-02-214-18/+18
| | | | | | | | There is no longer any need to avoid "free(NULL)" (since SunOS 4), so replace each "if (P) free (P);" with "free (P);". * src/inp.c (get_input_file): Remove unnecessary test-before-free. * src/patch.c (get_some_switches): Likewise. * src/pch.c (intuit_diff_type, pch_swap): Likewise.
* don't warn twice about the same invalid file nameJim Meyering2011-02-173-1/+40
| | | | | * src/pch.c (name_is_valid): Don't warn about the same name twice. * tests/bad-filenames (emit_patch): Exercise the new code.
* Ignore dangerous filenames instead of failing immediatelyAndreas Gruenbacher2011-02-165-28/+120
| | | | | | | | | | * src/pch.c (name_is_valid): New function. (intuit_diff_type, best_name): Use name_is_valid() here. (strip_leading_slashes): Remove name validation tests from here. * tests/bad-filenames: Add more tests for covering more of the file name guessing corner cases in intuit_diff_type(), update the existing tests. * NEWS: Update.
* Fix use of uninitialized outstAndreas Gruenbacher2011-02-152-3/+9
| | | | | * src/patch.c (main): Fix use of initialized outst and add an additional assert. Reported by Jim Meyering.