Page MenuHomeFreeBSD

Move all shell related files to bin/sh/ and bin/csh/
ClosedPublic

Authored by brd on Aug 14 2018, 2:14 PM.

Details

Test Plan

Compare the results from make distribution

Diff Detail

Repository
rS FreeBSD src repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

brd created this revision.Aug 14 2018, 2:14 PM
will requested changes to this revision.Aug 14 2018, 8:57 PM

Issues using LINKS, on cursory inspection.

bin/csh/Makefile
53 ↗(On Diff #46659)

bsd.links.mk prepends ${DESTDIR} to both source and target.

bin/sh/Makefile
40 ↗(On Diff #46659)

See above.

68 ↗(On Diff #46659)

Why is this appropriate for this particular file?

This revision now requires changes to proceed.Aug 14 2018, 8:57 PM
brd updated this revision to Diff 46696.Aug 15 2018, 10:46 AM
brd marked 2 inline comments as done.

Update to remove DESTDIR from LINKS

brd added inline comments.Aug 15 2018, 10:48 AM
bin/sh/Makefile
68 ↗(On Diff #46659)

That is what we do today in etc/Makefile.. but it has been there for almost 10 years, so perhaps it is time for it to go.

will accepted this revision.Aug 15 2018, 2:10 PM
This revision is now accepted and ready to land.Aug 15 2018, 2:10 PM
This revision was automatically updated to reflect the committed changes.