Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

compilation fails when --with-valgrind in effect #2836

Closed
monetdb-team opened this issue Nov 30, 2020 · 0 comments
Closed

compilation fails when --with-valgrind in effect #2836

monetdb-team opened this issue Nov 30, 2020 · 0 comments

Comments

@monetdb-team
Copy link

@monetdb-team monetdb-team commented Nov 30, 2020

Date: 2011-07-11 19:10:08 +0200
From: @sjoerdmullender
To: buildtools devs <>
Version: 11.3.7 (Apr2011-SP2) [obsolete]

Last updated: 2011-09-16 15:04:29 +0200

Comment 15934

Date: 2011-07-11 19:10:08 +0200
From: @sjoerdmullender

User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20100101 Firefox/5.0
Build Identifier:

At least on Fedora 15, if you have valgrind-devel installed, compilation fails with the default configure flags. The error is:
gdk/gdk_utils.mx: In function 'GDKmmap':
gdk/gdk_utils.mx:1436:3: error: variable '_qzz_res' set but not used [-Werror=unused-but-set-variable]
gdk/gdk_utils.mx: In function 'GDKmunmap':
gdk/gdk_utils.mx:1451:2: error: variable '_qzz_res' set but not used [-Werror=unused-but-set-variable]
cc1: all warnings being treated as errors

The reason is that the VALGRIND_MALLOCLIKE_BLOCK and VALGRIND_FREELIKE_BLOCK macros contain variables that are assigned to but never otherwise used. Unfortunately this is not something we can fix in our code.

Options are: remove the valgrind-specific code (the only thing it does is provide a better stack trace); or use less strict compilation flags when compiling gdk_utils.

Reproducible: Always

Comment 15983

Date: 2011-07-25 10:59:27 +0200
From: @sjoerdmullender

Changeset 6f076133eeb0 made by Sjoerd Mullender sjoerd@acm.org in the MonetDB repo, refers to this bug.

For complete details, see http//devmonetdborg/hg/MonetDB?cmd=changeset;node=6f076133eeb0

Changeset description:

Don't look for valgrind by default.
This should do for fixing bug #2836.  It is up to the person
configuring MonetDB to use less strict compiler flags when including
valgrind support.

Comment 16018

Date: 2011-07-29 10:58:18 +0200
From: @sjoerdmullender

Apr2011-SP2 has been released.

Comment 16053

Date: 2011-07-29 13:22:59 +0200
From: @sjoerdmullender

This has been "fixed" in the sense that by default valgrind support is not enabled anymore. If you want valgrind support, you have to enable it, and you have to disable "strict" compilation.

Comment 16230

Date: 2011-09-16 15:04:29 +0200
From: @sjoerdmullender

The Aug2011 version has been released.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant