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

tailscale: build with less features #24706

Closed
wants to merge 2 commits into from
Closed

Conversation

mochaaP
Copy link
Contributor

@mochaaP mochaaP commented Aug 1, 2024

Maintainer: me
Compile tested: aarch64_cortex-a53 @ SNAPSHOT
Run tested: not yet

Description:
See https://github.com/tailscale/tailscale/blob/main/build_dist.sh#L40.
This shaves off about 500kb from the final executable.

@BKPepe
Copy link
Member

BKPepe commented Aug 1, 2024

There is missing commit description itself to know in CLI and as well within git commands to know why those features were disabled.

@hnyman
Copy link
Contributor

hnyman commented Aug 2, 2024

package maintainer @ja-pa has been absent for a year. You might need to remove him as maintainer and assume the maintainer status in Makefile by yourself, as you @mochaaP or @p-w-p have been doing the updates lately.

See https://github.com/tailscale/tailscale/blob/main/build_dist.sh#L40.
This shaves off about 500kb from the final executable.

Signed-off-by: Zephyr Lykos <[email protected]>
@@ -28,7 +28,7 @@ GO_PKG:=\
tailscale.com/cmd/tailscaled
GO_PKG_LDFLAGS:=-X 'tailscale.com/version.longStamp=$(PKG_VERSION)-$(PKG_RELEASE) (OpenWrt)'
GO_PKG_LDFLAGS_X:=tailscale.com/version.shortStamp=$(PKG_VERSION)
GO_PKG_TAGS:=ts_include_cli
GO_PKG_TAGS:=ts_include_cli,ts_omit_aws,ts_omit_bird,ts_omit_tap,ts_omit_kube,ts_omit_completion
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

However, you added commit description exactly the same as it is in the pull request, but still you have decided to build it with less features, you pointed us to https://github.com/tailscale/tailscale/blob/main/build_dist.sh#L40 and still I am not sure what was the reason to have fewer features.

While looking at https://openwrt.org/submitting-patches, I see:

It should explain to a competent reader why you made this commit.

@systemcrash
Copy link
Contributor

ts_omit_aws omits AWS from the build

tailscale/tailscale@4f5ef82

ts_omit_bird omits BIRD from the build.

etc.

These omit flags are those proposed by the project authors themselves to reduce binary footprint (at cost of lesser used functionality).

@mochaaP mochaaP closed this Sep 18, 2024
@mochaaP
Copy link
Contributor Author

mochaaP commented Sep 18, 2024

continued in #24987

@BKPepe
Copy link
Member

BKPepe commented Sep 18, 2024 via email

@mochaaP
Copy link
Contributor Author

mochaaP commented Sep 18, 2024

This one was on the master branch from my repo, so I have to open a new one.

@BKPepe
Copy link
Member

BKPepe commented Sep 18, 2024

Never ever create pull request from the master branch, thats why you should use feature branches. You could also use master branch and create the other pull requests from different branch.

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.

4 participants