Page MenuHomeFreeBSD
Differential D45355

cross-build: This is not glibc specific, but Linux specific
ClosedPublic
Actions

Authored by imp on Fri, May 24, 9:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 3, 3:27 PM
Unknown Object (File)
Sun, May 26, 6:56 PM
Unknown Object (File)
Sun, May 26, 1:32 PM
Unknown Object (File)
Sat, May 25, 4:42 PM
Subscribers
None

Details

Reviewers
jrtc27
val_packett.cool
Commits
rG0e03402139da: cross-build: This is not glibc specific, but Linux specific
Summary

Neither glibc nor musl define these interfaces, so provide them for all
Linux builds.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp requested review of this revision.Fri, May 24, 9:08 PM
imp created this revision.
Harbormaster completed remote builds in B57909: Diff 139066.Fri, May 24, 9:08 PM
imp added a parent revision: D45354: cross-build: Define __*int*_t and include sys/cdefs.h.Fri, May 24, 9:08 PM
val_packett.cool accepted this revision.Sat, May 25, 5:07 PM
This revision is now accepted and ready to land.Sat, May 25, 5:07 PM
Closed by commit rG0e03402139da: cross-build: This is not glibc specific, but Linux specific (authored by imp). · Explain WhySun, May 26, 5:41 PM
This revision was automatically updated to reflect the committed changes.
imp added a commit: rG0e03402139da: cross-build: This is not glibc specific, but Linux specific.
jrtc27 added inline comments.Sun, May 26, 6:11 PM
tools/build/cross-build/progname.c
39

The ifdef should just be dropped; Makefile.boot already makes the addition of the source file conditional on it being Linux

imp added inline comments.Sun, May 26, 6:15 PM
tools/build/cross-build/progname.c
39

Ah. That's a better solution. I'd missed we'd done that. I'll do a follow-up commit. I have one or two others too.

jrtc27 added inline comments.Sun, May 26, 6:31 PM
tools/build/cross-build/progname.c
39

Yeah; see the comments on the GitHub PR from myself and later Alex

imp added a comment.Sun, May 26, 9:24 PM

Fixed in https://reviews.freebsd.org/D45374

imp added inline comments.Sun, May 26, 9:26 PM
tools/build/cross-build/progname.c
39

I see that I missed them now...

Revision Contents
Changeset List

PathSize
tools/
build/
cross-build/
4 lines

Diff 139123

View Options

tools/build/cross-build/progname.c

Loading...