fix(snap) : #1255 fix notification not included in snap binary (#1281)

* fix snap packaging * try fix source location * change source to dir * revert comments --------- Co-authored-by: Budhi <[email protected]>
hyperjumptech · May 7, 2024 · b4c1256 · b4c1256
1 parent 48b30df
commit b4c1256
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/.github/workflows/publish-snap.yml b/.github/workflows/publish-snap.yml
@@ -45,14 +45,14 @@ jobs:
         name: Build workspaces
       - run: npm ci
         name: Install dependencies
-      - run: sed -i 's#packages/notification/hyperjumptech-monika-notification#../../packages/notification/hyperjumptech-monika-notification#g' package.json
-        name: Change monika notification package path
       - run: npm run pack-tarballs
         name: Pack as tarball
-      - run: mv dist/monika*.tar.gz .
-        name: Move the tarball from dist folder to root
-      - run: mv monika*.tar.gz monika.tar.gz
-        name: Rename tarball
+      - run: mv dist/monika*.tar.gz dist/monika.tar.gz
+        name: Rename tarball in dist/ directory
+      - run: tar -zvxf dist/monika.tar.gz -C dist/
+        name: Extract tarball into dist/ directory
+      - run: cp -vr packages/notification/lib/* dist/monika/packages/notification/
+        name: Copy built notification into dist/ directory
       - uses: snapcore/action-build@v1
         id: snapcraft
         name: Build Snap package

diff --git a/snapcraft.yaml b/snapcraft.yaml
@@ -25,7 +25,7 @@ apps:
 parts:
   monika:
     plugin: dump
-    source: ./monika.tar.gz
+    source: ./dist/monika
     override-pull: |
       sudo apt-get install -y jq curl
       CLI_VERSION=$(curl https://api.github.com/repos/hyperjumptech/monika/releases/latest -s | jq .tag_name -r)