Allow CROSS_TOOLCHAIN to be a path to a file.
ClosedPublic
Actions

Authored by brooks on Feb 2 2018, 10:40 PM.

Tags

None

Referenced Files

	F132427493: D14178.diff
	Thu, Oct 16, 8:37 PM

	Unknown Object (File)
	Tue, Oct 14, 2:57 PM

	Unknown Object (File)
	Tue, Oct 14, 2:24 PM

	Unknown Object (File)
	Tue, Oct 14, 2:24 PM

	Unknown Object (File)
	Tue, Oct 14, 2:23 PM

	Unknown Object (File)
	Tue, Oct 14, 12:41 AM

	Unknown Object (File)
	Sun, Oct 12, 10:10 AM

	Unknown Object (File)
	Thu, Oct 2, 3:26 AM

View All 82 Files

Subscribers

imp

Details

Reviewers

bapt
bdrewery
jhb
emaste

Commits

rS329993: Allow CROSS_TOOLCHAIN to be a path to a file.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

brooks created this revision.Feb 2 2018, 10:40 PM

Harbormaster completed remote builds in B14750: Diff 38799.Feb 2 2018, 10:40 PM

poke

This revision was not accepted when it landed; it landed in state Needs Review.Feb 25 2018, 8:21 PM

Closed by commit rS329993: Allow CROSS_TOOLCHAIN to be a path to a file. (authored by brooks). · Explain Why

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptFeb 25 2018, 8:21 PM

emaste added inline comments.Feb 25 2018, 10:10 PM

head/Makefile.inc1
64	Maybe `CROSS_TOOLCHAIN ${CROSS_TOOLCHAIN} not found`? My thought is that someone may find instructions to run e.g. `make CROSS_TOOLCHAIN=powerpc64-gcc TARGET=powerpc TARGET_ARCH=powerpc64 buildworld`, and if it's not installed reporting that it's invalid could be confusing.

jhb added inline comments.Feb 26 2018, 5:51 PM

head/Makefile.inc1
64	+1 on Ed's suggestion (a fan of the overall change though)

Revision Contents
Changeset List

Path

Size

head/

Makefile.inc1

6 lines

Diff 39717

View Options

Allow CROSS_TOOLCHAIN to be a path to a file.ClosedPublicActions