0.84b4: Updater logic - version compare logic typo

Reported bugs that have been closed and/or resolved

Moderators: XnTriq, helmut, xnview, Dreamer

Post Reply
VictorVG
Posts: 38
Joined: Mon Aug 24, 2009 10:54 pm

0.84b4: Updater logic - version compare logic typo

Post by VictorVG »

Step fo respond: just unzip and start v0.84 b4
Visual effect: update logic used information on VERSION_INFO, but this structure included old version 0.82.0.0 user can see screenshot:
1.png
1.png (36.04 KiB) Viewed 1051 times
I open section Resource on XnViewMP.exe use Far Manager v3.0 + plug-in ImpEx as "Resources\VERSION\00001.0x040C.rc" and see this structure as .RC file:

Code: Select all

#include <windows.h>

VS_VERSION_INFO VERSIONINFO
 FILEVERSION    0,82,0,0
 PRODUCTVERSION 0,82,0,0
 FILEFLAGSMASK  0x17L    // VS_FF_DEBUG|VS_FF_INFOINFERRED|VS_FF_PATCHED|VS_FF_PRERELEASE
 FILEFLAGS      0x0L
 FILEOS         0x4L     // VOS__WINDOWS32
 FILETYPE       0x1L     // VFT_APP
 FILESUBTYPE    0x0L
BEGIN
    BLOCK "StringFileInfo"
    BEGIN
        BLOCK "040c04b0"
        BEGIN
            VALUE "CompanyName",       "XnView, http://www.xnview.com\0"
            VALUE "FileDescription",   "XnViewMP\0"
            // Zero-length item found
            VALUE "FileVersion",       "0.82.0.0\0"
            // Zero-length item found
            VALUE "InternalName",      "XnViewMP\0"
            // Zero-length item found
            VALUE "LegalCopyright",    "Copyright © 2008-2016 XnView\0"
            // Zero-length item found
            VALUE "OriginalFilename",  "xnviewmp.exe\0"
            // Zero-length item found
            VALUE "ProductName",       "XnViewMP Application\0"
            // Zero-length item found
            VALUE "ProductVersion",    "0.82.0.0\0"
        END
    END
    BLOCK "VarFileInfo"
    BEGIN
        VALUE "Translation", 0x40C, 1200
    END
END
I think there is build scripts so that they improve AUTOMATIC change the value of the build number for the exclusion of such amusing incidents.
1.png
1.png (36.04 KiB) Viewed 1051 times
Xeon E5 2697v2/C602/128 GB PC3-14900L Quad Channel/GTX 1660 Ti/9 Tb HDD
Xeon E5-2697v2/C602J/128 Gb PC3-14900L Quad Channel/GTX 1660 Ti/7Tb HDD
User avatar
helmut
Posts: 8705
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: Updater logic - version compare logic typo

Post by helmut »

Thanks for reporting, VictorVG. This bug has been observed and reported by various testers, already.

@Pierre:
Would be good if this wrong version comparison was fixed, now. As you can see, this causes confusion over and over, again.
VictorVG
Posts: 38
Joined: Mon Aug 24, 2009 10:54 pm

Re: 0.84b4: Updater logic - version compare logic typo

Post by VictorVG »

Earlier, I am having a similar problem, and it's where I rehil introduction of checks on SVN revision number through quite a simple condition:

If SavedSvnRev <= CurrenSvnRev Then Update; Else Skip;

but for the operation of the mechanism I had to keep the car on the SVN revision number of the user in a separate registry subkey. For check SVN revision I use simple NSIS code...
Xeon E5 2697v2/C602/128 GB PC3-14900L Quad Channel/GTX 1660 Ti/9 Tb HDD
Xeon E5-2697v2/C602J/128 Gb PC3-14900L Quad Channel/GTX 1660 Ti/7Tb HDD
User avatar
xnview
Author of XnView
Posts: 43442
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Re: 0.84b4: Updater logic - version compare logic typo

Post by xnview »

O.k., thank you, I can also reproduce the problem. Issue 1134 is fixed in next version.
Pierre.
User avatar
xnview
Author of XnView
Posts: 43442
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Re: 0.84b4: Updater logic - version compare logic typo

Post by xnview »

This problem is supposed to be fixed in XnView MP 0.85 beta 1. Please check and confirm the bug fix here.
Pierre.
User avatar
helmut
Posts: 8705
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: 0.84b4: Updater logic - version compare logic typo

Post by helmut »

When starting the new 0.85 beta 1 it doesn't display the message box telling that a new version is available.

A minor issue:
Though, when manually checking for an update using Help » Check For update nothing happens. Expected: A message box "You're running the latest version of XnView MP" should be displayed.
@Pierre: Let me know in case you want a new bug report for this remaining problem.
User avatar
xnview
Author of XnView
Posts: 43442
Joined: Mon Oct 13, 2003 7:31 am
Location: France
Contact:

Re: 0.84b4: Updater logic - version compare logic typo

Post by xnview »

This problem is supposed to be fixed in XnView MP 0.85 beta 2. Please check and confirm the bug fix here.
Pierre.
User avatar
helmut
Posts: 8705
Joined: Sun Oct 12, 2003 6:47 pm
Location: Frankfurt, Germany

Re: 0.84b4: Updater logic - version compare logic typo

Post by helmut »

Confirmed fixed in 0.85 beta 2 (Message box says "You're running the latest version of XnView MP.")
Post Reply