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-automations opened this issue Jun 17, 2016 · 6 comments
Closed

Firebird fails to build with USE_VALGRIND [CORE5284] #5562

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

Comments

@firebird-automations
Copy link
Collaborator

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-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-automations
Copy link
Collaborator Author

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-automations
Copy link
Collaborator Author

Commented by: @reevespaul

Building with USE_VALGRIND completes correctly.

Thanks.

@firebird-automations
Copy link
Collaborator Author

Modified by: @reevespaul

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

resolution: Fixed [ 1 ]

Fix Version: 3.0.1 [ 10730 ]

@firebird-automations
Copy link
Collaborator Author

Modified by: @AlexPeshkoff

Fix Version: 4.0 Alpha 1 [ 10731 ]

@firebird-automations
Copy link
Collaborator Author

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