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 building on FreeBSD #2395

Closed
wants to merge 2 commits into from
Closed

Fix building on FreeBSD #2395

wants to merge 2 commits into from

Conversation

wkloucek
Copy link
Contributor

@wkloucek wkloucek commented Dec 21, 2021

rebased version of #1499, closes #1499

Closes owncloud/ocis#1217

* Extracted common error methods to `errtypes` package
* Fixed type error on Bavail ( https://golang.org/src/syscall/ztypes_freebsd_arm64.go#L125 )
* Alias ENODATA to ENOATTR

How to test this on Linux:

```
export GOOS=freebsd
make
```
@wkloucek wkloucek self-assigned this Dec 23, 2021
@wkloucek wkloucek marked this pull request as ready for review January 5, 2022 08:26
@wkloucek wkloucek requested review from glpatcern, ishank011, labkode and a team as code owners January 5, 2022 08:26
@ishank011
Copy link
Contributor

@wkloucek the issue is that with the go-prompt update, taking user input doesn't work, at least on mac systems. https://github.com/cs3org/reva/blob/master/cmd/reva/login.go#L99

@wkloucek wkloucek closed this Jan 18, 2022
@wkloucek wkloucek deleted the fix-freebsd branch January 18, 2022 13:15
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