Bring back tcc support ##build#17295
Conversation
This comment has been minimized.
This comment has been minimized.
|
It fails because of the TCC error it seems: |
|
Removing from 4.5.0 because this has been an issue for over 3 years, I don't think it's that important at this point. |
|
Probably we should send a bug description (or a patch) to |
This comment has been minimized.
This comment has been minimized.
|
TinyCC developers fixed the problem, also attached the patch to radare2, but it's still not sufficient to build it: https://lists.nongnu.org/archive/html/tinycc-devel/2020-08/txt5WwrLjrHAf.txt |
|
I'm not sure about the extra dependencies, if those are really needed why they didnt failed in any other compiler? would be good to compare make depgraph to see if there's any issue there |
|
Good patches @XVilka ! is there any other patch for tcc needed to continue in here? can you post the fixes/commits/issues from their repo? thanks! |
|
Seems the only remaining problem is this library dependency, can you @trufae please check? Just be sure to update |
|
Still a dependency problem: |
Codecov Report
@@ Coverage Diff @@
## master #17295 +/- ##
=======================================
Coverage 40.16% 40.16%
=======================================
Files 1132 1132
Lines 356623 356623
=======================================
+ Hits 143238 143248 +10
+ Misses 213385 213375 -10
Continue to review full report at Codecov.
|
|
@XVilka any update from tcc devs? |
|
@trufae update for what? They fixed all bugs in git that were found during the radare2 compilation. It's now only the dependency problem that prevents radare2 being built. |
|
Amazing! great to see tcc back to r2land. ive checked the error and it seems like its not buildling r_io for some reason i ignore . therefor later it fails. can i have a shell in this env? otherwise i can try to repro in local. Can you repro? |
|
It builds r_io just fine, I think: |
|
@ret2libc please take a look at tree-sitter library changes. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@XVilka imho its fine to merge in the current state. the remaining bug is not in our side i think |
1578e53 to
0b7ca28
Compare
|
I cherry-picked to master all changes in radare2 codebase itself. Therefore, three subsystems that are in fact external projects:
We should try to mainline these changes. |
|
All possible changes were mainstreamed, all bugs in TinyCC were fixed. The only remaining issue is in radare2 - something about module dependency or so. @trufae |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
|
@trufae @condret you can merge this even now - just keep in mind that the change in jemalloc should be repeated again in case you decide to update jemalloc to a new version. Another thing - some tests are broken, but I think it's still better than broken compilation. Also don't forget to remove these lines: |
|
Appreciate |
Your checklist for this pull request
Detailed description
Building r2 with TCC is somewhat desirable. This PR aims to fix #4328
Test plan
Maybe add a github action with the following lines:
Closing issues
Closes #4328