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

RDContainer.mm/h are attached which are detached from the Readium SDK project. #66

Closed

Conversation

drminside
Copy link

Dear Readium SDK people.

I send a pull request for update SDKLauncher-ios.
This is to include unchanged RDContainer.mm/h files which are detached from the Readium SDK project.

The reason why those files are attached SDKLauncher-iOS rather than Readium SDK is:

  1. Since RDContainer.mm needs to be modified to include LCP related header file and initialization code int the future, it is better to be in the SDKLauncher-iOS side in order to make Readium-SDK a DRM agnostic library.
  2. In Launcher-OSX case, the initialization process including ePub3::InitializeSdk() is in the LauncherOSX/LOXePubSdkApi.mm file, not in the Readium SDK, which is pretty natural.

The merge should be done together with disposal of the pull request of the Readium SDK.
Because only one disposal of either SDKLauncher-iOS or Readium SDK could make duplication or missing files problem.

And for the consistency of the Readium codes, in the future, we recommend all Launcher related files should locate at Launcher project instead of Readium SDK like Launcher-OSX case.

@danielweck
Copy link
Member

Superseded: readium/readium-sdk#216

@danielweck danielweck closed this Nov 18, 2015
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.

3 participants