Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] gopass-jsonapi: update to 1.14.3.
@ 2022-07-26 22:51 kuhnchris
  2022-07-26 22:53 ` kuhnchris
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: kuhnchris @ 2022-07-26 22:51 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1101 bytes --]

There is a new pull request by kuhnchris against master on the void-packages repository

https://github.com/kuhnchris/void-packages master
https://github.com/void-linux/void-packages/pull/38289

gopass-jsonapi: update to 1.14.3.
#### Testing the changes
- I tested the changes in this PR: YES

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc

---
Previous revision accidently didn't set the -X main.version=${version} making gopassbridge complain about "please upgrade your gopass version to >1.8.3" despite having 1.13.X installed.

For future references, a test client for this is found in the gopassbridge repository (test-client directory) and a check can be invoked like this:
```
(master) gopassbridge/test-client % echo '{"type": "getVersion"}' | ./test-client | gopass-jsonapi listen                                                                                                
3{"version":"1.14.3","major":1,"minor":14,"patch":3}%       
```

A patch file from https://github.com/void-linux/void-packages/pull/38289.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-master-38289.patch --]
[-- Type: text/x-diff, Size: 866 bytes --]

From cd7b0d0c97c76643dc3149bc484d24623c044630 Mon Sep 17 00:00:00 2001
From: KuhnChris <kuhnchris@kuhnchris.eu>
Date: Wed, 27 Jul 2022 00:48:16 +0200
Subject: [PATCH] gopass-jsonapi: update to 1.14.3.

---
 srcpkgs/gopass-jsonapi/template | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/gopass-jsonapi/template b/srcpkgs/gopass-jsonapi/template
index 65408ca21596..e81f6d08cdb1 100644
--- a/srcpkgs/gopass-jsonapi/template
+++ b/srcpkgs/gopass-jsonapi/template
@@ -1,10 +1,11 @@
 # Template file for 'gopass-jsonapi'
 pkgname=gopass-jsonapi
 version=1.14.3
-revision=1
+revision=2
 create_wrksrc=yes
 build_style=go
 go_import_path=github.com/gopasspw/gopass-jsonapi
+go_ldflags="-X main.version=${version}"
 depends="gopass"
 short_desc="Gopass JSON bridge for extensions"
 maintainer="KuhnChris <kuhnchris+voidpackages@kuhnchris.eu>"

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: gopass-jsonapi: update to 1.14.3.
  2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
@ 2022-07-26 22:53 ` kuhnchris
  2022-07-29  7:30 ` classabbyamp
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: kuhnchris @ 2022-07-26 22:53 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 339 bytes --]

New comment by kuhnchris on void-packages repository

https://github.com/void-linux/void-packages/pull/38289#issuecomment-1196060980

Comment:
On a related note: the `go_ldflags` parameter is also not mentioned in the Contribution Guide / Manual. I just accidently found it while checking other templates like `docker-compose` or `revive`

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: gopass-jsonapi: update to 1.14.3.
  2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
  2022-07-26 22:53 ` kuhnchris
@ 2022-07-29  7:30 ` classabbyamp
  2022-07-29 13:20 ` kuhnchris
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: classabbyamp @ 2022-07-29  7:30 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 236 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/38289#issuecomment-1198968772

Comment:
commit message should be `gopass-jsonapi: add version to go_ldflags` or something similar

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: gopass-jsonapi: update to 1.14.3.
  2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
  2022-07-26 22:53 ` kuhnchris
  2022-07-29  7:30 ` classabbyamp
@ 2022-07-29 13:20 ` kuhnchris
  2022-07-29 15:32 ` paper42
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: kuhnchris @ 2022-07-29 13:20 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 402 bytes --]

New comment by kuhnchris on void-packages repository

https://github.com/void-linux/void-packages/pull/38289#issuecomment-1199278071

Comment:
> commit message should be `gopass-jsonapi: add version to go_ldflags` or something similar

really? I thought it was disallowed to add anything aside what `xbump` pre-fills? I can change the commit message, sure, just making sure I don't cause any issues.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: gopass-jsonapi: update to 1.14.3.
  2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
                   ` (2 preceding siblings ...)
  2022-07-29 13:20 ` kuhnchris
@ 2022-07-29 15:32 ` paper42
  2022-08-01 17:05 ` classabbyamp
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: paper42 @ 2022-07-29 15:32 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 560 bytes --]

New comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/38289#issuecomment-1199547462

Comment:
> > commit message should be `gopass-jsonapi: add version to go_ldflags` or something similar
> 
> really? I thought it was disallowed to add anything aside what `xbump` pre-fills? I can change the commit message, sure, just making sure I don't cause any issues.

xbump can only detect updates and new packages, so every other commit has to named manually. This is not an update, so the commit shouldn't say it is.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: gopass-jsonapi: update to 1.14.3.
  2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
                   ` (3 preceding siblings ...)
  2022-07-29 15:32 ` paper42
@ 2022-08-01 17:05 ` classabbyamp
  2022-08-01 17:05 ` [PR PATCH] [Closed]: " classabbyamp
  2022-08-01 18:24 ` kuhnchris
  6 siblings, 0 replies; 8+ messages in thread
From: classabbyamp @ 2022-08-01 17:05 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 195 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/38289#issuecomment-1201474856

Comment:
done in a3bf3340bf7dd3108b4362398f10de671819814c

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PR PATCH] [Closed]: gopass-jsonapi: update to 1.14.3.
  2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
                   ` (4 preceding siblings ...)
  2022-08-01 17:05 ` classabbyamp
@ 2022-08-01 17:05 ` classabbyamp
  2022-08-01 18:24 ` kuhnchris
  6 siblings, 0 replies; 8+ messages in thread
From: classabbyamp @ 2022-08-01 17:05 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 947 bytes --]

There's a closed pull request on the void-packages repository

gopass-jsonapi: update to 1.14.3.
https://github.com/void-linux/void-packages/pull/38289

Description:
#### Testing the changes
- I tested the changes in this PR: YES

#### Local build testing
- I built this PR locally for my native architecture, x86_64-glibc

---
Previous revision accidently didn't set the -X main.version=${version} making gopassbridge complain about "please upgrade your gopass version to >1.8.3" despite having 1.13.X installed.

For future references, a test client for this is found in the gopassbridge repository (test-client directory) and a check can be invoked like this:
```
(master) gopassbridge/test-client % echo '{"type": "getVersion"}' | ./test-client | gopass-jsonapi listen                                                                                                
3{"version":"1.14.3","major":1,"minor":14,"patch":3}%       
```

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: gopass-jsonapi: update to 1.14.3.
  2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
                   ` (5 preceding siblings ...)
  2022-08-01 17:05 ` [PR PATCH] [Closed]: " classabbyamp
@ 2022-08-01 18:24 ` kuhnchris
  6 siblings, 0 replies; 8+ messages in thread
From: kuhnchris @ 2022-08-01 18:24 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 150 bytes --]

New comment by kuhnchris on void-packages repository

https://github.com/void-linux/void-packages/pull/38289#issuecomment-1201554675

Comment:
thanks

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2022-08-01 18:24 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-26 22:51 [PR PATCH] gopass-jsonapi: update to 1.14.3 kuhnchris
2022-07-26 22:53 ` kuhnchris
2022-07-29  7:30 ` classabbyamp
2022-07-29 13:20 ` kuhnchris
2022-07-29 15:32 ` paper42
2022-08-01 17:05 ` classabbyamp
2022-08-01 17:05 ` [PR PATCH] [Closed]: " classabbyamp
2022-08-01 18:24 ` kuhnchris

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).