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

Firebird fails to build with USE_VALGRIND [CORE5284] #5562

Closed
firebird-issue-importer opened this issue Jun 17, 2016 · 6 comments
Closed

Firebird fails to build with USE_VALGRIND [CORE5284] #5562

firebird-issue-importer opened this issue Jun 17, 2016 · 6 comments

Comments

@firebird-issue-importer

Submitted by: @reevespaul

The first build problem arrives with DELAYED_EXTENT_COUNT being used in
anonymous namespace, before it has been defined globally. That was easy enough
to work around. But then this error is thrown:

alloc.cpp:1866:51: error: 'class Firebird::MemBlock' has no member
named 'length' VALGRIND_MAKE_WRITABLE(object, block->length));

Looking at MemBlock there is certainly no length member, but
MemBaseHunk does have one.

Commits: 44e97a7 ab0ddd8

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jun 17, 2016

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jul 7, 2016

Commented by: @AlexPeshkoff

Paul, I've fixed the build but can not check how does valgrind work - it's broken on gentoo. There is fix procedure but it involves semi0manual rebuild of C library turning off some optimizations. I do not use valgrind for a long time (tools that were built into Jim's memory manager are enough for me).

Please check are there any more issues. If yes, send them to me, or close an item.

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jul 8, 2016

Commented by: @reevespaul

Building with USE_VALGRIND completes correctly.

Thanks.

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jul 8, 2016

Modified by: @reevespaul

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 3.0.1 [ 10730 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jul 8, 2016

Modified by: @AlexPeshkoff

Fix Version: 4.0 Alpha 1 [ 10731 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Aug 4, 2016

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Cannot be tested

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment