Page MenuHomeFreeBSD
Differential D13937

Don't hardcode /usr/bin as the path for mktemp in build tools
ClosedPublic
Actions

Authored by arichardson on Jan 16 2018, 7:35 PM.
Tags
None
Referenced Files
F83218911: D13937.diff
Tue, May 7, 7:56 PM
Unknown Object (File)
Sun, May 5, 10:00 AM
Unknown Object (File)
Wed, Apr 24, 11:18 AM
Unknown Object (File)
Feb 23 2024, 11:14 PM
Unknown Object (File)
Feb 23 2024, 4:54 PM
Unknown Object (File)
Feb 23 2024, 4:54 PM
Unknown Object (File)
Feb 23 2024, 4:54 PM
Unknown Object (File)
Feb 23 2024, 4:39 PM
View All 29 Files
Subscribers
bdrewery
imp

Details

Reviewers
jhb
emaste
jilles
bdrewery
Commits
rS328934: Don't hardcode /usr/bin as the path for mktemp in build tools
Summary

It won't work e.g. when crossbuilding from Ubuntu Linux as mktemp is in
/bin there.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 14419
Build 14571: arc lint + arc unit

Event Timeline

arichardson created this revision.Jan 16 2018, 7:35 PM
Herald added a reviewer: jilles. · View Herald TranscriptJan 16 2018, 7:35 PM
Harbormaster completed remote builds in B14419: Diff 38050.Jan 16 2018, 7:35 PM
emaste added a subscriber: bdrewery.Jan 16 2018, 8:23 PM

This reverts rS88450, but the commit message there unfortunately states exactly what was done but not why.

bdrewery accepted this revision.Jan 24 2018, 8:33 PM
This revision is now accepted and ready to land.Jan 24 2018, 8:33 PM
emaste added a comment.Jan 24 2018, 8:42 PM

No objection from me.

jhb added a comment.Jan 30 2018, 12:03 AM

Ok by me (with my mentor hat)

Closed by commit rS328934: Don't hardcode /usr/bin as the path for mktemp in build tools (authored by arichardson). · Explain WhyFeb 6 2018, 3:41 PM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptFeb 6 2018, 3:41 PM

Revision Contents
Changeset List

PathSize
2 lines
bin/
sh/
2 lines
2 lines

Diff 38050

View Options

Makefile.inc1

Loading...
View Options

bin/sh/mkbuiltins

Loading...
View Options

bin/sh/mktokens

Loading...