Page MenuHomeFreeBSD
Differential D20476

makesyscalls.sh: always use absolute path for syscalls.conf
ClosedPublic
Actions

Authored by brooks on May 30 2019, 8:34 PM.
Tags
None
Referenced Files
F142476372: D20476.id58094.diff
Tue, Jan 20, 6:16 AM
Unknown Object (File)
Mon, Jan 19, 4:49 AM
Unknown Object (File)
Mon, Jan 19, 3:43 AM
Unknown Object (File)
Sun, Jan 18, 8:04 PM
Unknown Object (File)
Sun, Jan 18, 1:34 PM
Unknown Object (File)
Thu, Jan 8, 9:15 PM
Unknown Object (File)
Sun, Jan 4, 10:36 AM
Unknown Object (File)
Fri, Dec 26, 11:28 AM
View All Files
Subscribers
andrew
emaste
imp

Details

Reviewers
bdrewery
manu
Commits
rS351567: MFC r348446:
rS348446: makesyscalls.sh: always use absolute path for syscalls.conf
Summary

syscalls.conf is included using "." which per the Open Group:

If file does not contain a <slash>, the shell shall use the search path
specified by PATH to find the directory containing file.

POSIX shells don't fall back to the current working directory.

Submitted by: Nathaniel Wesley Filardo <nwf20@cl.cam.ac.uk>

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

brooks created this revision.May 30 2019, 8:34 PM
Herald added a reviewer: manu. · View Herald TranscriptMay 30 2019, 8:34 PM
Herald added subscribers: emaste, andrew. · View Herald Transcript
Harbormaster completed remote builds in B24609: Diff 58091.May 30 2019, 8:34 PM
bdrewery requested changes to this revision.May 30 2019, 8:43 PM
bdrewery added inline comments.
sys/compat/cloudabi64/Makefile
15 ↗(On Diff #58091)

Please use ${.CURDIR} instead of ./ in all of this. That will churn less with my change which does that.

This revision now requires changes to proceed.May 30 2019, 8:43 PM
brooks updated this revision to Diff 58093.May 30 2019, 8:47 PM
  • Use ${.CURDIR} instead of ".".
Harbormaster completed remote builds in B24610: Diff 58093.May 30 2019, 8:47 PM
bdrewery accepted this revision.May 30 2019, 8:50 PM
This revision is now accepted and ready to land.May 30 2019, 8:50 PM
brooks retitled this revision from makesyscalls.sh: always use relative path for syscalls.conf to makesyscalls.sh: always use absolute path for syscalls.conf.May 30 2019, 8:54 PM
Closed by commit rS348446: makesyscalls.sh: always use absolute path for syscalls.conf (authored by brooks). · Explain WhyMay 30 2019, 8:56 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptMay 30 2019, 8:56 PM
brooks added a commit: rS351567: MFC r348446:.Aug 28 2019, 8:05 AM

Revision Contents
Changeset List

PathSize
head/
sys/
amd64/
linux/
4 lines
linux32/
4 lines
arm64/
linux/
4 lines
compat/
cloudabi32/
4 lines
cloudabi64/
4 lines
freebsd32/
4 lines
i386/
linux/
4 lines
kern/
2 lines

Diff 58094

View Options

head/sys/amd64/linux/Makefile

Loading...
View Options

head/sys/amd64/linux32/Makefile

Loading...
View Options

head/sys/arm64/linux/Makefile

Loading...
View Options

head/sys/compat/cloudabi32/Makefile

Loading...
View Options

head/sys/compat/cloudabi64/Makefile

Loading...
View Options

head/sys/compat/freebsd32/Makefile

Loading...
View Options

head/sys/i386/linux/Makefile

Loading...
View Options

head/sys/kern/makesyscalls.sh

Loading...