Skip to content
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

Android adaptations #68

Merged
merged 33 commits into from
Dec 5, 2011
Merged

Android adaptations #68

merged 33 commits into from
Dec 5, 2011

Conversation

mbernasocchi
Copy link
Member

Hi Guys, this adaptations are the ones needed for crosscompiling for android. I tryed to keep everything in ifdefs.
the files I feel you should look at more "carefully" are:
src/app/main.cpp
src/app/qgspluginregistry.cpp
src/core/qgsproviderregistry.cpp
src/core/symbology-ng/qgsrendererv2.cpp
src/providers/ogr/qgsogrprovider.cpp

thanks a lot
Marco

mbernasocchi and others added 30 commits July 25, 2011 14:45
…s, writing only if there are no string type attributes.
- plugins are loaded only if called *plugin.so
- providers are loaded only if called *provider.so
IF (ANDROID)
ADD_LIBRARY(${QGIS_APP_NAME} SHARED ${QGIS_APP_SRCS} ${QGIS_APP_MOC_SRCS} ${INDEX_SRC} ${QGIS_APP_HDRS} ${QGIS_APP_MOC_HDRS} ${IMAGE_RCC_SRCS})
ELSE (ANDROID)
ADD_EXECUTABLE(${QGIS_APP_NAME} MACOSX_BUNDLE WIN32 ${QGIS_APP_SRCS} ${QGIS_APP_MOC_SRCS})
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about IMAGE_RCC_SRCS?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

who needs fancy icons when you have a mouse :) fixing it

@timlinux
Copy link
Member

timlinux commented Dec 2, 2011

Hi Marco

I had a look through your patch. Just two minor comments. If you address them I will apply it.

Thanks!

Regards

Tim

@mbernasocchi
Copy link
Member Author

On 12/02/2011 07:05 AM, Tim Sutton wrote:

Hi Marco

I had a look through your patch. Just two minor comments. If you address them I will apply it.

Thanks!

Regards

Tim


Reply to this email directly or view it on GitHub:
#68 (comment)
Thank you, fixed
:)
PS I added a file to the gitignore file. i did a separate commit in case
you dont like it nd wnt o cherrypick

ciao

Marco Bernasocchi
www.opengis.ch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants