Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

0.10.2 identifies itself as 0.10.1, then requests to be updated to 0.10.2 (!) #2119

Closed
thegreyshadow opened this issue Nov 25, 2021 · 13 comments
Labels
Bug It's a bug

Comments

@thegreyshadow
Copy link

Flameshot Version

Flameshot Version 0.10.2

Installation Type

Compiled from source

Operating System type and version

Slackware, 64-bit, 15.0 RC2 (Slackware-current)

Description

I downloaded and built (without problems) the 0.10.2 source tarball.
However, upon running flameshot I get an update prompt. Upon inspection of the systray icon, I get a notice: "New version 0.10.2 is available" and upon selecting the "About" menu entry, I am told that I'm running flameshot version 0.10.1.

Steps to reproduce

  1. Download the 0.10.2 source tarball
  2. build it
  3. install it
  4. run it
    and you will get the prompt

Screenshots or screen recordings

2021-11-25_17-34

Screenshot where it is seen that my source tarball is 0.10.2 while the app itself identifies as 0.10.1

System Information

1, OS: Slackware Linux 64-bit 15.0 RC2 (Slackware-current, development version)
2. Monitor: Laptop screen, 1920x1080, 60 Hz
3. Desktop environment: Plasma (5.23.3 using Plasma Frameworks 5.88.0). Window manager: kwin (5.23.3) using Wayland.

@thegreyshadow thegreyshadow added the Unconfirmed Bug The bug is not confirmed by anyone else. label Nov 25, 2021
@mmahmoudian
Copy link
Member

What is the sha256sum of the source code you downloaded? Can you provide the link?

I believe you mistakenly think that if you download the source code from GitHub master branch you have downloaded v0.10.2 . The v0.10.2 was a hot fix for wayland users and is not in-line with what has been committed to master. It is more or less a cherrypick of certain commits.

So in order for us to check your bug report, we would need the sha256sum of the source you downloaded and the link from which you download the source code.

@mmahmoudian mmahmoudian added the Waiting For Info Addressing the issue or merging the PR is halted and we are waiting for more info to be provided. label Nov 25, 2021
@farseerfc
Copy link
Contributor

farseerfc commented Nov 26, 2021

same here, the version number is not bumped for commit 2bf8a92 , which is tagged as v0.10.2 , and the appdata is also not updated. the branch v0.10.2 appearently has a commit c4081c6 that is not included in the tagged release.

951c8170b7be2f12e4446f6367e8fa176eb3489fe24c0cef8dfda0bd180e61f9 v0.10.2.tar.gz
downloaded from https://github.com/flameshot-org/flameshot/archive/v0.10.2.tar.gz , which is the same as https://github.com/flameshot-org/flameshot/archive/refs/tags/v0.10.2.tar.gz

@borgmanJeremy
Copy link
Contributor

Thanks for reporting. I'm going to pin this issue because I think it affects a few different packages.

I really really do not like having a maintenance release, but we needed to rush a fix out the door for gnome41. Our build pipeline are all geared towards releasing off the main branch and I clearly didn't patch them properly on the maintenance branch.

I know this is annoying but you can disable the popup. I'm not going to spend any time trying to fix this since a.) it's soul crushing to work on backports b.) it will be resolved when v11 is ready (TBD when that is).

tldr: I apologize that this is really annoying but I'm simply not motivated to spend time fixing it.

@farseerfc
Copy link
Contributor

Ok, understand. Switching to use the branch for the packaging for this release.

@thegreyshadow
Copy link
Author

thegreyshadow commented Nov 27, 2021

sha256sum of my downloaded source tarball:

951c8170b7be2f12e4446f6367e8fa176eb3489fe24c0cef8dfda0bd180e61f9 ./flameshot-0.10.2.tar.gz

The link from which I downloaded:

https://github.com/flameshot-org/flameshot/archive/refs/tags/v0.10.2.tar.gz

0.10.2 is shown by GitHub as the latest release, so I built from it.

@mmahmoudian mmahmoudian removed the Waiting For Info Addressing the issue or merging the PR is halted and we are waiting for more info to be provided. label Nov 28, 2021
@xvitaly
Copy link
Contributor

xvitaly commented Nov 28, 2021

I can confirm this issue on Fedora:

Flameshot v0.10.1
Compiled with Qt 5.15.2

Built from tag v0.10.2:

SHA512 (flameshot-0.10.2.tar.gz) = b0eb1decd87dab1d4a375708448dd4c6f85c101d33814b76edbb328150d0e9ee7a588be698fddcb24e70e39cd4331f9aa2ff082525ff6975d52303a6d9c1ab12

@xvitaly
Copy link
Contributor

xvitaly commented Nov 28, 2021

GNU/Linux downstreams can partially backport c4081c6 commit:

--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -1,7 +1,7 @@
 cmake_minimum_required(VERSION 3.13)
 # cmake_policy(SET CMP0076 OLD)
 
-set(FLAMESHOT_VERSION 0.10.1)
+set(FLAMESHOT_VERSION 0.10.2)
 
 # Flameshot-org
 set(GIT_API_URL "https://api.github.com/repos/flameshot-org/flameshot/releases/latest")
--- a/data/appdata/org.flameshot.Flameshot.metainfo.xml
+++ b/data/appdata/org.flameshot.Flameshot.metainfo.xml
@@ -9,6 +9,7 @@ SPDX-License-Identifier: CC0-1.0
   <project_license>GPL-3.0-or-later</project_license>
   <name>Flameshot</name>
   <releases> 
+    <release version="0.10.2" date="2021-11-13"/>
     <release version="0.10.1" date="2021-07-25"/>
     <release version="0.10.0" date="2021-05-16"/>
     <release version="0.9.0" date="2021-02-27"/>

After rebuilding with this patch the issue will be resolved.

@mmahmoudian mmahmoudian added Bug It's a bug and removed Unconfirmed Bug The bug is not confirmed by anyone else. labels Nov 29, 2021
@mmahmoudian
Copy link
Member

mmahmoudian commented Nov 29, 2021

@xvitaly thanks for the patch. But I think this is an unnecessary move at this point as the v0.10.2 was just a hotfix and releasing a new version just to fix the version info sounds like an overkill. Perhaps with your contribution in other issues we can soon release v0.11.0 and get back to normal situation. Btw, thanks for being active in various issues 👍🏼

@borgmanJeremy borgmanJeremy pinned this issue Dec 2, 2021
@AbdelrahmanMohamedDesoky

Is reporting false version info and trolling your users with auto update notification to update to 0.10.2 fun for almost a month?

This issue is not even reported/acknowledged yet on the release page (where you could at least tell users to ignore the auto update notifications) ?

I know you are all busy and have work and stuff, but these type of issues is what drive people crazy sometimes, especially people who don't have the experience to find the GitHub repo of the app and search the issues listing to find the stupid bug report and find it.

@mmahmoudian
Copy link
Member

mmahmoudian commented Dec 7, 2021

@AbdelrahmanMohamedDesoky thanks for your kind and considerate comment. The version 0.10.2 was just to fix some issues Gnome caused by changing their API. So this is NOT "false version" and is clearly NOT "trolling". This will take as much as it takes for us to get prepared for v0.11.0, so please be patient. If you are annoyed you can do either of the following:

  1. Press ignore on the notification
  2. Set the ignoreUpdateToVersion=0.10.2 in your Flameshot config file

None of the above would change your experience with Flameshot other than preventing the update notice messages from appearing.

This matter is acknowledged in multiple places and issues, but the release page is clearly not a place to mention such things.

Also I believe it is once again essential to point out the obvious:
we have pinned this issue (check the message above your comment) for two reasons:

  1. Be easy to find and the first thing someone sees
  2. To acknowledge that this is an important issue

image

Please feel free to keep suggesting enhancements and bugs [constructively] so that we can ultimately have a better Free and Libre OpenSource screenshot software :)

@razzeee
Copy link

razzeee commented Dec 24, 2021

The flatpak on flathub also says its 0.10.1

@mmahmoudian
Copy link
Member

This was addressed in the #2161 PR, but since the release of v0.11.0 is imminent, it was decided that we just move forward with the release rather than a point-fix. That said, big thank you to @johnrichardrinehart for following up the case and create the PR.

I'll close this issue as the new version with fix it.

@johnrichardrinehart
Copy link

Good job you guys!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug It's a bug
Projects
None yet
Development

No branches or pull requests

8 participants