Use the standard OpenBSD preprocessor definition

A few places uses OPENBSD instead of the typical __OpenBSD__.

changed milestone to %Tor: 0.3.0.x-final

added component::core tor/tor milestone::Tor: 0.3.0.x-final priority::medium resolution::fixed severity::normal status::closed type::task labels

Trac:
0001-Use-the-standard-OpenBSD-preprocessor-definition.patch

Trac:
Status: new to needs_review

Trac:
Milestone: N/A to Tor: 0.3.0.x-final

Is __OpenBSD__ defined for bitrig and other OpenBSD variants?

If so, this patch is fine, flip it into merge-ready. If not, we need to rethink how we make this change.

Trac:
Status: needs_review to needs_information

Replying to teor:

Is __OpenBSD__ defined for bitrig and other OpenBSD variants?

For context, Bitrig support was added in #6982 (moved) (commit d92d3f33356af002892ba5754d9d36cc4504c95f).

The Pre-defined Compiler Macros wiki contains a bunch of operating systems but not Bitrig. Issue 72 on the Bitrig bug tracker was about getting Bitrig added to this list.

The param.h header typically contains these preprocessor definitions. That header file in the Bitrig repository contains no __OpenBSD__. The current patch would therefore remove support for Bitrig. It does contain OpenBSD which seems to exists for other forks (like LibertyBSD) also.

If so, this patch is fine, flip it into merge-ready. IMO the patch is fine, because Bitrig isn't explicitly supported and other parts of the code already uses __OpenBSD__ without caring about Bitrig (a quick grep shows 10 lines using __OpenBSD__ and 4 lines using OPENBSD).

If not, we need to rethink how we make this change. If we really care about Bitrig and other OpenBSD forks, the patch could be fixed by replacing OPENBSD with OpenBSD but -1 from me on that.

I'll leave it to teor for the final word on this.

Trac:
Type: defect to task

Back in needs_review and there is an action item for teor.

Trac:
Cc: N/A to teor
Status: needs_information to needs_review

Given cypherpunks' comment, I think we should switch all uses of OpenBSD and OPENBSD to OpenBSD instead. (Unless there's some reason we only want those OSs defining one of the other macros.)

Ideally, we should avoid using these macros at all, and check for features with configure instead.

Trac:
Status: needs_review to needs_revision

See branch bug20980_030_01.

I followed teor's latest recommendation here using the original patch.

Trac:
Status: needs_revision to needs_review

ok, merging that!

Trac:
Resolution: N/A to fixed
Status: needs_review to closed

closed

mentioned in issue #28938 (moved)

moved to tpo/core/tor#20980 (closed)

mentioned in issue tpo/core/tor#28938 (closed)