The NetBSD Project

CVS log for src/usr.bin/make/unit-tests/modmisc.exp

[BACK] Up to [cvs.NetBSD.org] / src / usr.bin / make / unit-tests

Request diff between arbitrary revisions


Default branch: MAIN


Revision 1.48 / (download) - annotate - [select for diffs], Fri Jun 16 07:20:45 2023 UTC (8 months, 2 weeks ago) by rillig
Branch: MAIN
CVS Tags: HEAD
Changes since 1.47: +0 -1 lines
Diff to previous 1.47 (colored)

tests/make: clean up tests for the ':M' and ':S' modifiers

Revision 1.47 / (download) - annotate - [select for diffs], Sun Dec 20 19:29:06 2020 UTC (3 years, 2 months ago) by rillig
Branch: MAIN
CVS Tags: netbsd-10-base, netbsd-10-0-RC5, netbsd-10-0-RC4, netbsd-10-0-RC3, netbsd-10-0-RC2, netbsd-10-0-RC1, netbsd-10, cjep_sun2x-base1, cjep_sun2x-base, cjep_sun2x, cjep_staticlib_x-base1, cjep_staticlib_x-base, cjep_staticlib_x
Changes since 1.46: +0 -1 lines
Diff to previous 1.46 (colored)

make(1): move tests for indirect modifiers around

The next commit will error out on unknown modifiers and influence the
exit status.  The test modmisc.mk contains both parse time tests and run
time tests.  To prevent the latter from being run, the parse error is
moved to varmod-indirect.mk, which only contains parse time tests.

Revision 1.46 / (download) - annotate - [select for diffs], Sun Aug 23 15:18:43 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.45: +0 -3 lines
Diff to previous 1.45 (colored)

make(1): move tests for the :tu and :_ modifiers into separate files

Revision 1.45 / (download) - annotate - [select for diffs], Sun Aug 23 15:13:21 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.44: +0 -21 lines
Diff to previous 1.44 (colored)

make(1): move tests for :hash :range :localtime into separate files

Revision 1.44 / (download) - annotate - [select for diffs], Sun Aug 23 15:09:15 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.43: +0 -4 lines
Diff to previous 1.43 (colored)

make(1): move tests for :H :E :R :T into separate files

Revision 1.43 / (download) - annotate - [select for diffs], Sun Aug 16 12:48:55 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.42: +0 -20 lines
Diff to previous 1.42 (colored)

make(1): move tests for the :gmtime and ::= modifiers to separate files

Revision 1.42 / (download) - annotate - [select for diffs], Sun Aug 16 12:30:45 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.41: +0 -94 lines
Diff to previous 1.41 (colored)

make(1): move tests for :S, :C, :@ from modmisc to their separate tests

Revision 1.41 / (download) - annotate - [select for diffs], Sun Aug 9 08:03:31 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.40: +27 -0 lines
Diff to previous 1.40 (colored)

make(1): add test that demonstrates possible delimiters for :S

Revision 1.40 / (download) - annotate - [select for diffs], Sun Aug 9 07:41:14 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.39: +1 -0 lines
Diff to previous 1.39 (colored)

make(1): explain the mod-subst-dollar test

Revision 1.39 / (download) - annotate - [select for diffs], Sun Aug 9 07:18:03 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.38: +1 -0 lines
Diff to previous 1.38 (colored)

make(1): add another test case for mod-subst-dollar

Having only 8 dollar signs does not prove anything.  It might still be
the result of 3 substitutions in a row.  Having 5 substitutions in a row
is less likely though.

Revision 1.38 / (download) - annotate - [select for diffs], Sat Aug 8 13:09:55 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.37: +3 -0 lines
Diff to previous 1.37 (colored)

make(1): add test for the :gmtime modifier with indirect time

Revision 1.37 / (download) - annotate - [select for diffs], Sat Aug 8 13:03:13 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.36: +2 -0 lines
Diff to previous 1.36 (colored)

make(1): add test for unknown sub-modifier of the :S modifier

Revision 1.36 / (download) - annotate - [select for diffs], Fri Aug 7 20:10:35 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.35: +4 -0 lines
Diff to previous 1.35 (colored)

make(1): add tests for creative variable names in the :@ modifier

Revision 1.35 / (download) - annotate - [select for diffs], Sat Aug 1 18:14:08 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.34: +2 -2 lines
Diff to previous 1.34 (colored)

make(1): fix error message when regcomp fails

If regcomp fails, it's not the regex substitution that is erroneous, but
the regex compilation.

Revision 1.34 / (download) - annotate - [select for diffs], Sat Aug 1 17:20:42 2020 UTC (3 years, 6 months ago) by rillig
Branch: MAIN
Changes since 1.33: +1 -0 lines
Diff to previous 1.33 (colored)

make(1): add tests for indirect modifiers

Revision 1.33 / (download) - annotate - [select for diffs], Fri Jul 31 14:36:58 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.32: +2 -0 lines
Diff to previous 1.32 (colored)

make(1): add test for chaining the :S modifier without colon

Revision 1.32 / (download) - annotate - [select for diffs], Wed Jul 29 21:35:35 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.31: +1 -0 lines
Diff to previous 1.31 (colored)

make(1): remove unnecessary assignments to st->next

No change in the size of the resulting binary.  Apparently GCC already
knew these assignments were redundant.

Revision 1.31 / (download) - annotate - [select for diffs], Wed Jul 29 18:48:47 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.30: +25 -0 lines
Diff to previous 1.30 (colored)

make(1): add unit tests for parse errors in modifiers

Revision 1.30 / (download) - annotate - [select for diffs], Mon Jul 27 21:54:25 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.29: +2 -0 lines
Diff to previous 1.29 (colored)

make(1): replace macros with functions

Having the hidden parameter st->endc in the macro made it unnecessarily
difficult to understand the code.

Revision 1.29 / (download) - annotate - [select for diffs], Sun Jul 26 11:39:55 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.28: +15 -15 lines
Diff to previous 1.28 (colored)

make(1): use consistent test names in modmisc.mk

Revision 1.28 / (download) - annotate - [select for diffs], Sat Jul 25 20:35:35 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.27: +1 -0 lines
Diff to previous 1.27 (colored)

make(1): add test for brk_string with more than 50 words

Revision 1.27 / (download) - annotate - [select for diffs], Thu Jul 23 19:49:39 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.26: +0 -2 lines
Diff to previous 1.26 (colored)

make(1): fix wrong warning about missing delimiter

Revision 1.26 / (download) - annotate - [select for diffs], Thu Jul 23 19:46:55 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.25: +3 -0 lines
Diff to previous 1.25 (colored)

make(1): demonstrate wrong error message about missing delimiter

The current practice of setting st->delim at the beginning of each
modifier and resetting it again at the end produces too many error
messages.  In this case, there is no missing delimiter.

Revision 1.25 / (download) - annotate - [select for diffs], Tue Jul 21 23:19:46 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.24: +3 -0 lines
Diff to previous 1.24 (colored)

make(1): add test for :S,,sep

The empty string does not match anywhere.  In other implementations of
string replacement, an empty string matches at every position of the
source string.

This only works for the :S modifier.  The :C modifier does not allow an
empty search pattern.

Revision 1.24 / (download) - annotate - [select for diffs], Tue Jul 21 21:34:41 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.23: +3 -0 lines
Diff to previous 1.23 (colored)

make(1): add test for :Q and .newline

Revision 1.23 / (download) - annotate - [select for diffs], Mon Jul 20 19:53:40 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.22: +4 -4 lines
Diff to previous 1.22 (colored)

make(1): fix edge cases with $ at the end of the :@ modifier

In both parts of the :@ modifier, by passing &pflags to
ParseModifierPart, a final $ was interpreted as an anchor, which only
makes sense in the :S and :C modifiers.

This edge case is neither used by src nor by pkgsrc, except for the unit
tests that have been adjusted.

Revision 1.22 / (download) - annotate - [select for diffs], Mon Jul 20 19:03:25 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.21: +4 -0 lines
Diff to previous 1.21 (colored)

make(1): add test for nested assignments

This test demonstrates that in ApplyModifier_Assign, the call to
ParseModifierPart must get its pflags.

Revision 1.21 / (download) - annotate - [select for diffs], Mon Jul 20 16:12:02 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.20: +1 -0 lines
Diff to previous 1.20 (colored)

make(1): add test for :tu modifier with spaces

Revision 1.20 / (download) - annotate - [select for diffs], Mon Jul 20 14:50:41 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.19: +1 -1 lines
Diff to previous 1.19 (colored)

make(1): make modifier handling simpler

Implementing a modifier such as :S or :M should not be concerned with
separating the words of the resulting string.  Ideally this should be
done in the same way by all modifiers.

Before, the :R (filename root) modifier added a separator even if the
resulting filename root was an empty string.  The chances that this
change in behavior breaks anything are epsilon.

The :@ modifier, if it appeared after a :ts modifier, did not use the
word separator from the :ts modifier (which all other modifiers do) but
always added a space.  This behavior has been preserved for now.  It's an
unnecessary inconsistency though.

In contrast to Buffer, the newly added SepBuf uses size_t for memory
sizes and also uses the conventional parameter order (mem, memsize)
instead of the unusual (memsize, mem).

Revision 1.19 / (download) - annotate - [select for diffs], Sun Jul 19 20:49:44 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.18: +8 -0 lines
Diff to previous 1.18 (colored)

make(1): add tests for the ::= modifiers

Revision 1.18 / (download) - annotate - [select for diffs], Sun Jul 19 19:36:20 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.17: +1 -0 lines
Diff to previous 1.17 (colored)

make(1): add tests for maximum number of capturing groups in :C

Revision 1.17 / (download) - annotate - [select for diffs], Sun Jul 19 19:27:08 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.16: +8 -0 lines
Diff to previous 1.16 (colored)

make(1): fix off-by-one error in :C modifier

Previously this off-by-one error had invoked undefined behavior.
Until today there was no corresponding unit test though.

Revision 1.16 / (download) - annotate - [select for diffs], Sun Jul 19 17:24:22 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.15: +10 -0 lines
Diff to previous 1.15 (colored)

make(1): demonstrate off-by-one bug in :C modifier

Revision 1.15 / (download) - annotate - [select for diffs], Sun Jul 19 15:16:22 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.14: +3 -0 lines
Diff to previous 1.14 (colored)

make(1): add test for stray dollar signs in :@ modifier

Revision 1.14 / (download) - annotate - [select for diffs], Sat Jul 4 22:17:09 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.13: +1 -1 lines
Diff to previous 1.13 (colored)

make(1): ignore system-specific error message from regcomp in test

NetBSD 8 has: repetition-operator operand invalid
Ubuntu has: Invalid preceding regular expression

Revision 1.13 / (download) - annotate - [select for diffs], Sat Jul 4 18:54:18 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.12: +16 -0 lines
Diff to previous 1.12 (colored)

make(1): add tests for dollar characters in modifiers

Revision 1.12 / (download) - annotate - [select for diffs], Sat Jul 4 17:41:04 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.11: +1 -1 lines
Diff to previous 1.11 (colored)

make(1): expand the test for variable expansion in the :@ modifier

Revision 1.11 / (download) - annotate - [select for diffs], Sat Jul 4 17:10:33 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.10: +1 -0 lines
Diff to previous 1.10 (colored)

make(1): add debugging output for :@ modifier

The inner working of the :@ modifier is quite tricky.  To understand what
really happens, it helps to look at each word as it is being modified.

Revision 1.10 / (download) - annotate - [select for diffs], Sat Jul 4 16:15:21 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.9: +1 -0 lines
Diff to previous 1.9 (colored)

make(1): add test for an obscure edge case of using the :@ modifier

Revision 1.9 / (download) - annotate - [select for diffs], Sat Jul 4 09:21:30 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.8: +13 -0 lines
Diff to previous 1.8 (colored)

make(1): add tests for the :S and :C modifiers

Revision 1.8 / (download) - annotate - [select for diffs], Fri Jul 3 22:10:42 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.7: +6 -0 lines
Diff to previous 1.7 (colored)

make(1): clean up documentation, small refactorings for variables

- document the callback for VarModify
- clearly mark the callbacks
- shorten the documentation for some callback functions
- fix the documentation of VarSYSVMatch
- remove unnecessary null check from VarLoopExpand
- add test for applying modifiers to empty strings

Revision 1.7 / (download) - annotate - [select for diffs], Fri Jul 3 19:21:11 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.6: +4 -4 lines
Diff to previous 1.6 (colored)

make(1): add expected test result for modmisc

Revision 1.6 / (download) - annotate - [select for diffs], Fri Jul 3 18:41:50 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.5: +4 -4 lines
Diff to previous 1.5 (colored)

make(1): fix bug in :E modifier, introduced in r1.239 today

That case was not covered by the tests before.

Revision 1.5 / (download) - annotate - [select for diffs], Fri Jul 3 15:41:37 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.4: +1 -0 lines
Diff to previous 1.4 (colored)

make(1): add test for :R modifier

Revision 1.4 / (download) - annotate - [select for diffs], Fri Jul 3 15:33:01 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.3: +3 -2 lines
Diff to previous 1.3 (colored)

make(1): add more tests for :H, :T and :E modifiers

Revision 1.3 / (download) - annotate - [select for diffs], Fri Jul 3 15:22:35 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.2: +2 -4 lines
Diff to previous 1.2 (colored)

make(1): add test for :H and :T modifiers with several words

Revision 1.2 / (download) - annotate - [select for diffs], Fri Jul 3 14:59:17 2020 UTC (3 years, 7 months ago) by rillig
Branch: MAIN
Changes since 1.1: +4 -0 lines
Diff to previous 1.1 (colored)

make(1): add missing test for the :H and :T modifiers

Revision 1.1 / (download) - annotate - [select for diffs], Thu Aug 21 13:44:51 2014 UTC (9 years, 6 months ago) by apb
Branch: MAIN
CVS Tags: prg-localcount2-base3, prg-localcount2-base2, prg-localcount2-base1, prg-localcount2-base, prg-localcount2, phil-wifi-base, phil-wifi-20200421, phil-wifi-20200411, phil-wifi-20200406, phil-wifi-20191119, phil-wifi-20190609, phil-wifi, pgoyette-localcount-base, pgoyette-localcount-20170426, pgoyette-localcount-20170320, pgoyette-localcount-20170107, pgoyette-localcount-20161104, pgoyette-localcount-20160806, pgoyette-localcount-20160726, pgoyette-localcount, pgoyette-compat-merge-20190127, pgoyette-compat-base, pgoyette-compat-20190127, pgoyette-compat-20190118, pgoyette-compat-1226, pgoyette-compat-1126, pgoyette-compat-1020, pgoyette-compat-0930, pgoyette-compat-0906, pgoyette-compat-0728, pgoyette-compat-0625, pgoyette-compat-0521, pgoyette-compat-0502, pgoyette-compat-0422, pgoyette-compat-0415, pgoyette-compat-0407, pgoyette-compat-0330, pgoyette-compat-0322, pgoyette-compat-0315, pgoyette-compat, perseant-stdc-iso10646-base, perseant-stdc-iso10646, netbsd-9-base, netbsd-9-3-RELEASE, netbsd-9-2-RELEASE, netbsd-9-1-RELEASE, netbsd-9-0-RELEASE, netbsd-9-0-RC2, netbsd-9-0-RC1, netbsd-9, netbsd-8-base, netbsd-8-2-RELEASE, netbsd-8-1-RELEASE, netbsd-8-1-RC1, netbsd-8-0-RELEASE, netbsd-8-0-RC2, netbsd-8-0-RC1, netbsd-8, matt-nb8-mediatek-base, matt-nb8-mediatek, localcount-20160914, is-mlppp-base, is-mlppp, bouyer-socketcan-base1, bouyer-socketcan-base, bouyer-socketcan

Give each group of tests its own output file.

* Rename each sub-makefile to *.mk;
* Add a *.exp file of expected output for each sub-makefile;
* Remove test.exp, which is replaced by all the other *.exp files.
* Use suffix rules to generate *.rawout and *.out files for
  each test case.
* Rewrite the test and accept targets to adapt to the new way.

The old (now removed) test.exp file is almost identical to the
concatenation (in the correct order) of all the new *.exp files.  There
are expected differences in makefile names embedded in the output, and
the new "exit status" lines.  Some old "*** Error code 1 (ignored)"
lines are also removed (replaced by new "exit status 1" lines).

This form allows you to request diff's between any two revisions of a file. You may select a symbolic revision name using the selection box or you may type in a numeric name using the type-in text box.




CVSweb <webmaster@jp.NetBSD.org>