Fix stand/ build after r339671.
ClosedPublic
Actions

Authored by kib on Oct 23 2018, 10:44 PM.

Tags

None

Referenced Files

	F156693011: D17677.id49535.diff
	Fri, May 15, 5:43 PM

	F156692965: D17677.id49535.diff
	Fri, May 15, 5:43 PM

	F156692959: D17677.id49536.diff
	Fri, May 15, 5:42 PM

	F156689615: D17677.diff
	Fri, May 15, 5:18 PM

	Unknown Object (File)
	Mon, Apr 27, 11:29 AM

	Unknown Object (File)
	Wed, Apr 22, 4:19 PM

	Unknown Object (File)
	Apr 14 2026, 9:40 PM

	Unknown Object (File)
	Apr 13 2026, 5:49 PM

View All 100 Files

Subscribers

None

Details

Reviewers

imp
kevans

Commits

rS339673: Fix stand/ build after r339671.

Summary

ffs_subr.c requires calculate_crc32c() from libkern. Unfortunately we cannot just add libkern/crc32.c to libstand because crc32.o is already compiled from contrib/zlib/crc32.c. Use the include trick to rename the source.

Note that libstand also provides crc32.c which seems to be unused.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kib created this revision.Oct 23 2018, 10:44 PM

Harbormaster completed remote builds in B20376: Diff 49535.Oct 23 2018, 10:44 PM

This will fix the build. I'll look into the unused crc32 and sort out what can/should be common here after this commit.

This revision is now accepted and ready to land.Oct 23 2018, 10:52 PM

Closed by commit rS339673: Fix stand/ build after r339671. (authored by kib). · Explain WhyOct 23 2018, 11:11 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

stand/

libsa/

Makefile

4 lines

crc32_libkern.c

3 lines

Diff 49536

View Options

head/stand/libsa/Makefile