-
Notifications
You must be signed in to change notification settings - Fork 355
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
Attempt to build all #191
base: master
Are you sure you want to change the base?
Attempt to build all #191
Conversation
00ee427
to
d155fe6
Compare
But to be honest, to build all is currently a miracle to me |
943e547
to
3d4963a
Compare
Oh so you are starting to work on these! I haven't touched these at all yet. So I'm curious with the structural changes, what IDE are you using? Something that makes working with bash, c, etc. all easier? I assumed you were using Android Studio up to this point, which is what led me to my final questions in that thread. So what do you need help on in here exactly? Are you running on MacOS? I have a machine here that I could use, but I'd likely just be relying on the runners unless there was some more rigorous testing needed for some reason. |
There is no change here at all
Android Studio, but you can use any Intellij IDE The main question is: Where are the missing files ? |
3d4963a
to
99cf25e
Compare
As you pushed this and all checks passed, I assume you figured this out? |
Look, everything it there |
Yep, looks great! Everything you needed to generate the JARs and upload to jitpack is included in this PR (i.e. only in tools/make-all)? Or were the changes to some build.gradle files that allowed you to generate the JARs in addition to the AARs? |
This |
99cf25e
to
a2e2dad
Compare
a2e2dad
to
e319752
Compare
e319752
to
ec1614f
Compare
Is there a reason this wasn't run on the jitpack 6.2.0? I'm only seeing the Android specific libs there. (i.e. no IOIOLibCore) |
That's a bug. The question is where ? |
Huh, ok. Thanks for clarifying. I didn't notice it built in the release page. I don't really need this immediately, so may not work on this for a while, but just wanted to note that the sample apps HelloIOIO and HelloIOIOService (among others I presume) depend on IOIOLibCore, so I'm not sure the current README with reference to |
I found the reason it's #251 |
I'm not seeing the link between this and jitpack. Did you see a build error that references this change? Can you link to the build error if so? Does jitpack depend on the |
With |
7c6d98a
to
83b10a5
Compare
83b10a5
to
861e72c
Compare
It looks like |
I tried to build all, but I miss files
prjMakefilesGenerator.*
Btw, I started to support macOS