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

Support illumos and Solaris #139

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

jperkin
Copy link

@jperkin jperkin commented Nov 15, 2022

This fixes build failure in recent versions of Grafana, e.g.:

# github.com/blugelabs/bluge/index/lock
../.gopath/pkg/mod/github.com/blugelabs/[email protected]/index/lock/lock.go:33:9: undefined: open
../.gopath/pkg/mod/github.com/blugelabs/[email protected]/index/lock/lock.go:37:9: undefined: open
../.gopath/pkg/mod/github.com/blugelabs/[email protected]/index/lock/lock.go:49:11: e.unlock undefined (type *DefaultLockedFile has no field or method unlock)

I've tested this patch against grafana 9.2.4 and it now builds correctly.

@siepkes
Copy link

siepkes commented Nov 24, 2022

Bumped in to the same issue (building Grafana 9.2.5 on SmartOS) and can confirm this PR builds correctly on Illumos.

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Feb 9, 2023
pkgsrc changes:
---------------
	* Remove patches since pull request on bluge has been integrated
	  (blugelabs/bluge#139)

upstream changes:
-----------------
9.2.6 (2022-11-22)
Features and enhancements
  o Alerting: Support Prometheus durations in Provisioning API.
  o SSE: Keep value name from numeric table.
  o Transformations: Make Card Descriptions Clickable.
Bug fixes
  o MS/My/PostgresSQL: Migrate annotation query.
  o Search: Fixes issue with Recent/Starred section always displaying "General" folder.
  o Server: Write internal server error on missing write.
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.

3 participants