devel/crc32c: fix build on big-endian architectures
ClosedPublic
Actions

Authored by pkubaj on Jul 24 2019, 1:38 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Jun 23, 7:04 PM

	Unknown Object (File)
	Sun, Jun 21, 9:58 PM

	Unknown Object (File)
	Thu, Jun 18, 8:03 PM

	Unknown Object (File)
	Wed, Jun 17, 12:18 PM

	Unknown Object (File)
	Tue, Jun 16, 11:58 PM

	Unknown Object (File)
	Thu, Jun 4, 6:07 PM

	Unknown Object (File)
	Sat, May 30, 2:51 PM

	Unknown Object (File)
	May 26 2026, 12:47 PM

View All Files

Subscribers

None

Details

Reviewers

linimon
mat
tcberner

Commits

rP507823: devel/crc32c: fix build on big-endian architectures

Summary

C++11 compiler is needed:

Target "crc32c" requires the language dialect "CXX11" , but CMake does not
know the compile flags to use to enable it.

Additionally, big-endian code in src/crc32c_read_le.h needs fixing - the value is 64 bits wide, not 32 bits.

PR: 239420

Diff Detail

Repository

rP FreeBSD ports repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

pkubaj created this revision.Jul 24 2019, 1:38 PM

Harbormaster completed remote builds in B25492: Diff 60089.Jul 24 2019, 1:38 PM

linimon accepted this revision.Aug 1 2019, 12:41 AM

This revision is now accepted and ready to land.Aug 1 2019, 12:41 AM

Closed by commit rP507823: devel/crc32c: fix build on big-endian architectures (authored by pkubaj). · Explain WhyAug 1 2019, 7:12 PM

This revision was automatically updated to reflect the committed changes.

pkubaj added a commit: rP507823: devel/crc32c: fix build on big-endian architectures.

Revision Contents
Changeset List

Path

Size

head/

devel/

crc32c/

Makefile

2 lines

files/

patch-src_crc32c__read__le.h

25 lines

Diff 60368

View Options

head/devel/crc32c/Makefile

View Options

head/devel/crc32c/files/patch-src_crc32creadle.h

devel/crc32c: fix build on big-endian architecturesClosedPublicActions