megasync: cleanup

Author: GaetanLepage

pkgs/by-name/me/megasync/package.nix

@@ -1,40 +1,45 @@
 {
   lib,
   stdenv,
-  c-ares,
+  fetchFromGitHub,
+  fetchpatch,
+
+  # nativeBuildInputs
   cmake,
+  libsForQt5,
+  libtool,
+  pkg-config,
+  unzip,
+
+  # buildInputs
+  c-ares,
   cryptopp,
   curl,
-  fetchFromGitHub,
-  fetchpatch,
   ffmpeg,
   hicolor-icon-theme,
   icu,
   libmediainfo,
   libsodium,
-  libtool,
   libuv,
   libxcb,
   libzen,
   openssl,
-  pkg-config,
-  libsForQt5,
   readline,
   sqlite,
-  unzip,
   wget,
   xorg,
   zlib,
-  qt5,
+
+  nix-update-script,
 }:
-stdenv.mkDerivation rec {
+stdenv.mkDerivation (finalAttrs: {
   pname = "megasync";
   version = "5.8.0.2";
 
   src = fetchFromGitHub rec {
     owner = "meganz";
     repo = "MEGAsync";
-    rev = "v${version}_Linux";
+    tag = "v${finalAttrs.version}_Linux";
     hash = "sha256-/q7LD1/06+0MepDz3fVrlvGKh+rvNk6d1hm7Ng54Nmk=";
     fetchSubmodules = false; # DesignTokensImporter cannot be fetched, see #1010 in github:meganz/megasync
     leaveDotGit = true;
@@ -44,21 +49,46 @@ stdenv.mkDerivation rec {
       git remote add origin $url
       git fetch origin
       git clean -fdx
-      git checkout ${rev}
+      git checkout ${tag}
       git submodule update --init src/MEGASync/mega
 
       rm -rf .git
     ''; # Why so complicated, I just want a single submodule
   };
 
+  patches = [
+    (fetchpatch {
+      url = "https://aur.archlinux.org/cgit/aur.git/plain/020-megasync-sdk-fix-cmake-dependencies-detection.patch?h=megasync&id=ff59780039697591e7e3a966db058b23bee0451c";
+      hash = "sha256-hQY6tMwiV3B6M6WiFdOESdhahAtuWjdoj2eI2mst/K8=";
+      extraPrefix = "src/MEGASync/mega/";
+      stripLen = true;
+    })
+    (fetchpatch {
+      url = "https://aur.archlinux.org/cgit/aur.git/plain/030-megasync-app-fix-cmake-dependencies-detection.patch?h=megasync&id=ff59780039697591e7e3a966db058b23bee0451c";
+      hash = "sha256-11XWctv1veUEguc9Xvz2hMYw26CaCwu6M4hyA+5r81U=";
+    })
+    ./megasync-fix-cmake-install-bindir.patch
+    ./dont-fetch-clang-format.patch
+  ];
+
+  postPatch = ''
+    substituteInPlace cmake/modules/desktopapp_options.cmake \
+      --replace-fail "ENABLE_ISOLATED_GFX ON" "ENABLE_ISOLATED_GFX OFF"
+
+    for file in $(find src/ -type f \( -iname configure -o -iname \*.sh \) ); do
+      substituteInPlace "$file" --replace-warn "/bin/bash" "${stdenv.shell}"
+    done
+  '';
+
   nativeBuildInputs = [
     cmake
+    libsForQt5.qttools
+    libsForQt5.wrapQtAppsHook
     libtool
     pkg-config
-    libsForQt5.qttools
     unzip
-    libsForQt5.wrapQtAppsHook
   ];
+
   buildInputs = [
     c-ares
     cryptopp
@@ -67,46 +97,24 @@ stdenv.mkDerivation rec {
     hicolor-icon-theme
     icu
     libmediainfo
-    libsodium
-    libuv
-    libxcb
-    libzen
-    openssl
     libsForQt5.qtbase
     libsForQt5.qtdeclarative
     libsForQt5.qtgraphicaleffects
     libsForQt5.qtquickcontrols
     libsForQt5.qtquickcontrols2
     libsForQt5.qtsvg
     libsForQt5.qtx11extras
+    libsodium
+    libuv
+    libxcb
+    libzen
+    openssl
     readline
     sqlite
     wget
     zlib
   ];
 
-  patches = [
-    (fetchpatch {
-      url = "https://aur.archlinux.org/cgit/aur.git/plain/020-megasync-sdk-fix-cmake-dependencies-detection.patch?h=megasync&id=ff59780039697591e7e3a966db058b23bee0451c";
-      hash = "sha256-hQY6tMwiV3B6M6WiFdOESdhahAtuWjdoj2eI2mst/K8=";
-      extraPrefix = "src/MEGASync/mega/";
-      stripLen = true;
-    })
-    (fetchpatch {
-      url = "https://aur.archlinux.org/cgit/aur.git/plain/030-megasync-app-fix-cmake-dependencies-detection.patch?h=megasync&id=ff59780039697591e7e3a966db058b23bee0451c";
-      hash = "sha256-11XWctv1veUEguc9Xvz2hMYw26CaCwu6M4hyA+5r81U=";
-    })
-    ./megasync-fix-cmake-install-bindir.patch
-    ./dont-fetch-clang-format.patch
-  ];
-
-  postPatch = ''
-    substituteInPlace cmake/modules/desktopapp_options.cmake --replace-fail "ENABLE_ISOLATED_GFX ON" "ENABLE_ISOLATED_GFX OFF"
-    for file in $(find src/ -type f \( -iname configure -o -iname \*.sh \) ); do
-      substituteInPlace "$file" --replace "/bin/bash" "${stdenv.shell}"
-    done
-  '';
-
   dontUseQmakeConfigure = true;
   enableParallelBuilding = true;
 
@@ -120,14 +128,25 @@ stdenv.mkDerivation rec {
     qtWrapperArgs+=(--prefix PATH : ${lib.makeBinPath [ xorg.xrdb ]})
   '';
 
-  meta = with lib; {
+  passthru = {
+    updateScript = nix-update-script {
+      extraArgs = [
+        "--version-regex=^v(.*)_Linux$"
+      ];
+    };
+  };
+
+  meta = {
     description = "Easy automated syncing between your computers and your MEGA Cloud Drive";
     homepage = "https://mega.nz/";
-    license = licenses.unfree;
+    downloadPage = "https://github.com/meganz/MEGAsync";
+    changelog = "https://github.com/meganz/MEGAsync/releases/tag/v${finalAttrs.version}_Linux";
+    license = lib.licenses.unfree;
     platforms = [
       "i686-linux"
       "x86_64-linux"
     ];
     maintainers = [ ];
+    mainProgram = "megasync";
   };
-}
+})