PR#8: Update conmon to v2.1.8 - rpms/conmon

rpms / conmon

#8 Update conmon to v2.1.8

Merged 9 months ago by lsm5. Opened 9 months ago by pqatsi.

rpms/ pqatsi/conmon rawhide into rawhide

Leonardo Amaral • 9 months ago

f563ab0

conmon.spec

file modified

+1 -1

		`@@ -13,7 +13,7 @@`
		`%bcond_without docs`
		`%endif`

		`- %global built_tag v2.1.7`
		`+ %global built_tag v2.1.8`
		`%global built_tag_strip %(b=%{built_tag}; echo ${b:1})`
		`%global gen_version %(b=%{built_tag_strip}; echo ${b/-/"~"})`

sources

file modified

+1 -1

		`@@ -1,1 +1,1 @@`
		`- SHA512 (v2.1.7.tar.gz) = 95d394b399a19a62b894cdd03937ab79b81051eea1db461b1bf957ddd7626d6ca4aa108b8319ed8c08adbdf99fd960c5ba29146e8b0673b5c920708912a72973`
		`+ SHA512 (v2.1.8.tar.gz) = a277de8f8adf001c3e3ef6ef19d4bade36d48c5395a624ee99cb22708ea26a9954df76362006a1331efc06265cb790883b43e84f0006f80c1725a7470ff244c3`

pqatsi commented 9 months ago

Conmon released v2.1.8 stable with fixes that affects fedora implementations (Like https://github.com/containers/conmon/issues/438 ). This PR ask for upstream update. This release was tested on f38 with sucess.

# rpm -qi conmon
Name        : conmon
Epoch       : 2
Version     : 2.1.8
Release     : 1.fc38
Architecture: x86_64
Install Date: qua 20 set 2023 10:01:06
Group       : Unspecified
Size        : 332916
License     : Apache-2.0
Signature   : (none)
Source RPM  : conmon-2.1.8-1.fc38.src.rpm
Build Date  : qua 20 set 2023 09:58:35
Build Host  : c0de7417fbca
URL         : https://github.com/containers/conmon
Summary     : OCI container runtime monitor
Description :
OCI container runtime monitor.

lsm5 commented 9 months ago

thanks

Pull-Request has been merged by lsm5

9 months ago

pqatsi commented 9 months ago

Thank you @lsm5

I noticed missing .gitignore entry right after my commit, but you merged it first before I was able to push again. Sorry by this little mistake and thanks for https://src.fedoraproject.org/rpms/conmon/c/0fb64545686ab54ba893f7fccde0602a751ddf3e

lsm5 commented 9 months ago

Thank you @lsm5

I noticed missing .gitignore entry right after my commit, but you merged it first before I was able to push again. Sorry by this little mistake and thanks for https://src.fedoraproject.org/rpms/conmon/c/0fb64545686ab54ba893f7fccde0602a751ddf3e

No worries @pqatsi . Thing is there's an additional step involved fedpkg new-sources v2.1.8.tar.gz which uploads the tarball to lookaside cache and also updates .gitignore. I doubt you'd be able to run that step without commit access to the repo itself, so please don't worry about it.

If you're interested in being a regular fedora contributor / packager let me know.

pqatsi commented 9 months ago

@lsm5 Hello!

Thanks for the information about fedpkg new-sources!

About packaging for Fedora, I usually do fixes to my own system but I also try to contribute back when I can. For this fix, I felt wasn't hugelly complex to report a fix to upstream with a patch instead just open a bug report (I personally avoid this).

That said, I have interest to learn a bit more and this PR was my first attempt to try to return back to community some contribution. The bug at upstream was reported by too.