pin flutter version

.github/workflows/ios.yml

@@ -19,7 +19,7 @@ jobs:
             - name: Set up Flutter
               uses: subosito/flutter-action@v2
               with:
-                  flutter-version: 3.47.4
+                  flutter-version-file: pubspec.yaml
 
             - name: Set up Go
               uses: actions/setup-go@v6

.github/workflows/macos.yml

@@ -19,7 +19,7 @@ jobs:
             - name: Set up Flutter
               uses: subosito/flutter-action@v2
               with:
-                  flutter-version: 3.47.4
+                  flutter-version-file: pubspec.yaml
 
             - name: Set up Go
               uses: actions/setup-go@v6

.github/workflows/windows.yml

@@ -19,7 +19,7 @@ jobs:
             - name: Set up Flutter
               uses: subosito/flutter-action@v2
               with:
-                  flutter-version: 3.47.4
+                  flutter-version-file: pubspec.yaml
 
             - name: Set up Go
               uses: actions/setup-go@v6

android/app/build.gradle.kts

@@ -14,13 +14,6 @@ if (keystorePropertiesFile.exists()) {
     }
 }
 
-fun versionCodeOf(versionName: String): Int {
-    val parts = versionName.split("-", limit = 2)
-    val (major, minor, patch) = parts[0].split(".").map(String::toInt)
-    val preRelease = parts.getOrNull(1)?.let { Regex("\\d+").find(it)?.value?.toInt() ?: 0 } ?: 999
-    return major * 100_000_000 + minor * 1_000_000 + patch * 10_000 + preRelease
-}
-
 android {
     namespace = "nexus.federated.nexus"
     compileSdk = 37
@@ -38,7 +31,7 @@ android {
 		minSdk = 29
         targetSdk = flutter.targetSdkVersion
 
-        versionCode = versionCodeOf(flutter.versionName!!)
+        versionCode = flutter.versionCode
         versionName = flutter.versionName
 
 		multiDexEnabled = true

hook/build.dart

@@ -64,10 +64,14 @@ Future<void> main(List<String> args) => build(args, (input, output) async {
 
       final targetNdkApi = codeConfig.android.targetNdkApi;
 
+      // Treats empty variables as unset, since e.g. F-Droid sets ANDROID_NDK_HOME to "" without an NDK
+      String? env(String name) => switch (Platform.environment[name]) {
+        "" => null,
+        final value => value,
+      };
+
       Future<String?> findNdkFromSdk() async {
-        final androidHome =
+        final androidHome = env("ANDROID_HOME") ?? env("ANDROID_SDK_ROOT");
-            Platform.environment["ANDROID_HOME"] ??
-            Platform.environment["ANDROID_SDK_ROOT"];
         if (androidHome == null) return null;
 
         final ndkDir = Directory(join(androidHome, "ndk"));
@@ -78,9 +82,9 @@ Future<void> main(List<String> args) => build(args, (input, output) async {
       }
 
       final ndkHome =
-          Platform.environment["ANDROID_NDK_HOME"] ??
+          env("ANDROID_NDK_HOME") ??
-          Platform.environment["ANDROID_NDK_ROOT"] ??
+          env("ANDROID_NDK_ROOT") ??
-          Platform.environment["NDK_HOME"] ??
+          env("NDK_HOME") ??
           await findNdkFromSdk();
 
       if (ndkHome == null) {
@@ -91,8 +95,7 @@ Future<void> main(List<String> args) => build(args, (input, output) async {
 
       // TODO: Someone please give me a way to detect host architecture so I can change this
       final hostTag =
-          Platform.environment["ANDROID_HOST_TAG"] ??
+          env("ANDROID_HOST_TAG") ?? "${Platform.operatingSystem}-x86_64";
-          "${Platform.operatingSystem}-x86_64";
       final ccTriple = switch (targetArch) {
         Architecture.arm64 => "aarch64-linux-android",
         Architecture.arm => "armv7a-linux-androideabi",

lib/controllers/room_chat.dart

@@ -295,6 +295,7 @@ class RoomChatController(final ChatParams info)
                 : null,
           ),
         );
+        if (!ref.mounted) return;
 
         if (response.events.isEmpty) {
           state = .data(chat);
@@ -333,6 +334,7 @@ class RoomChatController(final ChatParams info)
                 : chat!.historicalData!.start,
           ),
         );
+        if (!ref.mounted) return;
 
         ref
             .read(RoomsController.provider.notifier)
@@ -345,6 +347,7 @@ class RoomChatController(final ChatParams info)
         final rootRowId = direction == .backward
             ? await threadRootBefore(paginationResponse)
             : null;
+        if (!ref.mounted) return;
         final eventRowIds = [
           ?rootRowId,
           ...(direction == .forward
@@ -383,6 +386,7 @@ class RoomChatController(final ChatParams info)
         );
       }
     } catch (_) {
+      if (!ref.mounted) return;
       state = .data(chat);
       rethrow;
     }

linux/nix/build_env.nix

@@ -5,6 +5,8 @@
     with pkgs;
     [
       stdenv.cc
+      # webcrypto's build hook runs CMake with the Unix Makefiles generator
+      gnumake
       go
       git
       ffmpeg-headless
@@ -20,13 +22,22 @@
     ]
     ++ lib.optionals pkgs.stdenv.isLinux [
       wayland
-      # nativeapi includes XInput2.h, which needs these, but xi.pc only lists them privately
-      libxext
-      libxfixes
       (callPackage ./pkg/hook_pkg_config.nix { })
     ];
 
   env = lib.optionalAttrs pkgs.stdenv.isLinux {
+    # nativeapi includes XInput2.h and GTK's Wayland headers, which need these, but xi.pc and gdk-3.0.pc
+    # only list them privately, so its hook's pkg-config doesn't return them. Hooks only get NIX_ variables.
+    "NIX_CFLAGS_COMPILE_${pkgs.stdenv.cc.suffixSalt}" =
+      lib.concatMapStringsSep " " (pkg: "-isystem ${lib.getDev pkg}/include")
+        (
+          with pkgs;
+          [
+            libxext
+            libxfixes
+            wayland
+          ]
+        );
     # better_player loads libmpv at runtime to play videos
     LD_LIBRARY_PATH = "./build/native_assets/linux:${
       lib.makeLibraryPath [

linux/nix/default.nix

@@ -65,6 +65,9 @@
       apps =
         let
           buildEnv = import ./build_env.nix { inherit pkgs lib; };
+          testEnv = buildEnv.env // {
+            DBUS_SESSION_CONFIG = "${pkgs.dbus}/share/dbus-1/session.conf";
+          };
           e2e = pkgs.writeShellApplication {
             name = "nexus-e2e";
             runtimeInputs =
@@ -75,7 +78,7 @@
                 curl
                 dbus
               ]);
-            runtimeEnv = buildEnv.env;
+            runtimeEnv = testEnv;
             text = builtins.readFile ./e2e.sh;
           };
           screenshots = pkgs.writeShellApplication {
@@ -87,7 +90,7 @@
                 oxipng
                 pngquant
               ]);
-            runtimeEnv = buildEnv.env;
+            runtimeEnv = testEnv;
             text = builtins.readFile ./screenshots.sh;
           };
 

linux/nix/e2e.sh

@@ -69,7 +69,7 @@ for file in "${files[@]}"; do
     name="$(basename "$file" .dart)"
     mkdir -p "$work/$name/data" "$work/$name/cache"
     XDG_DATA_HOME="$work/$name/data" XDG_CACHE_HOME="$work/$name/cache" \
-        dbus-run-session -- \
+        dbus-run-session --config-file="$DBUS_SESSION_CONFIG" -- \
         flutter test "$file" -d linux \
         --dart-define="NEXUS_E2E_HOMESERVER=$homeserver" "${flutter_args[@]}" ||
         status=$?

linux/nix/screenshots.sh

@@ -12,7 +12,7 @@ GOCACHE="$(go env GOCACHE)"
 export GOCACHE
 
 XDG_DATA_HOME="$work/data" XDG_CACHE_HOME="$work/cache" \
-    dbus-run-session -- \
+    dbus-run-session --config-file="$DBUS_SESSION_CONFIG" -- \
     flutter test integration_test/screenshots/screenshots_test.dart -d linux \
     --dart-define="SCREENSHOTS_DIR=$work/out" "$@"
 

pubspec.lock

@@ -1960,4 +1960,4 @@ packages:
     version: "2.2.4"
 sdks:
   dart: ">=3.13.3 <4.0.0"
-  flutter: ">=3.47.0"
+  flutter: "3.47.4"

pubspec.yaml

@@ -1,6 +1,6 @@
 name: nexus
 description: "A simple and user-friendly Matrix client"
-version: 1.1.1
+version: 1.1.1+101010999
 publish_to: none
 
 flutter:
@@ -16,7 +16,8 @@ flutter:
     uses-material-design: true
 
 environment:
-    sdk: ^3.13.3
+    sdk: 3.13.3
+    flutter: 3.47.4
 
 dependency_overrides:
     dynamic_color: 2.1.0