Page MenuHomeFreeBSD
Differential D57503

install: drop obsolete file size limit for -C
ClosedPublic
Actions

Authored by ray on Jun 8 2026, 2:44 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Jul 18, 11:42 AM
Unknown Object (File)
Thu, Jul 16, 11:49 AM
Unknown Object (File)
Wed, Jul 15, 7:25 AM
Unknown Object (File)
Wed, Jul 15, 7:25 AM
Unknown Object (File)
Wed, Jun 24, 10:06 AM
Unknown Object (File)
Mon, Jun 22, 9:07 PM
Unknown Object (File)
Mon, Jun 22, 9:03 PM
Unknown Object (File)
Mon, Jun 22, 3:09 PM
View All 35 Files
Subscribers
ziaee

Details

Reviewers
bdrewery
imp
glebius
ziaee
Commits
rG9d10b4d2c9e8: install: drop obsolete file size limit for -C
Summary

Removes the file size limit for -C comparisons. The limit was
meant to prevent oversized mmap allocations, which is no longer relevant
as mmap is no longer used here. Credit to bdrewery.
See: https://reviews.freebsd.org/D57230

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 73742
Build 70625: arc lint + arc unit

Event Timeline

ray created this revision.Jun 8 2026, 2:44 PM
Herald added a subscriber: ziaee. · View Herald TranscriptJun 8 2026, 2:44 PM
ray requested review of this revision.Jun 8 2026, 2:44 PM
Harbormaster completed remote builds in B73742: Diff 179387.Jun 8 2026, 2:44 PM
ray updated this revision to Diff 179389.Jun 8 2026, 2:49 PM

Update message.

Harbormaster completed remote builds in B73743: Diff 179389.Jun 8 2026, 2:49 PM
ray edited the summary of this revision. (Show Details)Jun 8 2026, 2:53 PM
ray mentioned this in D57230: install: add -z <max_cmp_size> option.Jun 8 2026, 2:55 PM
glebius accepted this revision.Jun 8 2026, 5:57 PM

Makes sense to mention the commit that had removed mmap() in the commit message.

No longer since XXXXXXX as mmap is removed
This revision is now accepted and ready to land.Jun 8 2026, 5:57 PM
bdrewery accepted this revision.Jun 10 2026, 4:16 PM

Thank you

ziaee accepted this revision.Jun 10 2026, 6:07 PM

lgtm from manpages

Closed by commit rG9d10b4d2c9e8: install: drop obsolete file size limit for -C (authored by ray). · Explain WhyJun 11 2026, 1:53 AM
This revision was automatically updated to reflect the committed changes.
ray added a commit: rG9d10b4d2c9e8: install: drop obsolete file size limit for -C.

Revision Contents
Changeset List

PathSize
usr.bin/
xinstall/
2 lines
13 lines
103 lines

Diff 179387

View Options

usr.bin/xinstall/Makefile

Loading...
View Options

usr.bin/xinstall/install.1

Loading...
View Options

usr.bin/xinstall/xinstall.c

Loading...