Page MenuHomeFreeBSD
Differential D56656

Makefile.inc1: Use ln -n instead of ln -h
ClosedPublic
Actions

Authored by ivy on Apr 27 2026, 4:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, May 21, 7:30 PM
Unknown Object (File)
Thu, May 14, 11:36 PM
Unknown Object (File)
Thu, May 14, 12:16 PM
Unknown Object (File)
Thu, May 14, 12:16 PM
Unknown Object (File)
Thu, May 14, 2:23 AM
Unknown Object (File)
Wed, May 13, 5:16 PM
Unknown Object (File)
Tue, May 12, 10:43 PM
Unknown Object (File)
Mon, May 11, 2:02 PM
View All 24 Files
Subscribers
emaste
imp
jrtc27

Details

Reviewers
imp
wosch
emaste
Group Reviewers
pkgbase
Commits
rGd455c4bb69ec: Makefile.inc1: Use ln -n instead of ln -h
Summary

We support both -h and -n, but GNU coreutils only supports -n. This fixes the package build on Linux.

MFC after: 2 weeks
Sponsored by: https://www.patreon.com/bsdivy

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

ivy created this revision.Apr 27 2026, 4:44 AM
Herald added subscribers: emaste, imp. · View Herald TranscriptApr 27 2026, 4:44 AM
ivy requested review of this revision.Apr 27 2026, 4:44 AM
Harbormaster completed remote builds in B72563: Diff 176605.Apr 27 2026, 4:44 AM
imp accepted this revision.Apr 27 2026, 5:21 AM
This revision is now accepted and ready to land.Apr 27 2026, 5:21 AM
wosch accepted this revision.Apr 27 2026, 8:24 AM
emaste added a comment.Apr 27 2026, 4:26 PM

Might be nice to have something like ${LN_SYMLINK_FLAGS} set to -Tfs or -hfs. I didn't find other obvious instances in the tree though so perhaps not worth effort right now

emaste mentioned this in D52883: pkg-stage: Improve symlink creation.Apr 27 2026, 6:26 PM
emaste added a subscriber: jrtc27.Apr 27 2026, 7:04 PM

ln -n per @jrtc27 in D52883?

ivy retitled this revision from Makefile.inc1: Use ln -T instead of ln -h on Linux to Makefile.inc1: Use ln -n instead of ln -h.Apr 27 2026, 7:36 PM
ivy edited the summary of this revision. (Show Details)
ivy updated this revision to Diff 176668.Apr 27 2026, 7:36 PM

use ln -n instead

This revision now requires review to proceed.Apr 27 2026, 7:36 PM
Harbormaster completed remote builds in B72585: Diff 176668.Apr 27 2026, 7:36 PM
emaste accepted this revision.Apr 27 2026, 7:45 PM
This revision is now accepted and ready to land.Apr 27 2026, 7:45 PM
Closed by commit rGd455c4bb69ec: Makefile.inc1: Use ln -n instead of ln -h (authored by ivy). · Explain WhyApr 27 2026, 10:06 PM
This revision was automatically updated to reflect the committed changes.
ivy added a commit: rGd455c4bb69ec: Makefile.inc1: Use ln -n instead of ln -h.

Revision Contents
Changeset List

PathSize
2 lines

Diff 176678

View Options

Makefile.inc1

Loading...