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

Use unix versions on Illumos #54

Merged
merged 1 commit into from Aug 12, 2023
Merged

Conversation

sjmulder
Copy link
Contributor

@sjmulder sjmulder commented Aug 10, 2023

Illumos is the OpenSolaris-based OS core of OpenIndiana, SmartOS, etc.

Did not enable the dbus path because:

go: downloading github.com/godbus/dbus/v5 v5.1.0
# github.com/godbus/dbus/v5
../../go/pkg/mod/github.com/godbus/dbus/v5@v5.1.0/conn.go:147:17: undefined: getSystemBusPlatformAddress
../../go/pkg/mod/github.com/godbus/dbus/v5@v5.1.0/conn.go:174:14: undefined: getSystemBusPlatformAddress

Illumos is the OpenSolaris-based OS core of OpenIndiana, SmartOS, etc.

Did not enable the dbus path because:

  go: downloading github.com/godbus/dbus/v5 v5.1.0
  # github.com/godbus/dbus/v5
  ../../go/pkg/mod/github.com/godbus/dbus/v5@v5.1.0/conn.go:147:17: undefined: getSystemBusPlatformAddress
  ../../go/pkg/mod/github.com/godbus/dbus/v5@v5.1.0/conn.go:174:14: undefined: getSystemBusPlatformAddress
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Aug 10, 2023
There were two issues with dependency 'beeep':

 1. The 'unix' backends weren't selected for Illumos
    gen2brain/beeep#54

 2. The fallback 'unsupported' backend had a signature mismatch:
    gen2brain/beeep#53

Both were fixed in a patched fork at https://github.com/sjmulder/beeep,
which is used here. Once the issue is fixed upstream the original
dependencies can be used again.
@gen2brain gen2brain merged commit 3e1549e into gen2brain:master Aug 12, 2023
3 checks passed
@gen2brain
Copy link
Owner

Merged, thanks!

sjmulder added a commit to sjmulder/tut that referenced this pull request Aug 13, 2023
 - Build error due to signature mismatch:
   gen2brain/beeep#53
 - Use 'unix' implementation rather than 'unsupported'
   gen2brain/beeep#54
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 this pull request may close these issues.

None yet

2 participants