From: classabbyamp <classabbyamp@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: [PR PATCH] [RFC] various: remove -w -s from go_ldflags
Date: Mon, 08 Aug 2022 19:44:31 +0200 [thread overview]
Message-ID: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-38538@inbox.vuxu.org> (raw)
[-- Attachment #1: Type: text/plain, Size: 1063 bytes --]
There is a new pull request by classabbyamp against master on the void-packages repository
https://github.com/classabbyamp/void-packages fix/remove-strip-go
https://github.com/void-linux/void-packages/pull/38538
[RFC] various: remove -w -s from go_ldflags
- aws-vault: remove -w -s from go_ldflags
- added in #21634
- circleci-cli: remove -w -s from go_ldflags
- added in #25101
- coredns: remove -w -s from go_ldflags
- added in #23578
- gotify-cli: remove -w -s from go_ldflags
- added in #34653
- gotify-server: remove -w -s from go_ldflags
- added in #34653
- v2ray: remove -w -s from go_ldflags
- added in #31653
I'd like to add something to the go build_style to prevent it from applying and/or put a warning in the log, but I'm still working on that
<!-- Uncomment relevant sections and delete options which are not applicable -->
#### Testing the changes
- I tested the changes in this PR: **YES**|**briefly**|**NO**
A patch file from https://github.com/void-linux/void-packages/pull/38538.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-fix/remove-strip-go-38538.patch --]
[-- Type: text/x-diff, Size: 6323 bytes --]
From c83e8fc2379856410b09ee6f04b70fe15558eba9 Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Mon, 8 Aug 2022 12:21:20 -0400
Subject: [PATCH 1/6] aws-vault: remove -w -s from go_ldflags
---
srcpkgs/aws-vault/template | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/aws-vault/template b/srcpkgs/aws-vault/template
index a3f6d60583c6..ce7b0be3307a 100644
--- a/srcpkgs/aws-vault/template
+++ b/srcpkgs/aws-vault/template
@@ -1,10 +1,10 @@
# Template file for 'aws-vault'
pkgname=aws-vault
version=6.3.1
-revision=1
+revision=2
build_style=go
go_import_path=github.com/99designs/aws-vault/v6
-go_ldflags="-X \"main.Version=${version}\" -s -w"
+go_ldflags="-X main.Version=${version}"
short_desc="Vault for securely storing and accessing AWS credentials"
maintainer="klardotsh <josh@klar.sh>"
license="MIT"
From c88b1f8d14cf469cab82bb6d797d7982522d79b8 Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Mon, 8 Aug 2022 12:21:25 -0400
Subject: [PATCH 2/6] circleci-cli: remove -w -s from go_ldflags
---
srcpkgs/circleci-cli/template | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/srcpkgs/circleci-cli/template b/srcpkgs/circleci-cli/template
index 61978ffded01..6c04611c72ee 100644
--- a/srcpkgs/circleci-cli/template
+++ b/srcpkgs/circleci-cli/template
@@ -1,12 +1,11 @@
# Template file for 'circleci-cli'
pkgname=circleci-cli
version=0.1.15224
-revision=1
+revision=2
build_style=go
build_helper=qemu
go_import_path=github.com/CircleCI-Public/${pkgname}
-go_ldflags="-s -w
- -X github.com/CircleCI-Public/${pkgname}/version.Version=${version}
+go_ldflags="-X github.com/CircleCI-Public/${pkgname}/version.Version=${version}
-X github.com/CircleCI-Public/${pkgname}/version.packageManager=xbps"
hostmakedepends="packr2"
short_desc="Use CircleCI from the command line"
From 834712d145cee8051720544085b31e2ac3a599a1 Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Mon, 8 Aug 2022 12:21:25 -0400
Subject: [PATCH 3/6] coredns: remove -w -s from go_ldflags
---
srcpkgs/coredns/template | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/coredns/template b/srcpkgs/coredns/template
index fe48c2347f91..263df82c6476 100644
--- a/srcpkgs/coredns/template
+++ b/srcpkgs/coredns/template
@@ -1,7 +1,7 @@
# Template file for 'coredns'
pkgname=coredns
version=1.8.6
-revision=1
+revision=2
build_style=go
go_import_path=github.com/coredns/coredns
hostmakedepends="git make mmark"
@@ -14,7 +14,7 @@ checksum=cbe3764afe2148b8047ea7e5cbba5108c298dee3a9a0391028e2980e35beaa2b
make_dirs="/etc/coredns 0750 root root"
_git_commit=f59c03d09c3a3a12f571ad1087b979325f3dae30
-go_ldflags+=" -s -w -X github.com/coredns/coredns/coremain.GitCommit=${_git_commit}"
+go_ldflags+="-X github.com/coredns/coredns/coremain.GitCommit=${_git_commit}"
post_build() {
make -f Makefile.doc MMARK="$(command -v mmark) -man" man/coredns.1 man/corefile.5 plugins
From a383c7b891d34b2a9d14737d88edef61a9a9cb70 Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Mon, 8 Aug 2022 12:21:26 -0400
Subject: [PATCH 4/6] gotify-cli: remove -w -s from go_ldflags
---
srcpkgs/gotify-cli/template | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/srcpkgs/gotify-cli/template b/srcpkgs/gotify-cli/template
index 91fea40fca9d..d52b7a8cd37a 100644
--- a/srcpkgs/gotify-cli/template
+++ b/srcpkgs/gotify-cli/template
@@ -1,12 +1,11 @@
# Template file for 'gotify-cli'
pkgname=gotify-cli
version=2.2.2
-revision=1
+revision=2
wrksrc="cli-$version"
build_style=go
go_import_path="github.com/gotify/cli/v2"
-go_ldflags="-w -s -X main.Version=${version} \
- -X main.BuildDate=${SOURCE_DATE_EPOCH} \
+go_ldflags="-X main.Version=${version} -X main.BuildDate=${SOURCE_DATE_EPOCH}
-X main.Mode=prod"
short_desc="Command line interface for pushing messages to a Gotify server"
maintainer="Joel Beckmeyer <joel@beckmeyer.us>"
From 0a9d73c19d8d095b12394b5668727124c7d7c5fa Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Mon, 8 Aug 2022 12:21:27 -0400
Subject: [PATCH 5/6] gotify-server: remove -w -s from go_ldflags
---
srcpkgs/gotify-server/template | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/srcpkgs/gotify-server/template b/srcpkgs/gotify-server/template
index e306deef9220..d48b32a2a843 100644
--- a/srcpkgs/gotify-server/template
+++ b/srcpkgs/gotify-server/template
@@ -1,14 +1,12 @@
# Template file for 'gotify-server'
pkgname=gotify-server
version=2.1.5
-revision=1
+revision=2
wrksrc="server-$version"
build_style=go
go_import_path="github.com/gotify/server/v2"
-go_ldflags="-w -s -extldflags=-fuse-ld=bfd \
- -X main.Version=${version} \
- -X main.BuildDate=${SOURCE_DATE_EPOCH} \
- -X main.Mode=prod"
+go_ldflags="-extldflags=-fuse-ld=bfd -X main.Version=${version}
+ -X main.BuildDate=${SOURCE_DATE_EPOCH} -X main.Mode=prod"
hostmakedepends="yarn packr2"
short_desc="Simple server for sending and receiving messages"
maintainer="Joel Beckmeyer <joel@beckmeyer.us>"
From 8a46a6f12ea4221bbe41b5f8d814ebd7f92c1daa Mon Sep 17 00:00:00 2001
From: classabbyamp <void@placeviolette.net>
Date: Mon, 8 Aug 2022 12:21:27 -0400
Subject: [PATCH 6/6] v2ray: remove -w -s from go_ldflags
---
srcpkgs/v2ray/template | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/v2ray/template b/srcpkgs/v2ray/template
index cf6cd28a534a..d22b5247498f 100644
--- a/srcpkgs/v2ray/template
+++ b/srcpkgs/v2ray/template
@@ -1,11 +1,13 @@
# Template file for 'v2ray'
pkgname=v2ray
version=4.44.0
-revision=1
+revision=2
wrksrc=${pkgname}-core-${version}
build_style=go
go_import_path="github.com/v2fly/v2ray-core/v4"
-go_ldflags="-X github.com/v2fly/v2ray-core/v4.codename=$pkgname -X github.com/v2fly/v2ray-core/v4.version=$version -X github.com/v2fly/v2ray-core/v4.build=$SOURCE_DATE_EPOCH -s -w -buildid="
+go_ldflags="-X github.com/v2fly/v2ray-core/v4.codename=$pkgname
+ -X github.com/v2fly/v2ray-core/v4.version=$version
+ -X github.com/v2fly/v2ray-core/v4.build=$SOURCE_DATE_EPOCH -buildid="
short_desc="Platform for building proxies to bypass network restrictions"
maintainer="ipkalm <ipkalm@outlook.com>"
license="MIT"
next reply other threads:[~2022-08-08 17:44 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-08-08 17:44 classabbyamp [this message]
2022-08-09 0:22 ` classabbyamp
2022-08-09 0:26 ` classabbyamp
2022-08-09 8:48 ` ketlrznt
2022-08-09 8:48 ` ketlrznt
2022-08-09 8:49 ` ketlrznt
2022-08-09 10:22 ` ketlrznt
2022-08-09 10:23 ` ketlrznt
2022-08-09 13:54 ` ketlrznt
2022-08-09 13:54 ` classabbyamp
2022-08-09 13:54 ` classabbyamp
2022-08-10 7:21 ` [PR PATCH] [Updated] " classabbyamp
2022-08-10 7:34 ` classabbyamp
2022-08-10 17:01 ` [RFC] various: remove -s/-w " the-maldridge
2022-08-10 17:24 ` classabbyamp
2022-08-10 17:25 ` the-maldridge
2022-08-10 19:01 ` [PR PATCH] [Merged]: " classabbyamp
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-38538@inbox.vuxu.org \
--to=classabbyamp@users.noreply.github.com \
--cc=ml@inbox.vuxu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).