303975 |
11-Aug-2016 |
gjb |
Copy stable/11@r303970 to releng/11.0 as part of the 11.0-RELEASE cycle.
Prune svn:mergeinfo from the new branch, and rename it to RC1.
Update __FreeBSD_version.
Use the quarterly branch for the default FreeBSD.conf pkg(8) repo and the dvd1.iso packages population.
Approved by: re (implicit) Sponsored by: The FreeBSD Foundation |
303064 |
20-Jul-2016 |
pfg |
MFC r302973: sed(1): Fix off by one introduced in r299211.
Detected by running the gsed tests.
Submitted by: Mikhail Teterin PR: 195929 Approved by: re (gjb)
|
302408 |
08-Jul-2016 |
gjb |
Copy head@r302406 to stable/11 as part of the 11.0-RELEASE cycle. Prune svn:mergeinfo from the new branch, as nothing has been merged here.
Additional commits post-branch will follow.
Approved by: re (implicit) Sponsored by: The FreeBSD Foundation
|
301138 |
01-Jun-2016 |
pfg |
sed(1): Fix a mismatch and sync with the OpenBSD's commit.
This was causing some strange behaviour.
Reported by: olivier Obtained from: OpenBSD (CVS rev. 1.28)
|
300684 |
25-May-2016 |
pfg |
sed: convert sed to use REG_STARTEND more explicitly.
Summarizing the findings in the OpenBSD list:
This solves a reproduceable issue with very recent Mesa where REG_NOTBOL combined with a match at the begin of the string causes our regex library to treat the word as not begin of word.
Thanks to Martijn van Duren and Ingo Schwarze for taking the time to solve this in the least invasive way.
PR: 209352, 209387 Taken from: openbsd-tech (Martijn van Duren) MFC after: 1 month
|
299377 |
10-May-2016 |
pfg |
sed.1: Correction for the case insensitive case.
Use the capital I instead of the lowercase.
Submitted by: Mikhail T. PR: 195929 MFC after: 2 weeks
|
299294 |
10-May-2016 |
pfg |
Revert r299279: Simplify redundant malloc'ing in sed -e.
It is causing havoc in the ports tree:
===> Configuring for wxsvg-1.5.7 sed: 1: "/gcc_dir=\\`/s/gcc /$CC /": bad flag in substitute command: '/' *** Error code 1
===> Patching for vips-8.3.1 sed: 1: "1s|^#![[:space:]]*/usr/ ...": bad flag in substitute command: 's' *** Error code 1
PR: 195929 Reported by: danilo
|
299279 |
09-May-2016 |
pfg |
Simplify redundant malloc'ing in sed -e.
When encountering an -e argument, sed currently mallocs a string to COPY the optarg -- with '\n' appended. The appendage does not seem necessary -- indeed, the same call to add_compunit processing the sole command (given without -e) passes the *argv verbatim: without making a copy, and without appending newline.
This matches what is done in other BSDs.
Submitted by: Mikhail T. PR: 195929 MFC after: 2 weeks
|
299211 |
07-May-2016 |
pfg |
sed: rewrite the main loop.
Rewrite the main loop of the "sed s/..." command, shortening it by ten lines and simplifying it by removing the switch statement implementing /g, /1, and /2 separately and repetitively.
This will be needed to bring a fix from OpenBSD later.
Obtained from: OpenBSD (schwarze CVS Rev. 1.18) MFC after: 3 weeks
|
299094 |
04-May-2016 |
ngie |
Merge ^/user/ngie/release-pkg-fix-tests to unbreak how test files are installed after r298107
Summary of changes:
- Replace all instances of FILES/TESTS with ${PACKAGE}FILES. This ensures that namespacing is kept with FILES appropriately, and that this shouldn't need to be repeated if the namespace changes -- only the definition of PACKAGE needs to be changed - Allow PACKAGE to be overridden by callers instead of forcing it to always be `tests`. In the event we get to the point where things can be split up enough in the base system, it would make more sense to group the tests with the blocks they're a part of, e.g. byacc with byacc-tests, etc - Remove PACKAGE definitions where possible, i.e. where FILES wasn't used previously. - Remove unnecessary TESTSPACKAGE definitions; this has been elided into bsd.tests.mk - Remove unnecessary BINDIRs used previously with ${PACKAGE}FILES; ${PACKAGE}FILESDIR is now automatically defined in bsd.test.mk. - Fix installation of files under data/ subdirectories in lib/libc/tests/hash and lib/libc/tests/net/getaddrinfo - Remove unnecessary .include <bsd.own.mk>s (some opportunistic cleanup)
Document the proposed changes in share/examples/tests/tests/... via examples so it's clear that ${PACKAGES}FILES is the suggested way forward in terms of replacing FILES. share/mk/bsd.README didn't seem like the appropriate method of communicating that info.
MFC after: never probably X-MFC with: r298107 PR: 209114 Relnotes: yes Tested with: buildworld, installworld, checkworld; buildworld, packageworld Sponsored by: EMC / Isilon Storage Division
|
298440 |
21-Apr-2016 |
pfg |
kernel: use our nitems() macro when it is available through param.h.
No functional change, only trivial cases are done in this sweep,
Discussed in: freebsd-current
|
298107 |
16-Apr-2016 |
gjb |
Merge the projects/release-pkg branch to head.
This allows packaging the base system with pkg(8), including but not limited to providing the ability to provide upstream binary update possibilities for non-tier-1 architectures.
This merge is a requirement of the 11.0-RELEASE, and as such, thank you to everyone that has tested the project branch.
Documentation in build(7) etc. is still somewhat sparse, but updates to those parts will follow.
Sponsored by: The FreeBSD Foundation
|
297602 |
06-Apr-2016 |
pfg |
Fix sed functions 'i' and 'a' from discarding leading white space.
This appears to be implementation dependent but convenient and makes our sed behave more like GNU sed.
Given that it is not the historic behavior, bump FreeBSD_version should userland/ports somehow depend on it.
Obtained from: NetBSD (bin/49872)
Reviewed by: bdrewery PR: 208554 Merge after: NEVER
|
296587 |
09-Mar-2016 |
bdrewery |
DIRDEPS_BUILD: Connect MK_TESTS.
Sponsored by: EMC / Isilon Storage Division
|
289677 |
21-Oct-2015 |
eadler |
Fix a ton of speelling errors
arc lint is helpful
Reviewed By: allanjude, wblock, #manpages, chris@bsdjunk.com Differential Revision: https://reviews.freebsd.org/D3337
|
289172 |
12-Oct-2015 |
ngie |
Refactor the test/ Makefiles after recent changes to bsd.test.mk (r289158) and netbsd-tests.test.mk (r289151)
- Eliminate explicit OBJTOP/SRCTOP setting - Convert all ad hoc NetBSD test integration over to netbsd-tests.test.mk - Remove unnecessary TESTSDIR setting - Use SRCTOP where possible for clarity
MFC after: 2 weeks Sponsored by: EMC / Isilon Storage Divison
|
284345 |
13-Jun-2015 |
sjg |
Add META_MODE support.
Off by default, build behaves normally. WITH_META_MODE we get auto objdir creation, the ability to start build from anywhere in the tree.
Still need to add real targets under targets/ to build packages.
Differential Revision: D2796 Reviewed by: brooks imp
|
277811 |
27-Jan-2015 |
pfg |
sed: fix pasto from previous r277802.
MFC after: 3 days
|
277802 |
27-Jan-2015 |
pfg |
Fix resource leak and dereference after NULL.
process.c: Protect access against NULL.
main.c: Prevent outfile overwrite resource leak.
CID: 271181 CID: 1006930
Obtained from: NetBSD MFC after: 3 days
|
277099 |
13-Jan-2015 |
pfg |
Replace __inline GNUism with the standard inline.
MFC after: 1 week
|
276909 |
10-Jan-2015 |
jlh |
Add a regression test for PR 192108.
I won't go through the hassle of MFCing it since I expect all changes to go first through HEAD anyway.
PR: 192108
|
276837 |
08-Jan-2015 |
pfg |
Partial revert of r276832:
Do not bump the warns level as it still breaks the gcc build on sparc64
Reported by: jenkins
|
276832 |
08-Jan-2015 |
pfg |
sed: Address warnings with clang and gcc48.
MFC after: 2 weeks
|
275838 |
16-Dec-2014 |
pfg |
sed: Bounds check the file path used in the 'w' command.
Modified version of a diff from Sebastien Marie to prevent a crash found with the afl fuzzer.
Obtained from: OpenBSD (CVS Rev. 1.37) MFC after: 1 week
|
269729 |
08-Aug-2014 |
dumbbell |
sed(1): Don't force a newline on last line, if input stream doesn't have one
While here, change how we check if the current line is the last one. Before, we just checked if there were more files after the current one. Now, we check the actual content of those files: they files may not have a line at all. This matches the definition of the "last line" by the Open Group.
The new behavior is closer to GNU sed.
PR: 160745 Phabric: https://phabric.freebsd.org/D431 Reviewed by: jilles Approved by: jilles Exp-run by: antoine
|
269302 |
30-Jul-2014 |
jlh |
Fix relative numerical addressing (addr,+N).
As a bonus the patch untangles a bit the logic and makes the code easier to grasp.
PR: 192108 MFC after: 1 week
|
268203 |
03-Jul-2014 |
pfg |
More sed(1) usage fixing: the extension -i is not optional.
Pointed out by: jmallet MFC after: 3 days
|
268193 |
02-Jul-2014 |
pfg |
Fix sed(1) usage: the extension -i is not an optional.
MFC after: 3 days
|
267773 |
23-Jun-2014 |
bapt |
use .Mt to mark up email addresses consistently (part3)
PR: 191174 Submitted by: Franco Fichtner <franco at lastsummer.de>
|
267703 |
21-Jun-2014 |
pfg |
Drop ifdef nonsense: just use setvbuf(3).
Pointed out by: kib, bde
|
267692 |
20-Jun-2014 |
pfg |
Add -u (unbuffered output) after GNU sed.
Obtained from: NetBSD MFC after: 1 week
|
265420 |
06-May-2014 |
imp |
Use src.opts.mk in preference to bsd.own.mk except where we need stuff from the latter.
|
263388 |
19-Mar-2014 |
jmmv |
Mark multi_test as requiring /usr/share/dict/words.
The file may not be present if MK_DICT=no. Pointed out by Casey Peel.
|
263227 |
16-Mar-2014 |
jmmv |
Migrate most of tools/regression/usr.bin/ to the new tests layout.
I'm starting with the easy cases. The leftovers need to be looked at a bit more closely.
Note that this change _does_ modify the code of the old tests. This is required in order to allow the code to locate the data files in the source directory instead of the current directory, because Kyua automatically changes the latter to a temporary directory.
Also note that at least one test is known to be broken here. Actually, the test is not really broken: it's marked as a TODO but unfortunately Kyua's TAP parser currently does not understand that. Will have to be fixed separately.
|
259132 |
09-Dec-2013 |
eadler |
Per the resolution of POSIX bug 0000779 (note 0002050) add support for using 'i' as a case insensitive flag.
PR: standards/184641 Requested by: David A. Wheeler <dwheeler@dwheeler.com> MFC After: 1 week
|
252231 |
26-Jun-2013 |
pfg |
sed: use getline() instead of fgetln().
In BSD, fgetln() available in libc but in Illumos the Solaris port had to include it internally. It also seems to have caused problems [1].
Aid portability by using getline() instead.
Reference: https://www.illumos.org/issues/3820 [1]
Submitted by: Johann 'Myrkraverk' Oskarsson <johann@myrkraverk.com> Reviewed by: dds MFC after: 2 weeks
|
233648 |
29-Mar-2012 |
eadler |
Remove trailing whitespace per mdoc lint warning
Disussed with: gavin No objection from: doc Approved by: joel MFC after: 3 days
|
226895 |
29-Oct-2011 |
ache |
Reword '}' description to sound more clear.
Submitted by: dougb MFC after: 7 days
|
226889 |
28-Oct-2011 |
ache |
Update '}' description to reflect reality (and POSIX)
PR: 96236 Submitted by: "Andreas Kohn" <andreas@syndrom23.de> MFC after: 7 days
|
217133 |
08-Jan-2011 |
jilles |
sed: Try hard links to make -i target available continually.
When creating a backup file, sed renamed the original before renaming the changed copy into place, leading to a short time when no file with the original name was present (usually only visible on SMP systems). Try creating the backup file using a hard link instead, avoiding this problem. If creating the hard link fails for any reason, fall back to the old rename method.
When not creating a backup file, sed already renamed the changed copy onto the original. This remains unchanged.
I am not adding the suppression of redundant fchown/fchmod to this commit, because FreeBSD appears to check this in the kernel (for msdosfs at least).
PR: bin/153261 Submitted by: Pedro F. Giffuni Reviewed by: dds (older version) Obtained from: Illumos MFC after: 2 weeks
|
205992 |
31-Mar-2010 |
imp |
Make -r mean exactly the same thing as -E for increased compatibility with GNU sed.
MFC after: 7 days
|
201386 |
02-Jan-2010 |
ed |
Build usr.bin/ with WARNS=6 by default.
Also add some missing $FreeBSD$ to keep svn happy.
|
200462 |
13-Dec-2009 |
delphij |
Revert most part of 200420 as requested, as more review and polish is needed.
|
200420 |
11-Dec-2009 |
delphij |
Remove unneeded header includes from usr.bin/ except contributed code.
Tested with: make universe
|
197362 |
20-Sep-2009 |
dds |
IEEE Std 1003.1, 2004 Edition states:
"The escape sequence '\n' shall match a <newline> embedded in the pattern space."
It is unclear whether this also applies to a \n embedded in a character class. Disable the existing handling of \n in a character class following Mac OS X, GNU sed version 4.1.5 with --posix, and SunOS 5.10 /usr/bin/sed.
Pointed by: Marius Strobl Obtained from: Mac OS X
|
197361 |
20-Sep-2009 |
dds |
Follow POSIX (IEEE Std 1003.1, 2004 Edition) in the implementation of the y (translate) command.
"If a backslash character is immediately followed by a backslash character in string1 or string2, the two backslash characters shall be counted as a single literal backslash character"
Pointed by: Marius Strobl Obtained from: Mac OS X
|
197356 |
20-Sep-2009 |
dds |
Allow [ to be used as a delimiter.
Pointed by: Marius Strobl Obtained from: Apple
|
192773 |
25-May-2009 |
brian |
Bump the document date to reflect the recent address range enhancements.
Suggested by: throdes
|
192732 |
25-May-2009 |
brian |
Implement "addr1,+N" ranges - not dissimilar to grep's -A switch.
PR: 134856 Submitted by: Jeremie Le Hen - jeremie at le-hen dot org
|
184854 |
11-Nov-2008 |
dds |
Fix the code to conform to the "or more" part of the following POSIX specification and regression test regress:25.
"A function can be preceded by one or more '!' characters, in which case the function shall be applied if the addresses do not select the pattern space."
MFC after: 2 weeks
|
184777 |
09-Nov-2008 |
hrs |
Add workaround for a back reference when no corresponding parenthesized subexpression is defined. For example, the following command line caused unexpected behavior like segmentation fault:
% echo test | sed -e 's/test/\1/'
PR: bin/126682 MFC after: 1 week
|
182616 |
01-Sep-2008 |
yar |
Correct a silly typo I introduced: withing -> within.
Noticed by: das
|
182107 |
24-Aug-2008 |
yar |
Explicitly tell that one needs to start a context address with a backslash if he/she wants to use a non-traditional delimiter, i.e., anything other than a slash. That is, /abc/ works as is, but xabcx needs to be spelled as \xabcx.
Add appropriate markup.
Bump Dd.
Checked with: IEEE Std 1003.1, 2004 Edition MFC after: 3 days
|
176126 |
09-Feb-2008 |
dwmalone |
WARNS fixes: 1) Add missing parens around assignment that is compared to zero. 2) Make some variables that only take non-negative values unsigned. 3) Some casts/type changes to fix other constness warnings. 4) Make one variable a const char *. 5) Make sure termwidth is positive, it doesn't make sense for it to be negative.
Approved by: dds
|
173437 |
08-Nov-2007 |
dds |
Fix visibility of "inplace". Found by: CScout
|
171284 |
06-Jul-2007 |
delphij |
Before doing compile_re() which needs a parameter to identify whether we should ignore case, determine the flag by calling compile_flags() first. Also, make sure that we obtain an initialized cmd->u.s buffer before processing further. We may want to refine this solution later, but for now, make the changes in order to unbreak world build after a sed(1) with rev. 1.29 of compile.c is installed.
Approved by: re (hrs)
|
171206 |
04-Jul-2007 |
ssouhlal |
Add case-insensitive matching to sed, using the 'I' flag, similarly to GNU sed. For example, sed /foo/Id sed s/foo/bar/Ig
Reviewed by: dds Approved by: re (hrs)
|
170609 |
12-Jun-2007 |
yar |
Drop the argument to the OUT macro because it can't emit anything but the pattern space anyway. Apply style(9) to the macro.
Tested with: md5(1)
|
170608 |
12-Jun-2007 |
yar |
Don't forget to clear out the hold space for each subsequent file when in -i mode so that each file gets a clean context of its own.
Add a regression test for the bug.
Tested with: regression tests
|
170605 |
12-Jun-2007 |
yar |
There is a symbolic antonym for REPLACE as a flag to cspace() and mf_fgets(): APPEND. So use it instead of a 0 constant for clarity.
Tested with: md5(1)
|
168922 |
21-Apr-2007 |
yar |
Don't forget to bump document date after changing the content.
|
168921 |
21-Apr-2007 |
yar |
Change the semantics of -i (in-place editing) so that it treats each file independently from other files. The new semantics are desired in the most of practical cases, e.g.: delete lines 5-9 from each file.
Keep the previous semantics of -i under a new option, -I, which uses a single continuous address space covering all files to edit in-place -- they are too cool to just drop them.
Add regression tests for -i and -I.
Approved by: dds Compared with: GNU sed Discussed on: -hackers MFC after: 2 weeks
|
168417 |
06-Apr-2007 |
yar |
style(9): Use a single space before $FreeBSD$ to avoid terminal line overflow.
Pointed out by: ru (The All-Seeing Eye)
|
168389 |
05-Apr-2007 |
yar |
Add an important detail to paragraph 12: the range is reset only if its second address is a line number.
|
168386 |
05-Apr-2007 |
yar |
Reflect the change in rev. 1.44 of process.c.
Add $FreeBSD$ to please commit_prep.pl.
|
168258 |
02-Apr-2007 |
yar |
Don't forget to close the range if we branched over its end and had no chance to match it by the 2nd address precisely. Otherwise the unclosed range would bogusly extend to the end of stream.
Add a basic regression test for the bug fixed. (This change also fixes the more complex case 5.3 from `multitest.t'.)
Compared with: SUN and GNU seds Tested by: regression tests MFC after: 1 week
|
168255 |
02-Apr-2007 |
yar |
Prevent foot-shooting in advance: Put the MATCH() macro's value in parentheses. The ?: operator has a remarkably low precedence, so expressions like (MATCH(foo) && bar) would have an unexpected meaning w/o the parentheses around MATCH().
Tested with: md5(1)
|
168211 |
01-Apr-2007 |
yar |
This trivial change should fix at least 3 similar bugs. All of them are related to the `c' function's need to know if we are at the actual end of the address range. (It must print the text not earlier than the whole pattern space was deleted.) It appears the only sed function with this requirement.
There is `lastaddr' set by applies(), which is to notify the `c' function, but it can't always help because it's false when we are hitting the end of file early. There is also a bug in applies() due to which `lastaddr' isn't set to true on degenerate ranges such as `$,$' or `N,$' if N appears the last line number.
Handling early EOF condition in applies() could look more logical, but it would effectively revert sed to the unreasonable behaviour rev. 1.26 of main.c fought against, as it would require lastline() be called for each line within each address range. So it's better to call lastline() only if needed by the `c' function.
Together with this change to sed go regression tests for the bugs fixed (c1-c3). A basic test of `c' (c0) is also added as it helped me to spot my own error.
Discussed with: dds Tested by: the regression tests MFC after: 1 week
|
168120 |
31-Mar-2007 |
yar |
Make the comment for cspace() match reality.
|
167556 |
14-Mar-2007 |
dds |
Test files repo-copied into tools/regression/usr.bin/sed and integrated into the regression testing framework.
|
158989 |
27-May-2006 |
krion |
Fix using "P" command on empty pattern space. If the "P" command is used once on a non-empty pattern space and then again on an empty pattern space, the second usage restores the pattern space length to the length that it had when the first "P" was used.
PR: bin/96052 Submitted by: Andrey Zholos <aaz@althenia.net> MFC after: 7 days
|
150513 |
24-Sep-2005 |
stefanf |
The function inplace_edit() doesn't exist anymore, remove the prototype.
Submitted by: Leonardo Chiquitto Filho
|
148692 |
04-Aug-2005 |
dds |
Bug fix: a numeric flag specification in the substitute command would cause the next substitute flag to be ignored. While working at it, detect and report overflows.
Reported by: Jingsong Liu MFC after: 1 week
|
147370 |
14-Jun-2005 |
ru |
Markup and wording fixes.
Approved by: re (blanket)
|
146055 |
10-May-2005 |
glebius |
Add option -l, to make output line buffered.
PR: bin/78692 Submitted by: Eugene Grosbein MFC after: 2 weeks
|
144840 |
09-Apr-2005 |
stefanf |
Remove unused variables.
|
143318 |
09-Mar-2005 |
stefanf |
Fix dubious C code construct.
|
141846 |
13-Feb-2005 |
ru |
Expand *n't contractions.
|
141563 |
09-Feb-2005 |
stefanf |
Include missing headers.
|
140368 |
17-Jan-2005 |
ru |
Added the EXIT STATUS section where appropriate.
|
140055 |
11-Jan-2005 |
ru |
Scheduled mdoc(7) sweep.
|
133385 |
09-Aug-2004 |
dds |
Per letter dated July 22, 1999 remove 3rd clause of Berkeley derived software (original contributor).
Reviewed by: imp
|
132275 |
17-Jul-2004 |
tjr |
Document the limitation that multibyte characters cannot be used as delimiters with the 's' and 'y' commands.
|
132147 |
14-Jul-2004 |
tjr |
Document the -E and -a options as being extensions.
|
132146 |
14-Jul-2004 |
tjr |
Update BUGS section to reflect current state of multibyte character support.
|
132145 |
14-Jul-2004 |
tjr |
Make the 'y' (translate) command aware of multibyte characters.
|
132083 |
13-Jul-2004 |
tjr |
Overhaul lputs() to deal with multibyte characters, characters that take up more than one column position, and null bytes.
|
131511 |
03-Jul-2004 |
tjr |
Document missing multibyte character support in utilities specified by POSIX.
|
131491 |
02-Jul-2004 |
ru |
Mechanically kill hard sentence breaks.
|
131472 |
02-Jul-2004 |
ru |
Removed trailing whitespace.
|
130039 |
03-Jun-2004 |
brian |
Plug a file descriptor leak.
When sed is asked to inline-edit files, it forgets to close the temporary file and runs out of descriptors for long command lines (assuming you reset kern.maxfilesperproc to something sane that's less than the number of files passed to sed).
|
128785 |
01-May-2004 |
smkelly |
A new version that does exponents and lots of other neat things. Update from the original author of math.sed.
Submitted by: K S Braunsdorf <sed@ksb.npcguild.org>
|
124127 |
04-Jan-2004 |
charnier |
Avoid using .Nm at the start of the sentence.
|
122084 |
04-Nov-2003 |
des |
Fix the case where the file is not in the current directory.
Discovered by: Vladimir Kravchenko <jimson@mostcom.ru> Pointy hat to: des
|
122049 |
04-Nov-2003 |
des |
Reimplement in-place editing in a slightly less disgusting manner. Also, make an effort to preserve the ownership and mode of the file we are editing.
Sponsored by: Registrar AS
|
122045 |
04-Nov-2003 |
des |
Whitespace cleanup
|
122044 |
04-Nov-2003 |
des |
ANSIfy
|
122043 |
04-Nov-2003 |
des |
Correct style bugs in the in-place editing code.
|
121915 |
02-Nov-2003 |
tjr |
Change the buffer length test in NEEDSP() so that it does not subtract one unsigned number from another potentially smaller one, leading to wraparound (and heap corruption, eventually).
PR: 58813 MFC after: 2 weeks
|
117901 |
22-Jul-2003 |
dds |
Fixed .Nm element, updated HISTORY section, added AUTHORS section.
Reviewed by: schweikh (mentor) MFC after: 2 weeks
|
117900 |
22-Jul-2003 |
dds |
Test 8.16 in sed.test tests the ability of a sed to handle an empty regular expression as the first argument to a substitute command. If used to test a sed which (erroneously) evaluates this at translation time rather than at execution time, the bugged sed is put into an infinite loop. This mode of failure seems excessive. Such a failing sed is the Free Software Foundation's sed 3.02.
The specific test was also not being executed for the BSD sed.
Both problems are now fixed.
PR: misc/25585 Submitted by: Walter Briscoe <w.briscoe@ponl.com> Approved by: schweikh (mentor) MFC after: 2 weeks
|
115871 |
05-Jun-2003 |
fanf |
Do not dribble zero bytes into the output, by replacing an obfuscated if whose true and false clauses were equivalent with a check that we are not about to stumble off the end of the line.
Reported by: peter Pointy hat to: fanf
|
115831 |
04-Jun-2003 |
fanf |
Fix substitution when the regex matches the zero-length string. There are two bugs: in the s///g case, the substitution didn't occur at the end of the line; in the s///N case, the code didn't count forwards along the line properly. See the sg, s3, s4, and s5 tests in src/tools/regression/usr.bin/sed/.
Reviewed by: tjr
|
108533 |
01-Jan-2003 |
schweikh |
Correct typos, mostly s/ a / an / where appropriate. Some whitespace cleanup, especially in troff files.
|
107276 |
26-Nov-2002 |
ru |
mdoc(7) police: markup polishing.
Approved by: re
|
103715 |
20-Sep-2002 |
eric |
Initialize a variable in order to fix when faced with picky compilers.
Sponsored by: Apple
|
101668 |
11-Aug-2002 |
tjr |
Correct boundary condition error in `D' and `P' commands when the last line of the pattern space is empty. Don't emit spurious newline when EOF is reached with the `N' command.
Pointed out by: Oleg Osyka MFC after: 1 week
|
100970 |
30-Jul-2002 |
fanf |
Fix some bugs in in-place editing: (1) errors from freopen were not reported correctly (2) large files were not handled correctly (3) read-only files broke things
MFC after: 1 week
|
100944 |
30-Jul-2002 |
tjr |
Ignore leading semicolons on commands; required by SUSv3.
Obtained from: NetBSD (kleink, Aymeric Vincent)
|
100359 |
19-Jul-2002 |
markm |
s/inline/__inline/
|
99573 |
08-Jul-2002 |
tjr |
Make cspace()'s second argument const. Make `escapes' in lputs() const.
|
99572 |
08-Jul-2002 |
tjr |
Silence WARNS=4 by passing a non-const buffer to regerror(). Since it doesn't actually use the buffer when errbuf_size == 0, pass NULL.
|
99400 |
04-Jul-2002 |
tjr |
Update another mention of <CR><CR> I missed the first time around (rev 1.2)
|
99355 |
03-Jul-2002 |
tjr |
Note that two carriage returns aren't required after entering the initial setup anymore (since process.c rev. 1.25). Add $FreeBSD$.
MFC after: 2 weeks
|
99352 |
03-Jul-2002 |
tjr |
Avoid checking whether each line is the last line of the file when we don't need to know. Instead, check when we are trying to match a "$" address. This does not change the way sed processes regular files, but makes it behave more sensibly when used interactively.
PR: 40101 MFC after: 2 weeks
|
99351 |
03-Jul-2002 |
tjr |
Don't let the pattern space become null if the `x' command is used when the hold space is null; some functions assume it's never null.
MFC after: 3 days
|
98807 |
25-Jun-2002 |
tjr |
Sort sections, and refer to sed as a utility, not a function.
|
98743 |
24-Jun-2002 |
tjr |
Fix off by one in `y' (transliterate) command which caused the last character of the pattern space not to be examined.
Noticed by: Motoyuki Konno <motoyuki@bsdclub.org>
|
98603 |
22-Jun-2002 |
tjr |
Now that the pattern space contains no trailing newline, modify the `l' command's output so it's the same as what SUSv3 specifies.
|
98601 |
22-Jun-2002 |
tjr |
Don't store newlines at the end of each line in the hold/pattern spaces, instead add the newline when the pattern space is printed. Make the `G' and `H' commands add a newline to the space before the data, remove bogus addition of newline from `x' command.
PR: 29790, 38195
|
98332 |
17-Jun-2002 |
sobomax |
Kill superfluous blank line.
|
98294 |
16-Jun-2002 |
tjr |
Correct the handling of (for example) the N command by only zeroing the input space in mf_fgets() if we reach the end of all input files.
|
98201 |
14-Jun-2002 |
tjr |
Exit with non-zero status if an input file can not be opened. This was broken by rev. 1.22.
|
98200 |
14-Jun-2002 |
sobomax |
Fix one serious bug and one potential problem with in-place editing code:
- original version of code worked incorrectly when more than one input files were specified - it was moving the last line from the 1st file to be the first line of the 2nd, last line of the 2nd to be the first line of the 3rd and so on;
- use mmap()->write() to create temporary file instead of malloc()->read()->write(), which was not only slower, but also did not bother to free allocated memory once backup file was created, potentially leading to memory exhausting when regex is applied to a big file or a large number of small ones.
|
98088 |
10-Jun-2002 |
tjr |
Refer to environ(7) for description of COLUMNS instead of describing it in-line. Document effects of locale environment variables in the same way.
|
98087 |
10-Jun-2002 |
tjr |
The 10 occurrence limit for the `s' command documented here has not existed for a very long time (or never did exist).
|
98086 |
10-Jun-2002 |
tjr |
Newlines are not escaped anymore.
|
98085 |
10-Jun-2002 |
tjr |
Don't write escape newlines with `l' command (SUSv3)
|
98082 |
10-Jun-2002 |
tjr |
Correctly handle global substitutions where the pattern is only "$", the EOL anchor, when the last input line does not end in a newline character. Picked up by the GNU sed test suite.
|
97808 |
04-Jun-2002 |
tjr |
Wrap lines correctly for the `l' command.
|
97802 |
04-Jun-2002 |
tjr |
Document the fact that the `l' command uses the COLUMNS environment variable (SUSv3)
|
97801 |
04-Jun-2002 |
tjr |
Ignore COLUMNS environment variable if it is empty (SUSv3)
|
97703 |
01-Jun-2002 |
green |
Fix a bug in sed(1)'s "s" command wherein if an escape ("\" character) was initiated at the last character of the line buffer, the Wrong Thing was done and sed barfed by interpreting the following NUL byte as a digit. Instead, pull up the next buffer and record that the "\" was last seen.
|
97544 |
30-May-2002 |
ru |
mdoc(7) police: typo in .Dd.
|
97238 |
24-May-2002 |
tjr |
If a file argument cannot be processed, process the remaining ones and exit non-zero (SUSv3)
|
96704 |
16-May-2002 |
trhodes |
More consistancy. file system > filesystem
|
96621 |
14-May-2002 |
green |
o Clean up tmp file usage a little by using mkstemp(3) instead of mktemp(3). It would be amazingly unlikely, but the former method could result in a symlink attack. A better solution would use ${TMPDIR}, though. o Make sed not overwrite old backup files with no warning.
|
96189 |
07-May-2002 |
jmallett |
Oops, update usage() to have -i.
|
96188 |
07-May-2002 |
jmallett |
Add handling for any nil-length string passed to -i for the backup extension.
Add a note that this is kinda-sorta dangerous to the manual page.
|
96185 |
07-May-2002 |
jmallett |
Un-shadow the `fname' variable.
|
96175 |
07-May-2002 |
jmallett |
Add a -i option to sed(1) to do inplace editing, to give us an alternative to Perl for such things. The key difference to Perl is that a backup extension *MUST* be specified, because on one hand it isn't recommended to have options which optionally take a parameter, and on the other hand, it'd be slightly unpleasent to implement proper handling for that.
The difference between this and the version posted to developers@ is that it does handle multiple files in argv after the getopt(3) handling "correctly", in that the inplace editing-specific code has been moved out to a function, and that function is used beyond the first file in our linked list.
This option has been documented as FreeBSD-specific in the manpage.
Reviewed by: developers@ (got feedback from: des, fanf, sobomax, roberto, obrien) MFC after: 1 week
|
95208 |
21-Apr-2002 |
keramida |
Remove duplicated text.
PR: docs/37287 Submitted by: Tony Finch <dot@dotat.at> MFC after: 3 days
|
94540 |
12-Apr-2002 |
sobomax |
Fix an ages-old bug in sed(1), which resulted in the absolutely valid substitution expressions in the form `s,[fooexp],[barexp],;...' treated as invalid when the third `,' is (_POSIX2_LINE_MAX * N)-th character in the line.
MFC after: 2 weeks
|
94012 |
07-Apr-2002 |
jmallett |
Fix sed(1) in the case where a last line is specified and hold space is not specified, and then the first part of the pattern space is deleted, when there are two or more input lines, as this results in subtraction of one from an unsigned integral value of '0'. That bogus value is used in one case for a loop (that will run far too many times in this case) and a function to search for a value within a specified range of memory, however now the range of memory is obscenely large and a segmentation fault will occur. This is fixed by checking for and appropriately handling a nil pattern space as if the specified search in memory failed, as indeed it obviously will with nil pattern space.
Submitted by: Tim J. Robbins <tim@robbins.dropbear.id.au> PR: bin/34813 Reviewed by: mike MFC after: 1 day
|
93860 |
05-Apr-2002 |
jmallett |
Fix sed(1) behaviour for 'G' when given null holdspace by making sure it contains a \n.
PR: misc/26153 Submitted by: ashp Reviewed by: mike Obtained from: NetBSD MFC after: 2 days
|
92922 |
22-Mar-2002 |
imp |
remove __P
|
90416 |
08-Feb-2002 |
markm |
Remove NO_WERRORs and WARNS=n's. To be revisited after GCC3.
|
87766 |
12-Dec-2001 |
markm |
WARNS=2 partial fix; use NO_WERROR to protect against some hard-to-fix warnings.
Use __FBSDID(), kill register keyword.
|
86193 |
08-Nov-2001 |
mikeh |
Don't allocate a zero byte segment.
PR: bin/11900 MFC after: 2 weeks
|
81687 |
15-Aug-2001 |
ru |
mdoc(7) police: utilize the new .Ex macro.
|
81251 |
07-Aug-2001 |
ru |
mdoc(7) police:
Avoid using parenthesis enclosure macros (.Pq and .Po/.Pc) with plain text. Not only this slows down the mdoc(7) processing significantly, but it also has an undesired (in this case) effect of disabling hyphenation within the entire enclosed block.
|
80286 |
24-Jul-2001 |
obrien |
Expand xmalloc in-place, along with xrealloc; which wasn't even ANSI in its implementation.
|
76804 |
18-May-2001 |
ru |
Don't leak memory when compiling text following the `a', `c' or `i' command.
Testcase:
echo FOO | sed "/FOO/c\\ `jot -b 'aaaa\' 500`"
Submitted by: Max Khon <fjoe@newst.net>
|
72432 |
13-Feb-2001 |
ru |
mdoc(7) police: use the default ``file ...'' feature of the .Ar macro.
|
71102 |
16-Jan-2001 |
ru |
Prepare for mdoc(7)NG.
|
70197 |
19-Dec-2000 |
ru |
Prepare for mdoc(7)NG.
|
68963 |
20-Nov-2000 |
ru |
mdoc(7) police: use the new features of the Nm macro.
|
68575 |
10-Nov-2000 |
ru |
Avoid use of direct troff requests in mdoc(7) manual pages.
|
60401 |
11-May-2000 |
nsayer |
Small style fix '=' -> ' = '
|
60394 |
11-May-2000 |
nsayer |
Fix compatibility issue in sed. Do so by explicitely adding a newline to the end of any -e argument.
PR: bin/18474
|
58628 |
26-Mar-2000 |
charnier |
Add DIAGNOSTICS section name
|
58407 |
20-Mar-2000 |
green |
Uncorrupt the SCCS id (s/ From: / /). Put a tab before the comment for "rflags".
Submitted by: bde
|
58388 |
20-Mar-2000 |
green |
Fix a couple hard sentence breaks I inadvertently introduced. Also, update a bit of English into something easier to read.
Prompted by: sheldonh (thanks!)
|
58309 |
19-Mar-2000 |
green |
Add a new flag: -E enables "extended" regular expressions.
|
58307 |
19-Mar-2000 |
green |
Add $FreeBSD$.
|
57219 |
15-Feb-2000 |
unfurl |
PR: 10850 Submitted by: Scott Hazen Mueller <scott@zorch.sf-bay.org>
|
55244 |
29-Dec-1999 |
steve |
Escape the ':' so that is actually makes it into the double quotes.
PR: 15775 Submitted by: Martin Kammerhofer <mkamm@gmx.net>
|
50477 |
28-Aug-1999 |
peter |
$Id$ -> $FreeBSD$
|
48792 |
12-Jul-1999 |
nik |
Add $Id$, to make it simpler for members of the translation teams to track.
The $Id$ line is normally at the bottom of the main comment block in the man page, separated from the rest of the manpage by an empty comment, like so;
.\" $Id$ .\"
If the immediately preceding comment is a @(#) format ID marker than the the $Id$ will line up underneath it with no intervening blank lines. Otherwise, an additional blank line is inserted.
Approved by: bde
|
46081 |
25-Apr-1999 |
imp |
More egcs warning fixes: o main returns int not void o use return 0 at end of main when needed o use braces to avoid potentially ambiguous else o don't default to type int o #ifdef 0 -> #if 0
Reviewed by: obrien and chuckr
|
41602 |
08-Dec-1998 |
archie |
Fix a new bug introduced by the previous bug fix
|
41573 |
07-Dec-1998 |
archie |
Fix brokenness compiling "s/pat/subst/" when length of subst is >= 4090 chars. PR: bin/7939
|
41572 |
07-Dec-1998 |
archie |
Compile -Wall
|
39571 |
22-Sep-1998 |
brian |
Terminate our output string correctly if we've got an ``a'' command that has an escaped newline on the last line of the last script that we're processing.
This fixes exmh2/scripts/build when /etc/malloc.conf -> AJ
|
35520 |
29-Apr-1998 |
ache |
Make 'y' command 8bit clean PR: 6458
|
28066 |
11-Aug-1997 |
charnier |
Remove local redefinition for err(). Add usage().
|
26826 |
23-Jun-1997 |
steve |
Show the real revision date and not the date that this manpage is being viewed.
|
24360 |
29-Mar-1997 |
imp |
compare return value from getopt against -1 rather than EOF, per the final posix standard on the topic.
|
23791 |
12-Mar-1997 |
mpp |
Add a missing '\' to some text.
Obtained from: NetBSD-bugs mailing list PR# 3321
|
19829 |
17-Nov-1996 |
wosch |
fix C programmer's bug number 12.2
submitted by: bruce
|
19825 |
17-Nov-1996 |
joerg |
Fix the C programmer's bug #1: EOF is of type `int', not `char'.
Strong 2.2 candidate.
Submitted by: wosch
|
17523 |
11-Aug-1996 |
ache |
Make it at least semi-worked
|
17522 |
11-Aug-1996 |
ache |
Localize it 8bit cleanup
|
17195 |
17-Jul-1996 |
bde |
Yet^2 another fix for the line continuation bug.
The fundamental problem with the original code is that it accesses p[-2] which is one before the beginning of the input buffer for empty lines. rev.1.6 just moved the problem from failures when p[-2] happens to be '\\' to failures when it happens to be '\0'. rev.1.5 was confused about the trailing newline and other things.
I went back to rev.1.5 and fixed it. The result is the same as Keith Bostic's final version in PR 1356 except it loses more gracefully for excessively long input lines.
|
16753 |
26-Jun-1996 |
dg |
Yet another fix for the line continuation bug in sed. Keith's patch introduced a new bug. This fix appears to work correctly. Fixes PR#1350.
Submitted by: mark@linus.demon.co.uk (Mark Valentine)
|
16510 |
19-Jun-1996 |
dg |
Fix from Keith Bostic <bostic@bsdi.com> for bug in sed dealing with continuation lines.
Submitted by: Keith Bostic via Kirk McKusick
|
10075 |
16-Aug-1995 |
jkh |
Merge various fixes from NetBSD. This will allow the WordPerfect for SCO installation to run all the way through (some POSIX fixes).
|
8874 |
30-May-1995 |
rgrimes |
Remove trailing whitespace.
|
7637 |
05-Apr-1995 |
joerg |
Sed dumps core when linked with some coalescing malloc because it doesn't initialize malloced structure.
Submitted by: vode@hut.fi
|
6666 |
23-Feb-1995 |
rgrimes |
Fix spelling error (scipt -> script). Submitted by: <jhawk@MIT.EDU> Obtained from: NetBSD-bugs mailling list
|
6209 |
06-Feb-1995 |
jmz |
Correct a bug in the P function: according to 7th edition UNIX (tm), the newline must be part of the pattern space i.e. `echo a|sed -e P' must print a a and not aa This is consistent with gnu sed, SunOS, Ultrix (and probably others!)
|
4891 |
01-Dec-1994 |
ats |
Submitted by: Philippe Charnier Fixed a little typo: lable -> label.
|
1591 |
27-May-1994 |
rgrimes |
This commit was generated by cvs2svn to compensate for changes in r1590, which included commits to RCS files with non-trunk default branches.
|