Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Reorganize Android build process #53

Open
kruton opened this Issue Apr 20, 2013 · 3 comments

Comments

Projects
None yet
2 participants

kruton commented Apr 20, 2013

It would be nice to organize the Android build of Avian so it can plug directly into a checked-out Android tree. This makes it convenient for users with a checked-out Android tree, but is should also make it easier for people that don't have an Android tree.

It can use a smaller repo manifest (e.g., "repo init -u http://...") to avoid the long setup instructions.

Member

dicej commented Apr 21, 2013

Yes, that would be nice. What would need to be changed and/or added?

kruton commented Apr 21, 2013

The default layout of the projects would help (external/fdlibm instead of just fdlibm, system/core instead of core).

An example of a pared-down repo manifest can be seen at https://android.googlesource.com/accessories/manifest/+/master/default.xml

Also pulling the build products from $(android)/out/host/{linux,darwin}-x86/obj/... would be totally awesome, but I'd need to add something to get x86-64 host artifacts build products.

Member

dicej commented Apr 22, 2013

I've updated the directory layout to start: 6400231

@dparnell dparnell pushed a commit to dparnell/avian that referenced this issue Dec 24, 2013

@dicej dicej use Android directory layout when accessing class library code
Per ReadyTalk#53, Avian should build
against a standard AOSP checkout, which means we should look for
subprojects in the directories the repo utility would place them.
6400231

@dicej dicej added a commit to dicej/avian that referenced this issue Apr 21, 2014

@dicej dicej use Android directory layout when accessing class library code
Per ReadyTalk#53, Avian should build
against a standard AOSP checkout, which means we should look for
subprojects in the directories the repo utility would place them.
ce1816b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment