NSIS版本号比较-VersionCompare Bug

问题：

今天发现自己之前写的NSIS脚本中有个BUG，BUG的来源正是NSIS的版本号比较函数：VersionCompare

这个函数在比较"2008.3.15.1"和"2008.05.1.10"返回的不是期待的2，却是1

于是，可怕的事情发生了……

 

 

解决：

同类的几个函数，经过一番测试，采用下面这个函数。

http://nsis.sourceforge.net/Comparing_Two_Version_Numbers

 

不过原来有个小bug，就是没有初始化自己的变量，会导致多次调用时的结果出错。

于是我也修改之后直接在wiki上提交了我的修改，并在这里转载这个函数，以便下次使用。

 

1. ;  From:http://nsis.sourceforge.net/Comparing_Two_Version_Numbers
2. ;  Created 1/5/05 by Comperio
3. ;
4. ;   Usage:
5. ;   ${VersionCheck} "Version1" "Version2" "outputVar"
6. ;       Version1 = 1st version number
7. ;       Version2 = 2nd version number
8. ;       outputVar = Variable used to store the ouput
9. ;
10. ;   Return values:
11. ;       0 = both versions are equal
12. ;       1 = Version 1 is NEWER than version 2
13. ;       2 = Version1 is OLDER than version 2
14. ;   Rules for Version Numbers:
15. ;       Version numbers must always be a string of numbers only.  (A dot
16. ;       in the name is optional). 
17. ;
18. ;   [Variables]
19. var P1  ; file pointer, used to "remember" the position in the Version1 string
20. var P2  ; file pointer, used to "remember" the position in the Version2 string
21. var V1  ;version number from Version1
22. var V2  ;version number from Version2
23. Var Reslt   ; holds the return flag
26. ;   [Macros]
27. !macro VersionCheck Ver1 Ver2 OutVar
28.     ;   To make this work, one character must be added to the version string:
29.     Push "x${Ver2}"
30.     Push "x${Ver1}"
31.     Call VersionCheckF
32.     Pop ${OutVar}
34. !macroend
36. ;   [Defines]
37. !define VersionCheck "!insertmacro VersionCheck"
39. ;   [Functions]
40. Function VersionCheckF
41.     Exch $1 ; $1 contains Version 1
42.     Exch
43.     Exch $2 ; $2 contains Version 2
44.     Exch
45.     Push $R0
46.     ;   initialize Variables
47.     StrCpy $V1 ""
48.     StrCpy $V2 ""
49.     StrCpy $P1 ""
50.     StrCpy $P2 ""
51.     StrCpy $Reslt ""
52.     ;   Set the file pointers:
53.     IntOp $P1 $P1 + 1
54.     IntOp $P2 $P2 + 1
55.     ;  ******************* Get 1st version number for Ver1 **********************
56.     V11:
57.     ;   I use $1 and $2 to help keep identify "Ver1" vs. "Ver2"
58.     StrCpy $R0 $1 1 $P1 ;$R0 contains the character at position $P1
59.     IntOp $P1 $P1 + 1   ;increments the file pointer for the next read
60.     StrCmp $R0 "" V11end 0  ;check for empty string
61.     strCmp $R0 "." v11end 0
62.     strCpy $V1 "$V1$R0"
63.     Goto V11
64.     V11End:
65.     StrCmp $V1 "" 0 +2
66.     StrCpy $V1 "0"  
67.     ;  ******************* Get 1st version number for Ver2 **********************
68.     V12:
69.     StrCpy $R0 $2 1 $P2 ;$R0 contains the character at position $P1
70.     IntOp $P2 $P2 + 1   ;increments the file pointer for the next read
71.     StrCmp $R0 "" V12end 0  ;check for empty string
72.     strCmp $R0 "." v12end 0
73.     strCpy $V2 "$V2$R0"
74.     Goto V12
75.     V12End:
76.     StrCmp $V2 "" 0 +2
77.     StrCpy $V2 "0"  
78.     ;   At this point, we can compare the results.  If the numbers are not
79.     ;       equal, then we can exit
80.     IntCmp $V1 $V2 cont1 older1 newer1
81.     older1: ; Version 1 is older (less than) than version 2
82.     StrCpy $Reslt 2
83.     Goto ExitFunction
84.     newer1: ; Version 1 is newer (greater than) Version 2
85.     StrCpy $Reslt 1
86.     Goto ExitFunction
87.     Cont1: ;Versions are the same.  Continue searching for differences
88.     ;   Reset $V1 and $V2
89.     StrCpy $V1 ""
90.     StrCpy $V2 ""
92.     ;  ******************* Get 2nd version number for Ver1 **********************   
93.     V21:
94.     StrCpy $R0 $1 1 $P1 ;$R0 contains the character at position $P1
95.     IntOp $P1 $P1 + 1   ;increments the file pointer for the next read
96.     StrCmp $R0 "" V21end 0  ;check for empty string
97.     strCmp $R0 "." v21end 0
98.     strCpy $V1 "$V1$R0"
99.     Goto V21
100.     V21End:
101.     StrCmp $V1 "" 0 +2
102.     StrCpy $V1 "0"  
103.     ;  ******************* Get 2nd version number for Ver2 **********************
104.     V22:
105.     StrCpy $R0 $2 1 $P2 ;$R0 contains the character at position $P1
106.     IntOp $P2 $P2 + 1   ;increments the file pointer for the next read
107.     StrCmp $R0 "" V22end 0  ;check for empty string
108.     strCmp $R0 "." V22end 0
109.     strCpy $V2 "$V2$R0"
110.     Goto V22
111.     V22End:
112.     StrCmp $V2 "" 0 +2
113.     StrCpy $V2 "0"  
114.     ;   At this point, we can compare the results.  If the numbers are not
115.     ;       equal, then we can exit
116.     IntCmp $V1 $V2 cont2 older2 newer2
117.     older2: ; Version 1 is older (less than) than version 2
118.     StrCpy $Reslt 2 
119.     Goto ExitFunction
120.     newer2: ; Version 1 is newer (greater than) Version 2
121.     StrCpy $Reslt 1
122.     Goto ExitFunction
123.     Cont2: ;Versions are the same.  Continue searching for differences
124.     ;   Reset $V1 and $V2
125.     StrCpy $V1 ""
126.     StrCpy $V2 ""   
127.     ;  ******************* Get 3rd version number for Ver1 **********************   
128.     V31:
129.     StrCpy $R0 $1 1 $P1 ;$R0 contains the character at position $P1
130.     IntOp $P1 $P1 + 1   ;increments the file pointer for the next read
131.     StrCmp $R0 "" V31end 0  ;check for empty string
132.     strCmp $R0 "." v31end 0
133.     strCpy $V1 "$V1$R0"
134.     Goto V31
135.     V31End:
136.     StrCmp $V1 "" 0 +2
137.     StrCpy $V1 "0"  
138.     ;  ******************* Get 3rd version number for Ver2 **********************
139.     V32:
140.     StrCpy $R0 $2 1 $P2 ;$R0 contains the character at position $P1
141.     IntOp $P2 $P2 + 1   ;increments the file pointer for the next read
142.     StrCmp $R0 "" V32end 0  ;check for empty string
143.     strCmp $R0 "." V32end 0
144.     strCpy $V2 "$V2$R0"
145.     Goto V32
146.     V32End:
147.     StrCmp $V2 "" 0 +2
148.     StrCpy $V2 "0"  
149.     ;   At this point, we can compare the results.  If the numbers are not
150.     ;       equal, then we can exit
151.     IntCmp $V1 $V2 cont3 older3 newer3
152.     older3: ; Version 1 is older (less than) than version 2
153.     StrCpy $Reslt 2
154.     Goto ExitFunction
155.     newer3: ; Version 1 is newer (greater than) Version 2
156.     StrCpy $Reslt 1
157.     Goto ExitFunction
158.     Cont3: ;Versions are the same.  Continue searching for differences
159.     ;   Reset $V1 and $V2
160.     StrCpy $V1 ""
161.     StrCpy $V2 ""
162.     ;  ******************* Get 4th version number for Ver1 **********************   
163.     V41:
164.     StrCpy $R0 $1 1 $P1 ;$R0 contains the character at position $P1
165.     IntOp $P1 $P1 + 1   ;increments the file pointer for the next read
166.     StrCmp $R0 "" V41end 0  ;check for empty string
167.     strCmp $R0 "." v41end 0
168.     strCpy $V1 "$V1$R0"
169.     Goto V41
170.     V41End:
171.     StrCmp $V1 "" 0 +2
172.     StrCpy $V1 "0"  
173.     ;  ******************* Get 4th version number for Ver2 **********************
174.     V42:
175.     StrCpy $R0 $2 1 $P2 ;$R0 contains the character at position $P1
176.     IntOp $P2 $P2 + 1   ;increments the file pointer for the next read
177.     StrCmp $R0 "" V42end 0  ;check for empty string
178.     strCmp $R0 "." V42end 0
179.     strCpy $V2 "$V2$R0"
180.     Goto V42
181.     V42End:
182.     StrCmp $V2 "" 0 +2
183.     StrCpy $V2 "0"  
184.     ;   At this point, we can compare the results.  If the numbers are not
185.     ;       equal, then we can exit
186.     IntCmp $V1 $V2 cont4 older4 newer4
187.     older4: ; Version 1 is older (less than) than version 2
188.     StrCpy $Reslt 2
189.     Goto ExitFunction
190.     newer4: ; Version 1 is newer (greater than) Version 2
191.     StrCpy $Reslt 1
192.     Goto ExitFunction
193.     Cont4: 
194.     ;Versions are the same.  We've reached the end of the version
195.     ;   strings, so set the function to 0 (equal) and exit
196.     StrCpy $Reslt 0
197.     ExitFunction:
198.     Pop $R0
199.     Pop $1
200.     Pop $2
201.     Push $Reslt
202. FunctionEnd

我的修改

1. (cur) (last)  12:35, 31 December 2008 219.131.237.209 (Talk) (6,929 bytes) (→if have not initialized the variables, the second call will be not correct.如果不初始化，之后的调用结果将是错误的) (undo)
3. Revision as of 10:16, 6 June 2006 (edit)
   Afrow UK ( Talk | contribs)
   (→Description)
   ← Older edit
   Current revision (12:35, 31 December 2008) (edit) (undo)
   219.131.237.209 ( Talk)
   (→if have not initialized the variables, the second call will be not correct.如果不初始化，之后的调用结果将是错误的)
    
   Line 55: Line 55:
   Exch
   Exch
   Push $R0
   Push $R0
    +
   ; initialize Variables
    +
   StrCpy $V1 ""
    +
   StrCpy $V2 ""
    +
   StrCpy $P1 ""
    +
   StrCpy $P2 ""
    +
   StrCpy $Reslt ""
   ; Set the file pointers:
   ; Set the file pointers:
   IntOp $P1 $P1 + 1
   IntOp $P1 $P1 + 1

The End.