[PR #5210] [CLOSED] update client code ios build tags to add tvos target #25249

Open
opened 2026-08-05 07:05:45 -04:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbirdio/netbird/pull/5210
Author: @shuuri-labs
Created: 1/29/2026
Status: Closed

Base: mainHead: bug/update-ios-client-code-build-tags


📝 Commits (1)

  • ab987f1 update client code ios build tags to add tvos target

📊 Changes

20 files changed (+20 additions, -20 deletions)

View changed files

📝 client/iface/iface_new_ios.go (+1 -1)
📝 client/iface/udpmux/mux_ios.go (+1 -1)
📝 client/internal/debug/debug_mobile.go (+1 -1)
📝 client/internal/dns/unclean_shutdown_mobile.go (+1 -1)
📝 client/internal/dns/upstream_ios.go (+1 -1)
📝 client/internal/networkmonitor/monitor_mobile.go (+1 -1)
📝 client/internal/routemanager/dynamic/route_ios.go (+1 -1)
📝 client/internal/routemanager/notifier/notifier_ios.go (+1 -1)
📝 client/internal/routemanager/systemops/systemops_ios.go (+1 -1)
📝 client/ios/NetBirdSDK/client.go (+1 -1)
📝 client/ios/NetBirdSDK/env_list.go (+1 -1)
📝 client/ios/NetBirdSDK/gomobile.go (+1 -1)
📝 client/ios/NetBirdSDK/logger.go (+1 -1)
📝 client/ios/NetBirdSDK/login.go (+1 -1)
📝 client/ios/NetBirdSDK/peer_notifier.go (+1 -1)
📝 client/ios/NetBirdSDK/preferences.go (+1 -1)
📝 client/ios/NetBirdSDK/preferences_test.go (+1 -1)
📝 client/ios/NetBirdSDK/routes.go (+1 -1)
📝 client/net/listen_ios.go (+1 -1)
📝 management/server/testutil/store_ios.go (+1 -1)

📄 Description

Describe your changes

build tags were added to the ios client code: go:build ios. these should also reference the tvos target: go:build ios || tvos

Stack

Checklist

  • Is it a bug fix
  • Is a typo/documentation fix
  • Is a feature enhancement
  • It is a refactor
  • Created tests that fail without the change (if possible)

By submitting this pull request, you confirm that you have read and agree to the terms of the Contributor License Agreement.

Documentation

Select exactly one:

  • I added/updated documentation for this change
  • Documentation is not needed for this change (explain why)

Docs PR URL (required if "docs added" is checked)

Paste the PR link from https://github.com/netbirdio/docs here:

https://github.com/netbirdio/docs/pull/__

Summary by CodeRabbit

  • New Features
    • Added tvOS support to the NetBird client, enabling compatibility with Apple TV devices alongside existing iOS and Android platforms.

✏️ Tip: You can customize this high-level summary in your review settings.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbirdio/netbird/pull/5210 **Author:** [@shuuri-labs](https://github.com/shuuri-labs) **Created:** 1/29/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `bug/update-ios-client-code-build-tags` --- ### 📝 Commits (1) - [`ab987f1`](https://github.com/netbirdio/netbird/commit/ab987f17f072b2e0a4840fe4c87307e7313f637e) update client code ios build tags to add tvos target ### 📊 Changes **20 files changed** (+20 additions, -20 deletions) <details> <summary>View changed files</summary> 📝 `client/iface/iface_new_ios.go` (+1 -1) 📝 `client/iface/udpmux/mux_ios.go` (+1 -1) 📝 `client/internal/debug/debug_mobile.go` (+1 -1) 📝 `client/internal/dns/unclean_shutdown_mobile.go` (+1 -1) 📝 `client/internal/dns/upstream_ios.go` (+1 -1) 📝 `client/internal/networkmonitor/monitor_mobile.go` (+1 -1) 📝 `client/internal/routemanager/dynamic/route_ios.go` (+1 -1) 📝 `client/internal/routemanager/notifier/notifier_ios.go` (+1 -1) 📝 `client/internal/routemanager/systemops/systemops_ios.go` (+1 -1) 📝 `client/ios/NetBirdSDK/client.go` (+1 -1) 📝 `client/ios/NetBirdSDK/env_list.go` (+1 -1) 📝 `client/ios/NetBirdSDK/gomobile.go` (+1 -1) 📝 `client/ios/NetBirdSDK/logger.go` (+1 -1) 📝 `client/ios/NetBirdSDK/login.go` (+1 -1) 📝 `client/ios/NetBirdSDK/peer_notifier.go` (+1 -1) 📝 `client/ios/NetBirdSDK/preferences.go` (+1 -1) 📝 `client/ios/NetBirdSDK/preferences_test.go` (+1 -1) 📝 `client/ios/NetBirdSDK/routes.go` (+1 -1) 📝 `client/net/listen_ios.go` (+1 -1) 📝 `management/server/testutil/store_ios.go` (+1 -1) </details> ### 📄 Description ## Describe your changes build tags were added to the ios client code: `go:build ios`. these should also reference the tvos target: `go:build ios || tvos` ## Issue ticket number and link ## Stack <!-- branch-stack --> ### Checklist - [x] Is it a bug fix - [ ] Is a typo/documentation fix - [ ] Is a feature enhancement - [ ] It is a refactor - [ ] Created tests that fail without the change (if possible) > By submitting this pull request, you confirm that you have read and agree to the terms of the [Contributor License Agreement](https://github.com/netbirdio/netbird/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT.md). ## Documentation Select exactly one: - [ ] I added/updated documentation for this change - [x] Documentation is **not needed** for this change (explain why) ### Docs PR URL (required if "docs added" is checked) Paste the PR link from https://github.com/netbirdio/docs here: https://github.com/netbirdio/docs/pull/__ <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added tvOS support to the NetBird client, enabling compatibility with Apple TV devices alongside existing iOS and Android platforms. <sub>✏️ Tip: You can customize this high-level summary in your review settings.</sub> <!-- end of auto-generated comment: release notes by coderabbit.ai --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2026-08-05 07:05:45 -04:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: DYNR/netbird#25249