Page MenuHomeFreeBSD
Differential D15609

Describe how to prevent *.core files from being created in sh(1) + pet mandoc
ClosedPublic
Actions

Authored by 0mp on May 29 2018, 8:54 PM.
Tags
Referenced Files
Unknown Object (File)
Sun, Jan 5, 10:35 AM
Unknown Object (File)
Dec 6 2024, 4:44 AM
Unknown Object (File)
Dec 5 2024, 5:45 PM
Unknown Object (File)
Dec 3 2024, 5:30 PM
Unknown Object (File)
Nov 24 2024, 1:38 AM
Unknown Object (File)
Oct 12 2024, 6:36 PM
Unknown Object (File)
Oct 3 2024, 10:06 AM
Unknown Object (File)
Sep 8 2024, 4:39 PM
View All 47 Files
Subscribers
imp
jilles
manpages

Details

Reviewers
eadler
jilles
krion
mat
Group Reviewers
manpages
Commits
rS336483: Describe how to prevent *.core files from being created using ulimit.
Summary

This patch add a note to the description of the -c of ulimit that calling ulimit -c 0 prevents core dump files from being created.

Also, while here: pet mandoc:

  • Get rid of Tn macros
  • Improve formatting of the *Grouping Commands Together* section.
  • Add \& in some places to calm mandoc down.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

0mp created this revision.May 29 2018, 8:54 PM
Herald added a subscriber: imp. · View Herald TranscriptMay 29 2018, 8:54 PM
jilles added a subscriber: jilles.May 30 2018, 9:09 PM
jilles added inline comments.
sh/sh.1
1746–1753 ↗(On Diff #43111)

Hmm, why do the binary operators and conditional operators need \& while the other two do not?

0mp added inline comments.May 30 2018, 10:36 PM
sh/sh.1
1746–1753 ↗(On Diff #43111)

This is a good question. I guess that mdoc doesn't like lines ending with ||".

For example for .Li " ||" it says:

STYLE: no blank before trailing delimiter: Li ||

eadler accepted this revision.Jun 18 2018, 2:26 AM
eadler added inline comments.
sh/sh.1
344 ↗(On Diff #43111)

Change "Another" to "A". No reason to reference the previous one.

This revision is now accepted and ready to land.Jun 18 2018, 2:26 AM
0mp updated this revision to Diff 44090.Jun 19 2018, 6:45 PM

s/Another/A/ as requested by @eadler

This revision now requires review to proceed.Jun 19 2018, 6:45 PM
Herald added a reviewer: jilles. · View Herald TranscriptJun 19 2018, 6:45 PM
jilles accepted this revision.Jun 19 2018, 9:37 PM
This revision is now accepted and ready to land.Jun 19 2018, 9:37 PM
0mp added reviewers: krion, mat.Jul 18 2018, 9:22 PM
mat added a comment.Jul 19 2018, 7:40 AM

Could you use devel/arcanist, or at least generate a diff with full context like it does, with svn diff -x -U9999 or git diff -U9999.

0mp updated this revision to Diff 45517.Jul 19 2018, 8:02 AM

Reupload the patch, this time using arcanist as requested by @mat.

This revision now requires review to proceed.Jul 19 2018, 8:02 AM
Harbormaster completed remote builds in B18155: Diff 45517.Jul 19 2018, 8:02 AM
mat accepted this revision.Jul 19 2018, 8:24 AM
This revision is now accepted and ready to land.Jul 19 2018, 8:24 AM
Closed by commit rS336483: Describe how to prevent *.core files from being created using ulimit. (authored by 0mp). · Explain WhyJul 19 2018, 1:09 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
bin/
sh/
46 lines

Diff 44090

View Options

bin/sh/sh.1

Loading...