Page MenuHomeFreeBSD
Differential D14244

crossbuild: Allow bootstrapping libopenbsd on Linux
ClosedPublic
Actions

Authored by arichardson on Feb 7 2018, 4:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 27, 8:05 AM
Unknown Object (File)
Mon, May 5, 12:15 PM
Unknown Object (File)
Apr 3 2025, 5:37 AM
Unknown Object (File)
Mar 29 2025, 9:33 AM
Unknown Object (File)
Mar 25 2025, 9:23 AM
Unknown Object (File)
Mar 11 2025, 7:53 PM
Unknown Object (File)
Mar 10 2025, 7:25 AM
Unknown Object (File)
Feb 22 2025, 7:10 AM
View All 84 Files
Subscribers
brooks

Details

Reviewers
emaste
imp
bdrewery
jhb
brooks
Commits
rS341515: Allow bootstrapping libopenbsd on Linux
Summary

The getdtablecount.c file won't compile on Linux but it seems like none of
the bootstrap tools actually need it.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 20566
Build 19987: arc lint + arc unit

Event Timeline

arichardson created this revision.Feb 7 2018, 4:04 PM
Harbormaster completed remote builds in B14845: Diff 39003.Feb 7 2018, 4:04 PM
arichardson updated this revision to Diff 49872.Nov 1 2018, 11:40 AM

Simplify and remove .MAKE.OS check

Harbormaster completed remote builds in B20565: Diff 49872.Nov 1 2018, 11:40 AM
arichardson updated this revision to Diff 49873.Nov 1 2018, 11:41 AM

Fix typo in comment

Harbormaster completed remote builds in B20566: Diff 49873.Nov 1 2018, 11:41 AM
arichardson added a comment.Nov 18 2018, 9:37 AM

ping?

emaste accepted this revision.Nov 22 2018, 11:29 PM

I don't really like this, but given that any failure should be fairly straightforward to track down OK.

(I'd probably prefer some sort of #ifdef linux in getdtablecount.c itself with a #warning and a stub implementation.)

This revision is now accepted and ready to land.Nov 22 2018, 11:29 PM
emaste added a subscriber: brooks.Nov 22 2018, 11:29 PM
brooks accepted this revision.Nov 26 2018, 7:42 PM

This seems fine, especially since libopenbsd is an INTERNALLIB.

Closed by commit rS341515: Allow bootstrapping libopenbsd on Linux (authored by arichardson). · Explain WhyDec 5 2018, 10:59 AM
This revision was automatically updated to reflect the committed changes.
bdrewery added a comment.Dec 5 2018, 7:49 PM
In D14244#388189, @emaste wrote:

I don't really like this, but given that any failure should be fairly straightforward to track down OK.

(I'd probably prefer some sort of #ifdef linux in getdtablecount.c itself with a #warning and a stub implementation.)

+1
Reviewed by me too (I think I'm late)

Revision Contents
Changeset List

PathSize
lib/
libopenbsd/
9 lines

Diff 49873

View Options

lib/libopenbsd/Makefile

Loading...