From: Omar Polo <o...@omarpolo.com>
Date: Thu, 03 Aug 2023 10:52:56 +0200

> On 2023/08/03 16:11:46 +0900, Masato Asou <a...@soum.co.jp> wrote:
>> Hi,
>> 
>> I have solved the problem of Valgrind not displaying function names
>> and file names when it detects a problem.
>>
>> [...]
>> 
>> comments, ok?
> 
> it's missing a REVISION bump in the Makefile since the content of the
> package changes.

I forget a REVISION bump.  Append following change.

diff -u -p -r1.30 Makefile
--- Makefile    18 Jul 2023 06:17:15 -0000      1.30
+++ Makefile    3 Aug 2023 20:29:59 -0000
@@ -5,6 +5,7 @@ CATEGORIES =            devel
 
 V =                    3.21.0
 DISTNAME =             valgrind-${V}
+REVISION =             0
 EXTRACT_SUFX =         .tar.bz2
 
 MASTER_SITES =         https://sourceware.org/pub/valgrind/

> other than this, while the changes are a bit over my head, I've
> quickly tested it and it seems to be working.  So, fwiw, OK op@.
> 
> I still get some ???  entries, for example with grep:
> 
>       $ make clean obj
>       [...]
>       $ make DEBUG='-O0 -g3'
>       [...]
>       $ valgrind obj/grep foo Makefile
>       [...]
>       ==31128== Use of uninitialised value of size 8
>       ==31128==    at 0x49D935D: ??? (in obj/grep)
>       ==31128==    by 0x10F385: grep_reallocarray (util.c:615)
>       ==31128==    by 0x10D626: add_pattern (grep.c:186)
>       ==31128==    by 0x10D426: add_patterns (grep.c:224)
>       ==31128==    by 0x10D03C: main (grep.c:473)
>       [...]

I also tried DEBUG="-O0 -g3", but ??? was not displayed.  Is your
libc.so stripped?

> but it's already so much better :-)

Thank you.  I will commit.
--
ASOU Masato

Reply via email to