antony-jr / AppImageUpdater

AppImage Updater for Humans built with QML/C++ with Qt5 :heart:.
GNU Lesser General Public License v3.0
40 stars 6 forks source link

Give clear message if GitHub Release does not have an AppImage #6

Closed probonopd closed 6 years ago

probonopd commented 6 years ago

The updater should give a clear message if GitHub Release does not have an AppImage

Test file: OldImageMagick-git.7a48608-x86_64.AppImage.zip

For comparison, AppImageUpdate says:

me@host:~$ /isodevice/Applications/AppImageUpdate-185-1c72f3b-x86_64.AppImage OldImageMagick-git.7a48608-x86_64.AppImage 
AppImageUpdate version 1-alpha (commit 1c72f3b), build 185 built on 2018-01-03 16:41:48 UTC
Checking for updates...
Fetching release information for tag "continuous" from GitHub API.
GitHub API request failed!
Could not find any artifacts in release data. Please contact the author of the AppImage and tell them the files are missing on the releases page.
ZSync URL not available. See previous messages for details.
antony-jr commented 6 years ago

got it! 👍

antony-jr commented 6 years ago

this is not enough ?

QAIUpdateInformation::  "gh-releases-zsync|probonopd|ImageMagick|continuous|ImageMagick*-x86_64.AppImage.zsync"
AIUpdaterBridge:: github release link :: QUrl("https://api.github.com/repos/probonopd/ImageMagick/releases/tags/continuous")
AIUpdaterBridge:: network error ::  QNetworkReply::NetworkError(ContentNotFoundError)
antony-jr commented 6 years ago

@probonopd why does the title bar icon is not showing up ? The icon only shows up if I run it outside of the AppImage.

antony-jr commented 6 years ago

Try the new version , The Updater should update itself by now ?

probonopd commented 6 years ago

this is not enough ?

QAIUpdateInformation
QAIUpdateInformation::  "gh-releases-zsync|probonopd|ImageMagick|continuous|ImageMagick*-x86_64.AppImage.zsync"
AIUpdaterBridge:: github release link :: QUrl("https://api.github.com/repos/probonopd/ImageMagick/releases/tags/continuous")
AIUpdaterBridge:: network error ::  QNetworkReply::NetworkError(ContentNotFoundError)

This looks good but I can't remember to have seen this when I tested originally.

probonopd commented 6 years ago

@probonopd why does the title bar icon is not showing up ? The icon only shows up if I run it outside of the AppImage.

Good question, as per https://discourse.appimage.org/t/appimage-does-not-show-icon-in-task-bar/40/1 I would think that

https://github.com/antony-jr/AppImageUpdater/blob/7690ab45ae06a2ac7585ec8e9ab6fe6315f5220a/AppImageUpdater.hpp#L84

should do it.

Reference: https://github.com/JvanKatwijk/qt-dab/issues/69

probonopd commented 6 years ago

Try the new version , The Updater should update itself by now ?

Indeed, seems to work :+1:

probonopd commented 6 years ago

As to the original topic of this issue, the text in the window still says "unknown network interruption" when in fact GitHub does not have an AppImage available.

antony-jr commented 6 years ago

okay then we might need more error codes to be generated from AppImageUpdaterBridge , check the available error codes here , https://antony-jr.github.io/AppImageUpdaterBridge/docs/AppImageUpdaterBridgeErrorCodes.html

antony-jr commented 6 years ago

@probonopd This issue is irrelevant now since all errors are emitted explicitly now , I just need some time to convert these error codes to human readable Message box which is a pain in the neck but I will finish it soon , The logging is good now , Try the new version!