Skip to content

Commit

Permalink
Disable LTO on Android (#94)
Browse files Browse the repository at this point in the history
  • Loading branch information
jonasfj authored Mar 6, 2024
1 parent 60cf08f commit de8dbd9
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
# 0.5.6
* Disable LTO on Android to fix [#80](https://github.com/google/webcrypto.dart/issues/80).
* Migrate to `dart:js_interop` to ensure [wasm compatibility](https://dart.dev/interop/js-interop/package-web#package-web-vs-dart-html).

# 0.5.5
* Adds a namespace in `build.gradle` for compatibility with AGP 8.0.

Expand Down
7 changes: 4 additions & 3 deletions android/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,10 @@ include(
# https://boringssl.googlesource.com/boringssl/+/HEAD/BUILDING.md#binary-size
add_definitions(-DOPENSSL_SMALL)

# Enable link time optimization
set(CMAKE_C_FLAGS "${CMAKE_CXX_FLAGS} -flto")
set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -flto")
# TODO: Enable link time optimization
# Requires fixing https://github.com/google/webcrypto.dart/issues/80
# set(CMAKE_C_FLAGS "${CMAKE_CXX_FLAGS} -flto")
# set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -flto")

# Android ABIs, see also:
# https://developer.android.com/ndk/guides/abis
Expand Down
2 changes: 1 addition & 1 deletion pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# limitations under the License.

name: webcrypto
version: 0.5.5
version: 0.5.6
description: Cross-platform implementation of Web Cryptography APIs for Flutter.
repository: https://github.com/google/webcrypto.dart

Expand Down

0 comments on commit de8dbd9

Please sign in to comment.