[valgrind] [Bug 371060] New: memcheck/tests/leak_cpp_interior has undefined references with clang

2016-10-17 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=371060

Bug ID: 371060
   Summary: memcheck/tests/leak_cpp_interior has undefined
references with clang
   Product: valgrind
   Version: unspecified
  Platform: Compiled Sources
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: jsew...@acm.org
  Reporter: austinengl...@gmail.com

clang -Winline -Wall -Wshadow -Wno-long-long -g -fno-stack-protector
-Wno-unused-private-field -m64-o leak_cpp_interior leak_cpp_interior.o  
leak_cpp_interior.o: In function `doit()':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:105: undefined
reference to `std::__cxx11::basic_string,
std::allocator >::operator=(char const*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:106: undefined
reference to `std::__cxx11::basic_string,
std::allocator >::operator=(std::__cxx11::basic_string, std::allocator > const&)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:107: undefined
reference to `operator new[](unsigned long)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:115: undefined
reference to `operator new[](unsigned long)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:118: undefined
reference to `operator new(unsigned long)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:119: undefined
reference to `operator new(unsigned long)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:120: undefined
reference to `operator new(unsigned long)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:121: undefined
reference to `operator new(unsigned long)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:124: undefined
reference to `std::__cxx11::basic_string,
std::allocator >::operator+=(char const*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:120: undefined
reference to `operator delete(void*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:121: undefined
reference to `operator delete(void*)'
leak_cpp_interior.o: In function `main':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:157: undefined
reference to `operator delete[](void*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:158: undefined
reference to `operator delete[](void*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:157: undefined
reference to `operator delete[](void*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:158: undefined
reference to `operator delete[](void*)'
leak_cpp_interior.o: In function `__cxx_global_var_init':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:91: undefined
reference to `std::__cxx11::basic_string,
std::allocator >::basic_string()'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:91: undefined
reference to `std::__cxx11::basic_string,
std::allocator >::~basic_string()'
leak_cpp_interior.o: In function `__cxx_global_var_init.1':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:92: undefined
reference to `std::__cxx11::basic_string,
std::allocator >::basic_string()'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:92: undefined
reference to `std::__cxx11::basic_string,
std::allocator >::~basic_string()'
leak_cpp_interior.o: In function `__clang_call_terminate':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:(.text.__clang_call_terminate[__clang_call_terminate]+0x2):
undefined reference to `__cxa_begin_catch'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:(.text.__clang_call_terminate[__clang_call_terminate]+0xb):
undefined reference to `std::terminate()'
leak_cpp_interior.o: In function `~Ce':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:39: undefined
reference to `operator delete(void*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:39: undefined
reference to `operator delete(void*)'
leak_cpp_interior.o: In function `~Ae':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:27: undefined
reference to `operator delete(void*)'
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:27: undefined
reference to `operator delete(void*)'
leak_cpp_interior.o: In function `~Be':
/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:33: undefined
reference to `operator delete(void*)'
leak_cpp_interior.o:/home/austin/src/valgrind/memcheck/tests/leak_cpp_interior.cpp:33:
more undefined references to `operator delete(void*)' follow
leak_cpp_interior.o:(.rodata._ZTI2Ae[_ZTI2Ae]+0x0): undefined reference to
`vtable for __cxxabiv1::__class_type_info'
leak_cpp_interior.o:(.rodata._ZTI2Be[_ZTI2Be]+0x0): undefined reference to
`vtable for __cxxabiv1::__class_type_info'
leak_cpp_interior.o:(.rodata._ZTI2Ce[_ZTI2Ce]+0x0): undefined reference to
`vtable for __cxxabiv1::__vmi_c

[valgrind] [Bug 371060] memcheck/tests/leak_cpp_interior has undefined references with clang

2016-10-17 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=371060

austinengl...@gmail.com  changed:

   What|Removed |Added

Version|unspecified |3.12 SVN

-- 
You are receiving this mail because:
You are watching all bug changes.


[valgrind] [Bug 371061] New: memcheck/tests/demangle has undefined references with clang

2016-10-17 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=371061

Bug ID: 371061
   Summary: memcheck/tests/demangle has undefined references with
clang
   Product: valgrind
   Version: 3.12 SVN
  Platform: Compiled Sources
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: jsew...@acm.org
  Reporter: austinengl...@gmail.com

clang -Winline -Wall -Wshadow -Wno-long-long -g -fno-stack-protector
-Wno-unused-private-field -m64-o demangle demangle.o  
demangle.o: In function `main':
/home/austin/src/valgrind/memcheck/tests/demangle.cpp:25: undefined reference
to `operator new(unsigned long)'
/home/austin/src/valgrind/memcheck/tests/demangle.cpp:27: undefined reference
to `operator new(unsigned long)'
demangle.o: In function `magic::xyzzy(int*)':
/home/austin/src/valgrind/memcheck/tests/demangle.cpp:19: undefined reference
to `operator new(unsigned long)'
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Makefile:1917: recipe for target 'demangle' failed
make[5]: *** [demangle] Error 1
make[5]: Leaving directory '/tmp/valgrind-clang/memcheck/tests'

works with:
austin@debian-laptop:/tmp/valgrind-clang$ gcc --version
gcc (Debian 6.2.0-6) 6.2.0 20161010
Copyright (C) 2016 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

fails with:
austin@debian-laptop:/tmp/valgrind-clang$ clang --version
clang version 3.8.1-12 (tags/RELEASE_381/final)
Target: x86_64-pc-linux-gnu
Thread model: posix
InstalledDir: /usr/bin

r16070, debian sid.

Reproducible: Always

Steps to Reproduce:
1. ./configure CC=clang CXX=clang
2. make -j8
3. make check

-- 
You are receiving this mail because:
You are watching all bug changes.


[valgrind] [Bug 371060] memcheck/tests/leak_cpp_interior has undefined references with clang

2016-10-18 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=371060

austinengl...@gmail.com  changed:

   What|Removed |Added

 Resolution|--- |INVALID
 Status|UNCONFIRMED |RESOLVED

--- Comment #1 from austinengl...@gmail.com  ---
User error, I had CC=clang / CXX=clang, when it should've been CC=clang
CXX=clang++

So unless we want to add a sanity check to configure, this is invalid.

-- 
You are receiving this mail because:
You are watching all bug changes.


[valgrind] [Bug 371061] memcheck/tests/demangle has undefined references with clang

2016-10-18 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=371061

austinengl...@gmail.com  changed:

   What|Removed |Added

 Resolution|--- |INVALID
 Status|UNCONFIRMED |RESOLVED

--- Comment #1 from austinengl...@gmail.com  ---
User error, I had CC=clang / CXX=clang, when it should've been CC=clang
CXX=clang++

So unless we want to add a sanity check to configure, this is invalid.

-- 
You are receiving this mail because:
You are watching all bug changes.


[valgrind] [Bug 352767] Wine/valgrind: Warning: noted but unhandled ioctl 0x5307 with no size/direction hints. (CDROMSTOP)

2016-10-31 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352767

--- Comment #4 from austinengl...@gmail.com  ---
Created attachment 101937
  --> https://bugs.kde.org/attachment.cgi?id=101937&action=edit
patch1

This adds missing ioctl defines to cdrom ioctls

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 348616] Wine/valgrind: Warning: noted but unhandled ioctl 0x5390 with no size/direction hints. (DVD_READ_STRUCT)

2016-10-31 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=348616

--- Comment #1 from austinengl...@gmail.com  ---
Created attachment 101939
  --> https://bugs.kde.org/attachment.cgi?id=101939&action=edit
patch3

Note: depends on patches to 352767

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 352767] Wine/valgrind: Warning: noted but unhandled ioctl 0x5307 with no size/direction hints. (CDROMSTOP)

2016-10-31 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352767

--- Comment #5 from austinengl...@gmail.com  ---
Created attachment 101938
  --> https://bugs.kde.org/attachment.cgi?id=101938&action=edit
patch2

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 352395] Please provide SVN revision info in --version

2016-11-01 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352395

--- Comment #3 from austinengl...@gmail.com  ---
Created attachment 101947
  --> https://bugs.kde.org/attachment.cgi?id=101947&action=edit
try 2

This only shows svn info if -v/--verbose is given.

Need to find a way to avoid redefining the package version..

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 352395] Please provide SVN revision info in --version

2016-11-01 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352395

austinengl...@gmail.com  changed:

   What|Removed |Added

  Attachment #94574|0   |1
is obsolete||
 Attachment #101947|0   |1
is obsolete||

--- Comment #4 from austinengl...@gmail.com  ---
Created attachment 101951
  --> https://bugs.kde.org/attachment.cgi?id=101951&action=edit
try 3

This version uses PACKAGE_VERSION, avoiding the redefinition.

Tested with both a proper subversion tree, as well as a git svn checkout.

Please review / apply

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 352767] Wine/valgrind: Warning: noted but unhandled ioctl 0x5307 with no size/direction hints. (CDROMSTOP)

2016-09-13 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352767

--- Comment #2 from austinengl...@gmail.com  ---
(In reply to Julian Seward from comment #1)
> Austin, do you have a patch for this?  Or for bug 348616 ?

Not currently, but I took a quick look. There are several more syscalls that
wine uses in the source that are bsd/osx specific, but I can't easily test.
Should I stub those / put fixme's, or just fix linux/generic? Or only fix the
two that currently have bugs?

-- 
You are receiving this mail because:
You are watching all bug changes.


[valgrind] [Bug 352395] Please provide SVN revision info in --version

2016-11-18 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352395

--- Comment #6 from austinengl...@gmail.com  ---
Created attachment 102288
  --> https://bugs.kde.org/attachment.cgi?id=102288&action=edit
try 4, run from make instead of configure

Makefiles are not my specialty, so comments welcome.

This has been tested working with svn and git svn.

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 352395] Please provide SVN revision info in --version

2016-11-21 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352395

austinengl...@gmail.com  changed:

   What|Removed |Added

 Attachment #101951|0   |1
is obsolete||
 Attachment #102288|0   |1
is obsolete||

--- Comment #9 from austinengl...@gmail.com  ---
Created attachment 102375
  --> https://bugs.kde.org/attachment.cgi?id=102375&action=edit
try 5

Try 5 only updates the header if if has changed, and moves the git/svn parsing
code to a Makefile variable.

Additionally, in the case of git svn, it checks that .git/svn exists rather
than .git. There are (unofficial) git repos for valgrind, e.g.,
https://github.com/mslusarz/valgrind, that this would fail for. Adding git
support would be doable, but I don't think it's worth it when valgrind itself
is on SVN. I'd like git svn to be supported for the official tree (since that's
what I use for development) :).

-- 
You are receiving this mail because:
You are watching all bug changes.

[valgrind] [Bug 352395] Please provide SVN revision info in --version

2016-11-21 Thread austinenglish
https://bugs.kde.org/show_bug.cgi?id=352395

--- Comment #10 from austinengl...@gmail.com  ---
(In reply to Ivo Raisr from comment #7)
> I had a look and overall it looks pretty good.
> However 'pushd/popd' is a bash'ism and won't work in other shells.
> Please can you try this also under shells, for example ksh?

Thanks, fixed.

> What happens (and what is the version) if this is run not inside a SVN/GIT
> repository? For example one can run 'make dist', take the tarball, unpack it
> somewhere else and build the stuff again.

The svn revisions are empty. E.g., in the case of an SVN/git svn checkout:
austin@austin2:~/src/valgrind$ ./vg-in-place -v --version
valgrind-3.12.0.SVN-16106M-vex-3283
austin@austin2:~/src/valgrind$ ./vg-in-place --version
valgrind-3.12.0.SVN

and not:
austin@austin2:~/src/valgrind$ ./vg-in-place -v --version
valgrind-3.12.0.SVN--vex-
austin@austin2:~/src/valgrind$ ./vg-in-place --version
valgrind-3.12.0.SVN

> Also this construct:
> "if test -d .svn; then svnversion -n .; elif test -d .git; then git svn info
> | \
>  grep ^Revision | cut -d ' ' -f2 | tr -d '\n'; fi"
> can be reused, for example through a Makefile variable:
> 
> EXTRACT_SVN_VERSION = ""
> vgversion.h:
> ...
> ... $(shell $(EXTRACT_SVN_VERSION)) ...

Thanks. I had trouble getting that working for a while. The variable contents
should NOT be quoted, then it works :)

-- 
You are receiving this mail because:
You are watching all bug changes.