THRIFT-5776: Cpp cross test fail test#2968
Closed
CJCombrink wants to merge 2 commits intoapache:masterfrom
Closed
Conversation
CJCombrink
force-pushed
the
bugfix/THRIFT-5776_cross_test_uuid
branch
from
da9310f to
d01ceb8
Compare
MaoZiming
pushed a commit
to Trinity-data-store/Trinity
that referenced
this pull request
Apr 30, 2024
* fix: include flex and bison in dependency setup script * fix: space missing for thrift installation block * fix: add pkgconf in setup script * refactor: switch to a stable release of thrift main stream seems to have a bug that prevents compilation at this time apache/thrift#2968
Member
|
Still draft? |
Contributor
Author
|
@Jens-G I still need to look into the latest errors as mentioned on the ticket. |
CJCombrink
force-pushed
the
bugfix/THRIFT-5776_cross_test_uuid
branch
from
5ebbe73 to
3f708a7
Compare
Contributor
Author
|
@Jens-G Sorry for the delay, I am looking into this this week, I could not over the weekend. |
Contributor
Author
|
@Jens-G The last commits at least fixes the compile errors in the 'ubuntu-jammy' docker containers as well. After these changes I now also get THRIFT-5775 as per the original ticket tracking this issue. |
CJCombrink
force-pushed
the
bugfix/THRIFT-5776_cross_test_uuid
branch
from
2ae8f2a to
7a05842
Compare
- Not yet sure where the issue might be
- Define the functions - Cross test of UUID not implemented
CJCombrink
force-pushed
the
bugfix/THRIFT-5776_cross_test_uuid
branch
from
7a05842 to
5032009
Compare
5 tasks
Contributor
Author
|
Fixed as part of #2958 |
bl-rr
pushed a commit
to bl-rr/trinity-fork-1
that referenced
this pull request
May 18, 2025
* fix: include flex and bison in dependency setup script * fix: space missing for thrift installation block * fix: add pkgconf in setup script * refactor: switch to a stable release of thrift main stream seems to have a bug that prevents compilation at this time apache/thrift#2968
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
THRIFT-5776
[skip ci]anywhere in the commit message to free up build resources.