From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Andreas Helmcke <ahelmcke@ela-soft.com>
Subject: Re: [ptxdist] [APPLIED] atop: version bump 2.3.0 -> 2.10.0
Date: Mon, 5 Aug 2024 08:49:32 +0200 [thread overview]
Message-ID: <20240805064932.889878-1-m.olbrich@pengutronix.de> (raw)
In-Reply-To: <20240716123405.59362-1-ahelmcke@ela-soft.com>
Thanks, applied as 7a607c14507dd1a3d010f77640f1d6eda713e2b3.
Michael
[sent from post-receive hook]
On Mon, 05 Aug 2024 08:49:32 +0200, Andreas Helmcke <ahelmcke@ela-soft.com> wrote:
> Various bug fixes and changes.
> link: https://www.atoptool.nl/downloadatop.php
>
> * added dependency on glib
> * patch removed, not needed for gcc-13
>
> Signed-off-by: Andreas Helmcke <ahelmcke@ela-soft.com>
> Message-Id: <20240716123405.59362-1-ahelmcke@ela-soft.com>
> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
>
> diff --git a/patches/atop-2.3.0/0001-Include-sysmacros.h-to-compile-with-newer-gcc.patch b/patches/atop-2.3.0/0001-Include-sysmacros.h-to-compile-with-newer-gcc.patch
> deleted file mode 100644
> index 0d47cdd4ebab..000000000000
> --- a/patches/atop-2.3.0/0001-Include-sysmacros.h-to-compile-with-newer-gcc.patch
> +++ /dev/null
> @@ -1,46 +0,0 @@
> -From: SjonHortensius <SjonHortensius@users.noreply.github.com>
> -Date: Fri, 24 Aug 2018 18:26:58 +0200
> -Subject: [PATCH] Include sysmacros.h to compile with newer gcc
> -MIME-Version: 1.0
> -Content-Type: text/plain; charset=UTF-8
> -Content-Transfer-Encoding: 8bit
> -
> -Older gcc throws a warning
> -```
> -photosyst.c: In function 'lvmmapname':
> -photosyst.c:1465:13: warning: In the GNU C Library, "major" is defined
> - by <sys/sysmacros.h>. For historical compatibility, it is
> - currently defined by <sys/types.h> as well, but we plan to
> - remove this soon. To use "major", include <sys/sysmacros.h>
> - directly. If you did not intend to use a system-defined macro
> - "major", you should undefine it after including <sys/types.h>.
> - dmp->major = major(statbuf.st_rdev);
> -```
> -
> -Newer gcc throws an error:
> -
> -```
> -photosyst.c: In function ‘lvmmapname’:
> -photosyst.c:1482:19: error: called object ‘major’ is not a function or function pointer
> - dmp->major = major(statbuf.st_rdev);
> - ^~~~~
> -photosyst.c:1437:25: note: declared here
> - lvmmapname(unsigned int major, unsigned int minor,
> - ~~~~~~~~~~~~~^~~~~
> -```
> ----
> - photosyst.c | 1 +
> - 1 file changed, 1 insertion(+)
> -
> -diff --git a/photosyst.c b/photosyst.c
> -index 38828ec498a5..caed3529ae27 100644
> ---- a/photosyst.c
> -+++ b/photosyst.c
> -@@ -152,6 +152,7 @@
> - static const char rcsid[] = "$Id: photosyst.c,v 1.38 2010/11/19 07:40:40 gerlof Exp $";
> -
> - #include <sys/types.h>
> -+#include <sys/sysmacros.h>
> - #include <stdio.h>
> - #include <string.h>
> - #include <unistd.h>
> diff --git a/patches/atop-2.3.0/series b/patches/atop-2.3.0/series
> deleted file mode 100644
> index 95bd83266e79..000000000000
> --- a/patches/atop-2.3.0/series
> +++ /dev/null
> @@ -1,4 +0,0 @@
> -# generated by git-ptx-patches
> -#tag:base --start-number 1
> -0001-Include-sysmacros.h-to-compile-with-newer-gcc.patch
> -# 7ac7bb46bb4801fb2be3b7d3f950bbd3 - git-ptx-patches magic
> diff --git a/rules/atop.in b/rules/atop.in
> index 3bdeb47b92a8..cffc1356cfc1 100644
> --- a/rules/atop.in
> +++ b/rules/atop.in
> @@ -5,6 +5,7 @@ config ATOP
> select ZLIB
> select NCURSES
> select LIBC_M
> + select GLIB
> help
> Monitor for system resources and process activity
>
> diff --git a/rules/atop.make b/rules/atop.make
> index 5a6c573859f9..193ced1fc2b9 100644
> --- a/rules/atop.make
> +++ b/rules/atop.make
> @@ -15,8 +15,8 @@ PACKAGES-$(PTXCONF_ATOP) += atop
> #
> # Paths and names
> #
> -ATOP_VERSION := 2.3.0
> -ATOP_MD5 := 48e1dbef8c7d826e68829a8d5fc920fc
> +ATOP_VERSION := 2.10.0
> +ATOP_MD5 := 6d14559b59e25d15dbcfa978ed0ec50e
> ATOP := atop-$(ATOP_VERSION)
> ATOP_URL := http://www.atoptool.nl/download/$(ATOP).tar.gz
> ATOP_SOURCE := $(SRCDIR)/$(ATOP).tar.gz
prev parent reply other threads:[~2024-08-05 6:52 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-16 12:34 [ptxdist] [PATCH] " Andreas Helmcke
2024-08-05 6:49 ` Michael Olbrich [this message]
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=20240805064932.889878-1-m.olbrich@pengutronix.de \
--to=m.olbrich@pengutronix.de \
--cc=ahelmcke@ela-soft.com \
--cc=ptxdist@pengutronix.de \
/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