Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] metalog: update to 20230707
@ 2023-07-07 14:00 jovanlanik
  2023-07-07 14:48 ` jovanlanik
                   ` (8 more replies)
  0 siblings, 9 replies; 10+ messages in thread
From: jovanlanik @ 2023-07-07 14:00 UTC (permalink / raw)
  To: ml

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

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

https://github.com/jovanlanik/void-packages master
https://github.com/void-linux/void-packages/pull/44912

metalog: update to 20230707
#### Testing the changes
- I tested the changes in this PR: **briefly**

Updated and added changelog as asked in #35617 (seems it was only recently changed again)

@teldra Would it also possible be for me to adopt metalog as maintainer? I am a contributor and user of the program so I follow development closely...


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

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

From 4b1325ad95441b1b9550b07d29211711e504ff90 Mon Sep 17 00:00:00 2001
From: Jovan Lanik <jox969@gmail.com>
Date: Fri, 7 Jul 2023 15:54:34 +0200
Subject: [PATCH] metalog: update to 20230707

---
 srcpkgs/metalog/template | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/metalog/template b/srcpkgs/metalog/template
index 792b4bd32b48..b47c7fa2db31 100644
--- a/srcpkgs/metalog/template
+++ b/srcpkgs/metalog/template
@@ -1,6 +1,6 @@
 # Template file for 'metalog'
 pkgname=metalog
-version=20220214
+version=20230707
 revision=1
 build_style=gnu-configure
 conf_files="/etc/metalog.conf"
@@ -10,8 +10,9 @@ short_desc="Modern replacement for syslogd and klogd"
 maintainer="teldra <teldra@rotce.de>"
 license="GPL-2.0-only"
 homepage="https://github.com/hvisage/metalog"
+changelog="https://github.com/hvisage/metalog/blob/master/NEWS"
 distfiles="https://github.com/hvisage/metalog/archive/metalog-${version}.tar.gz"
-checksum=3a9dd6f1460d90296771c81aa71fc0f45dd25ef22af63f48e6cbe5422a88f3bf
+checksum=135caf2cc9ce48364f7cd39a58c904cbd63631c9963e1c239b2df8b2ca023dc1
 
 pre_configure() {
 	./autogen.sh

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

* Re: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
@ 2023-07-07 14:48 ` jovanlanik
  2023-07-07 15:00 ` jovanlanik
                   ` (7 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: jovanlanik @ 2023-07-07 14:48 UTC (permalink / raw)
  To: ml

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

New comment by jovanlanik on void-packages repository

https://github.com/void-linux/void-packages/pull/44912#issuecomment-1625527124

Comment:
Upon further testing it stopped my system from booting...

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

* Re: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
  2023-07-07 14:48 ` jovanlanik
@ 2023-07-07 15:00 ` jovanlanik
  2023-07-07 16:01 ` jovanlanik
                   ` (6 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: jovanlanik @ 2023-07-07 15:00 UTC (permalink / raw)
  To: ml

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

New comment by jovanlanik on void-packages repository

https://github.com/void-linux/void-packages/pull/44912#issuecomment-1625544731

Comment:
Might have something to do with #42026...

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

* Re: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
  2023-07-07 14:48 ` jovanlanik
  2023-07-07 15:00 ` jovanlanik
@ 2023-07-07 16:01 ` jovanlanik
  2023-07-07 16:04 ` classabbyamp
                   ` (5 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: jovanlanik @ 2023-07-07 16:01 UTC (permalink / raw)
  To: ml

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

New comment by jovanlanik on void-packages repository

https://github.com/void-linux/void-packages/pull/44912#issuecomment-1625625388

Comment:
Seems like metalog with the `-v` flag outputs all logs to stdout and stdout is logged back to metalog, the message continues getting logged again and again...
```
Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 16:25:06 [metalog] Jul 07 15:47:49 [sudo] pam_unix(sudo:session): session opened for user root(uid=0) by (
```
I can think of a couple of solutions but the most similar result to current package is deleting the log directory :
```
post_install() {
	vsv metalog
	rm -rf "${DESTDIR}/etc/sv/metalog/log"
}
```

@classabbyamp Maybe the `vsv` function could have a flag to opt-out of logging?

@teldra What do you think?

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

* Re: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
                   ` (2 preceding siblings ...)
  2023-07-07 16:01 ` jovanlanik
@ 2023-07-07 16:04 ` classabbyamp
  2023-07-07 16:05 ` classabbyamp
                   ` (4 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: classabbyamp @ 2023-07-07 16:04 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/44912#issuecomment-1625629867

Comment:
i think rm after vsv is a fine way to do it

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

* Re: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
                   ` (3 preceding siblings ...)
  2023-07-07 16:04 ` classabbyamp
@ 2023-07-07 16:05 ` classabbyamp
  2023-07-07 16:20 ` [PR PATCH] [Updated] " jovanlanik
                   ` (3 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: classabbyamp @ 2023-07-07 16:05 UTC (permalink / raw)
  To: ml

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/44912#issuecomment-1625629867

Comment:
i think rm after vsv is a fine way to do it. maybe put a note as to why

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

* Re: [PR PATCH] [Updated] metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
                   ` (4 preceding siblings ...)
  2023-07-07 16:05 ` classabbyamp
@ 2023-07-07 16:20 ` jovanlanik
  2023-07-08 15:13 ` jovanlanik
                   ` (2 subsequent siblings)
  8 siblings, 0 replies; 10+ messages in thread
From: jovanlanik @ 2023-07-07 16:20 UTC (permalink / raw)
  To: ml

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

There is an updated pull request by jovanlanik against master on the void-packages repository

https://github.com/jovanlanik/void-packages master
https://github.com/void-linux/void-packages/pull/44912

metalog: update to 20230707
#### Testing the changes
- I tested the changes in this PR: **briefly**

Updated and added changelog as asked in #35617 (seems it was only recently changed again)

@teldra Would it also possible be for me to adopt metalog as maintainer? I am a contributor and user of the program so I follow development closely...


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

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

From 081020aab8dc4edc3f120a751f8d482328c4e402 Mon Sep 17 00:00:00 2001
From: Jovan Lanik <jox969@gmail.com>
Date: Fri, 7 Jul 2023 15:54:34 +0200
Subject: [PATCH] metalog: update to 20230707

---
 srcpkgs/metalog/template | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/metalog/template b/srcpkgs/metalog/template
index 792b4bd32b48..ee54b2c82a7f 100644
--- a/srcpkgs/metalog/template
+++ b/srcpkgs/metalog/template
@@ -1,6 +1,6 @@
 # Template file for 'metalog'
 pkgname=metalog
-version=20220214
+version=20230707
 revision=1
 build_style=gnu-configure
 conf_files="/etc/metalog.conf"
@@ -10,8 +10,9 @@ short_desc="Modern replacement for syslogd and klogd"
 maintainer="teldra <teldra@rotce.de>"
 license="GPL-2.0-only"
 homepage="https://github.com/hvisage/metalog"
+changelog="https://github.com/hvisage/metalog/blob/master/NEWS"
 distfiles="https://github.com/hvisage/metalog/archive/metalog-${version}.tar.gz"
-checksum=3a9dd6f1460d90296771c81aa71fc0f45dd25ef22af63f48e6cbe5422a88f3bf
+checksum=135caf2cc9ce48364f7cd39a58c904cbd63631c9963e1c239b2df8b2ca023dc1
 
 pre_configure() {
 	./autogen.sh
@@ -19,4 +20,8 @@ pre_configure() {
 
 post_install() {
 	vsv metalog
+
+	# Disable logging for metaloga to avoid any issues
+	# https://github.com/void-linux/void-packages/pull/44912#issuecomment-1625625388
+	rm -rf "${DESTDIR}/etc/sv/metalog/log"
 }

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

* Re: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
                   ` (5 preceding siblings ...)
  2023-07-07 16:20 ` [PR PATCH] [Updated] " jovanlanik
@ 2023-07-08 15:13 ` jovanlanik
  2023-07-12 13:43 ` [PR PATCH] [Merged]: " Duncaen
  2023-08-18 10:06 ` sgn
  8 siblings, 0 replies; 10+ messages in thread
From: jovanlanik @ 2023-07-08 15:13 UTC (permalink / raw)
  To: ml

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

New comment by jovanlanik on void-packages repository

https://github.com/void-linux/void-packages/pull/44912#issuecomment-1627374788

Comment:
No problems after deleting the log directory.

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

* Re: [PR PATCH] [Merged]: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
                   ` (6 preceding siblings ...)
  2023-07-08 15:13 ` jovanlanik
@ 2023-07-12 13:43 ` Duncaen
  2023-08-18 10:06 ` sgn
  8 siblings, 0 replies; 10+ messages in thread
From: Duncaen @ 2023-07-12 13:43 UTC (permalink / raw)
  To: ml

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

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

metalog: update to 20230707
https://github.com/void-linux/void-packages/pull/44912

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

Updated and added changelog as asked in #35617 (seems it was only recently changed again)

@teldra Would it also possible be for me to adopt metalog as maintainer? I am a contributor and user of the program so I follow development closely...


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

* Re: metalog: update to 20230707
  2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
                   ` (7 preceding siblings ...)
  2023-07-12 13:43 ` [PR PATCH] [Merged]: " Duncaen
@ 2023-08-18 10:06 ` sgn
  8 siblings, 0 replies; 10+ messages in thread
From: sgn @ 2023-08-18 10:06 UTC (permalink / raw)
  To: ml

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

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/44912#issuecomment-1683679779

Comment:
Should we remove `-v` instead?

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

end of thread, other threads:[~2023-08-18 10:06 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-07 14:00 [PR PATCH] metalog: update to 20230707 jovanlanik
2023-07-07 14:48 ` jovanlanik
2023-07-07 15:00 ` jovanlanik
2023-07-07 16:01 ` jovanlanik
2023-07-07 16:04 ` classabbyamp
2023-07-07 16:05 ` classabbyamp
2023-07-07 16:20 ` [PR PATCH] [Updated] " jovanlanik
2023-07-08 15:13 ` jovanlanik
2023-07-12 13:43 ` [PR PATCH] [Merged]: " Duncaen
2023-08-18 10:06 ` sgn

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