Skip to content
This repository was archived by the owner on Aug 5, 2022. It is now read-only.

Conversation

@krocard
Copy link
Contributor

@krocard krocard commented Jun 9, 2015

The PFw as recently removed (this trick was not working on osx)
it's *_include libraries that were used to include headers
without linking against the libraries in the android build system.

There is to way to include the headers:

  • copy the headers (but that would be in contradiction with how the main pfw headers are included)
  • link against the library (chosen solution)

In order for the plugins to avoid linking against a static
libxmlserializer - ie add it and it's dependency to all plugins -
the xmlserializer library is now dynamic.

The alsa and filesystem plugins are also updated.

@krocard krocard force-pushed the 64_bit_support_port_from_android branch from eb8cd61 to 6f364e0 Compare June 9, 2015 18:45
@krocard krocard changed the title Make libxmserializer dynamic Android: Make libxmserializer dynamic Jun 9, 2015
The PFw as recently removed (this trick was not working on osx)
it's *_include libraries that were used to include headers
without linking against the libraries in the android build system.

There is to way to include the headers:
 - copy the headers (but that would be in contradiction with how the main pfw headers are included)
 - link against the library (chosen solution)

In order for the plugins to avoid linking against a static
libxmlserializer - ie add it and it's dependency to all plugins -
the xmlserializer library is now dynamic.

Change-Id: I79feb434c7b44ad4473d62276efc3c4b77928894
@krocard
Copy link
Contributor Author

krocard commented Jun 9, 2015

@OznOg @dawagner Please review this pull request and the 2 attached.

@krocard
Copy link
Contributor Author

krocard commented Jun 9, 2015

Wrong branch name, closing and reopening an new PR: #136

@krocard krocard closed this Jun 9, 2015
@krocard krocard deleted the 64_bit_support_port_from_android branch August 19, 2015 10:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant