Unify start.c of all bootstraps to one file#1500
Unify start.c of all bootstraps to one file#1500AndreMiras merged 1 commit intomasterfrom unknown repository
Conversation
|
Okay, so I tested the |
|
At a quick glance, looks very solid so far. I've been meaning for a while to make the start.c code less awful, do you have any intention to try that? It might be an opportunity to avoid some of the ifdef mess we currently have. However, it isn't essential, just merging the files is a great improvement. |
Eventually, but so far I'm still mostly busy with fixing architectural issues to get my app back to building from p4a master. This involves rescheduling my I am interested in also cleaning up the code itself some more, I just haven't really gotten around yet. But this pull request should be a splendid foundation for anyone to do that in the near future - and if you want to jump into it first, by all means go for it (but I would suggest it's best done in a separate followup request, because this one has high conflict potential and should go in soon) |
|
Right, since the only thing remaining is more testing, I'll remove the |
|
(latest force push is rebase onto master with gradle issue fixed, no change in this pull request) Testing progress:
|
inclement
left a comment
There was a problem hiding this comment.
Looks good to me, I don't see any real issues.
Since the webview and service_only bootstraps are broken anyway, no need to worry about them for now.
I did a quick build test with an app I was working on, without any issues. I'll try a bit more than that just to check, but other than that it looks good to go as far as I'm concerned.
pythonforandroid/bootstraps/common/build/jni/application/src/start.c
Outdated
Show resolved
Hide resolved
|
I did another Python 2 sdl 2 bootstrap testbuild with the force-pushed version that fixes the |
|
Also looks good to me, I'll give it a run-time try with |
AndreMiras
left a comment
There was a problem hiding this comment.
Tested on host python3, target python3crystax with both sdl2 and service_only bootstraps.
Everything was OK.
The git stats looks great 😄 well done:
git diff --shortstat upstream/master
9 files changed, 92 insertions(+), 856 deletions(-)
I have started reviewing the
start.cfiles of all bootstraps for changes, and preparing a unification process to remove the differentstart.cfiles. This pull request is the result of this.WIP because I still need to test this myself.