Opened 4 years ago

Closed 4 years ago

#19758 closed defect (fixed)

Fix OpenBSD build bot errors

Reported by: teor Owned by:
Priority: Medium Milestone: Tor: 0.2.9.x-final
Component: Core Tor/Tor Version:
Severity: Normal Keywords: build, openbsd, TorCoreTeam201611
Cc: cb@… Actual Points:
Parent ID: Points: 0.5
Reviewer: Sponsor:


The OpenBSD build bot shows a lot of errors like
(Needs a non-HSTS browser, the cert has expired.)

There are a few warnings, some GCC pragmas that don't work, and evdns isn't linking correctly. So that might require help from the admin.

Child Tickets

Attachments (1)

tor2.patch (680 bytes) - added by rubiate 4 years ago.
Readd check for libevent2 headers to fix build on OpenBSD, and change the fallback path to an error

Download all attachments as: .zip

Change History (7)

comment:1 Changed 4 years ago by rubiate

The libevent2 problem is that commit 265e40b481d23ea0ee8b4e1705fb013532d1f6b2 removed a check for the headers, and there's a conditional that relies on the result of that check, so that conditional always fails and configure falls back to linking libevent2 with -levent, it just happens that this works on every platform that isn't OpenBSD (the whole conditional path that gets skipped was added specifically for OpenBSD in the first place).

Since libevent2 is mandatory now I think it makes sense to add the header check back in and change the fall back path to an error.

This patch gets tor successfully building HEAD (5e571900b341b) on OpenBSD -current amd64

Changed 4 years ago by rubiate

Attachment: tor2.patch added

Readd check for libevent2 headers to fix build on OpenBSD, and change the fallback path to an error

comment:2 Changed 4 years ago by rubiate

Cc: cb@… added

comment:3 Changed 4 years ago by arma

Does some branch of Tor not build on openbsd right now? If so, should we file specific tickets about the issue(s), in the right branches?

comment:4 Changed 4 years ago by rubiate

On the latest released OpenBSD (6.0), Tor builds on at least the latest of 0.2.{7,8,9} and 0.3.0. The libevent problem was fixed (again!) in #19902.

There are a few warnings but I think they're all just the ancient GCC OpenBSD uses getting confused, and there's probably no way to deal with them, ie. the pragmas not working to silence the overlong string warnings.

There's one test (util/time) that fails on 6.0 but passes on 6.0-current -- the problem was a bug in OpenBSD libc, and the OpenBSD devs didn't think it was worth backporting the fix to 6.0-stable, so I don't know if that matters.

So everything is mostly fine on the OpenBSD front at the moment.

comment:5 Changed 4 years ago by arma

Ok. I will leave it to teor (the original reporter) to close this ticket, or do something else if he desires.

comment:6 Changed 4 years ago by teor

Keywords: build openbsd TorCoreTeam201611 added
Milestone: Tor: 0.2.???Tor: 0.2.9.x-final
Resolution: fixed
Status: newclosed

I'm going to call this fixed in 0.2.9, please open a separate ticket for any further issues.

Note: See TracTickets for help on using tickets.