axolotl-chat / axolotl

A Signal compatible cross plattform client written in Go, Rust and Vuejs
https://axolotl.chat
GNU General Public License v3.0
329 stars 52 forks source link

[BUG] flatpak web build process breaks #293

Open mororor opened 3 years ago

mororor commented 3 years ago

short verison:

please fix the links of the .png and .xml files in the flatpak web build manifest. the files are located in the ./flatpak/ directory not in the ./flatpak/web/ directory

thanks for your great work.

Description

After flatpak build process completed the install breaks with:

Running: install -Dm 644 src/github.com/nanu-c/axolotl/flatpak/web/org.nanuc.Axolotl.png ${FLATPAK_DEST}/share/icons/hicolor/128x128/apps/${FLATPAK_ID}.png
install: cannot stat 'src/github.com/nanu-c/axolotl/flatpak/web/org.nanuc.Axolotl.png': No such file or directory
Error: module axolotl-web: The childprocess quit with error code 1  // translated from german

Steps to Reproduce

sudo apt install flatpak flatpak-builder

flatpak install org.freedesktop.Platform//20.08
flatpak install org.freedesktop.Sdk//20.08
flatpak install org.freedesktop.Sdk.Extension.golang//20.08
flatpak install org.freedesktop.Sdk.Extension.node12//20.08

wget https://raw.githubusercontent.com/nanu-c/axolotl/main/flatpak/web/org.nanuc.Axolotl.yml
sudo flatpak-builder --install build org.nanuc.Axolotl.yml

Expected behavior: path to png should be correct Actual behavior: png is not found

Versions

0.9.0.1 with freshly merged pr #273

Device

pinephone mobian

Link to Debug Log

see above

joshbowyer commented 3 years ago

For whatever reason, building the flatpak from just the .yml file broke for me, but cloning the whole repo and then building from the .yml inside of the cloned repo worked perfectly. I dont know enough about flatpak to have any idea why that could be.

olof-nord commented 2 years ago

Two years later, but yes the Flatpak manifest references other files found in the repo such as libraries, metadata and icons. As such, just downloading the axolotl manifest is not enough.