Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix compilation of vcpkg on OpenBSD #36812

Closed
SchaichAlonso opened this issue Feb 17, 2024 · 1 comment · Fixed by microsoft/vcpkg-tool#1347
Closed

Fix compilation of vcpkg on OpenBSD #36812

SchaichAlonso opened this issue Feb 17, 2024 · 1 comment · Fixed by microsoft/vcpkg-tool#1347
Assignees

Comments

@SchaichAlonso
Copy link
Contributor

SchaichAlonso commented Feb 17, 2024

Describe the bug
The SIOCGIFHWADDR name, referenced in the fallback branch within vcpkg::get_user_mac_hash, is not defined on OpenBSD.

Further, the environ name referenced in vcpkg::cmd_execute_background, among others, is not defined on OpenBSD, either.

Environment

  • OS: OpenBSD vultr.guest 7.4 GENERIC.MP#2 amd64
  • Compiler: clang version 16.0.6

To Reproduce
Steps to reproduce the behavior:

  1. git clone [email protected]:microsoft/vcpkg-tool.git
  2. mkdir vcpkg-tool/build
  3. cmake -S vcpkg-tool -B vcpkg-tool/build
  4. cmake --build vcpkg-tool/build

Expected behavior
cmake compiles a usable vcpkg binary

Failure logs
stdout.txt

Additional context
For most parts of it, OpenBSD is compatible with FreeBSD, which already has platform branches in vcpkg-tool to resolve these issues.

@jimwang118 jimwang118 self-assigned this Feb 18, 2024
@jimwang118 jimwang118 added the category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly label Feb 18, 2024
@BillyONeal BillyONeal added requires:tool-release An issue that has been fixed in the microsoft/vcpkg-tool repo and is waiting for a release thereof and removed category:tool-update The issue is with build tool or build script, which requires update or should be executed correctly labels Feb 21, 2024
@BillyONeal BillyONeal reopened this Feb 21, 2024
@BillyONeal BillyONeal removed the requires:tool-release An issue that has been fixed in the microsoft/vcpkg-tool repo and is waiting for a release thereof label May 6, 2024
@BillyONeal
Copy link
Member

We did a few tool releases since this :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants