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

util/sleep: exclude iOS #113

Merged
merged 1 commit into from
Sep 25, 2024
Merged

util/sleep: exclude iOS #113

merged 1 commit into from
Sep 25, 2024

Conversation

benma
Copy link
Contributor

@benma benma commented Sep 25, 2024

The previous filters would also compile the sleep_darwin.go file for iOS, failing compilation, as the darwin build tag also matches on iOS:

https://pkg.go.dev/cmd/go

Using GOOS=ios matches build tags and files as for GOOS=darwin in
addition to ios tags and files.

We need to exclude iOS explicitly here.

The previous filters would also compile the sleep_darwin.go file for
iOS, failing compilation, as the `darwin` build tag also matches on iOS:

https://pkg.go.dev/cmd/go

```
Using GOOS=ios matches build tags and files as for GOOS=darwin in
addition to ios tags and files.
```

We need to exclude iOS explicitly here.
@benma benma merged commit a2115fe into BitBoxSwiss:master Sep 25, 2024
3 checks passed
@benma benma deleted the ios branch September 25, 2024 08:05
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.

1 participant