Page MenuHomeFreeBSD
Differential D15197

Remove compression support from bsdgrep
ClosedPublic
Actions

Authored by bapt on Apr 25 2018, 1:50 PM.
Tags
None
Referenced Files
F169056558: D15197.id41858.diff
Mon, Aug 31, 7:00 AM
F169043601: D15197.id41851.diff
Mon, Aug 31, 5:39 AM
F169029236: D15197.id.diff
Mon, Aug 31, 4:18 AM
F169024658: D15197.id41852.diff
Mon, Aug 31, 3:54 AM
F169022874: D15197.diff
Mon, Aug 31, 3:45 AM
F168923935: D15197.id41852.diff
Sun, Aug 30, 7:05 PM
Unknown Object (File)
Sat, Aug 29, 9:18 PM
Unknown Object (File)
Tue, Aug 25, 5:10 PM
View All Files
Subscribers
imp

Details

Reviewers
kevans
Group Reviewers
manpages
Commits
rS332995: Remove compression support from bsdgrep
Summary

Compression support is now handled by an external script, remove it from the
bsdgrep(1) utility.
This removes the support for -Z -J -X and -M

Note1: that it matches the changes it newer GNU grep

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16304
Build 16245: arc lint + arc unit

Event Timeline

bapt created this revision.Apr 25 2018, 1:50 PM
Herald added a reviewer: manpages. · View Herald TranscriptApr 25 2018, 1:50 PM
Herald added a subscriber: imp. · View Herald Transcript
Harbormaster completed remote builds in B16303: Diff 41851.Apr 25 2018, 1:50 PM
kevans accepted this revision.Apr 25 2018, 1:53 PM

Looks good to me- we should also re-number the comments on errstr and update the nls/ catalogs, but I'm OK with making a pass to do that after the fact.

This revision is now accepted and ready to land.Apr 25 2018, 1:53 PM
bapt updated this revision to Diff 41852.Apr 25 2018, 1:53 PM

Also remove zlib inclusion

This revision now requires review to proceed.Apr 25 2018, 1:53 PM
Harbormaster completed remote builds in B16304: Diff 41852.Apr 25 2018, 1:53 PM
kevans accepted this revision.Apr 25 2018, 2:04 PM
This revision is now accepted and ready to land.Apr 25 2018, 2:04 PM
Closed by commit rS332995: Remove compression support from bsdgrep (authored by bapt). · Explain WhyApr 25 2018, 2:40 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
usr.bin/
grep/
14 lines
135 lines
4 lines
30 lines
57 lines

Diff 41852

View Options

usr.bin/grep/Makefile

Loading...
View Options

usr.bin/grep/file.c

Loading...
View Options

usr.bin/grep/grep.h

Loading...
View Options

usr.bin/grep/grep.1

Loading...
View Options

usr.bin/grep/grep.c

Loading...