Merge lp:~yrke/tapaal/tapaal-fix-1855675-notWorkingWithLatestUPPAAL into lp:tapaal

Proposed by Kenneth Yrke Jørgensen
Status: Merged
Approved by: Jiri Srba
Approved revision: 1034
Merged at revision: 1036
Proposed branch: lp:~yrke/tapaal/tapaal-fix-1855675-notWorkingWithLatestUPPAAL
Merge into: lp:tapaal
Diff against target: 52 lines (+16/-7)
2 files modified
src/dk/aau/cs/verification/UPPAAL/Verifyta.java (+15/-6)
src/pipe/gui/Pipe.java (+1/-1)
To merge this branch: bzr merge lp:~yrke/tapaal/tapaal-fix-1855675-notWorkingWithLatestUPPAAL
Reviewer Review Type Date Requested Status
Jiri Srba Approve
Review via email: mp+376632@code.launchpad.net

Commit message

Merged tapaal-fix-1855675-notWorkingWithLatestUPPAAL, Fixes bug #1855675

To post a comment you must log in.
Revision history for this message
Kenneth Yrke Jørgensen (yrke) wrote :

I don't like the fact that it duplicates the version check code, this should really be extracted to a function. But as a bugfix this works.

Also changes the requred version of UPPAAL to 4.1.23 (latest), I made a request to UPPAAL devs if the can give us a version number based on the rev number.

1034. By Kenneth Yrke Jørgensen

Updated uppaal version number to match rev. 5648.

Revision history for this message
Kenneth Yrke Jørgensen (yrke) wrote :

Latest version set UPPAAL version to 4.1.19 to match rev. 5648

Revision history for this message
Jiri Srba (srba) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/dk/aau/cs/verification/UPPAAL/Verifyta.java'
--- src/dk/aau/cs/verification/UPPAAL/Verifyta.java 2019-03-27 08:54:41 +0000
+++ src/dk/aau/cs/verification/UPPAAL/Verifyta.java 2019-12-12 08:15:25 +0000
@@ -134,11 +134,20 @@
134 resetVerifyta();134 resetVerifyta();
135 return false;135 return false;
136 } else {136 } else {
137 int version = Integer.parseInt(versionAsString);137
138138 String[] version = getVersion().split("\\.");
139 if (version < Pipe.verifytaMinRev) {139 String[] targetversion = Pipe.verifytaMinRev.split("\\.");
140 return false;140
141 for (int i = 0; i < targetversion.length; i++) {
142 if (version.length < i + 1) version[i] = "0";
143 int diff = Integer.parseInt(version[i]) - Integer.parseInt(targetversion[i]);
144 if (diff > 0) {
145 break;
146 } else if (diff < 0) {
147 return false;
148 }
141 }149 }
150
142 }151 }
143152
144 return true;153 return true;
@@ -165,8 +174,8 @@
165 } catch (IOException e) {174 } catch (IOException e) {
166 result = null;175 result = null;
167 }176 }
168 177
169 Pattern pattern = Pattern.compile("\\((?:rev. )?(\\d+)\\)");178 Pattern pattern = Pattern.compile("(\\d+\\.\\d+\\.\\d+)");
170 Matcher m = pattern.matcher(versioninfo);179 Matcher m = pattern.matcher(versioninfo);
171 m.find();180 m.find();
172 result = m.group(1);181 result = m.group(1);
173182
=== modified file 'src/pipe/gui/Pipe.java'
--- src/pipe/gui/Pipe.java 2019-10-23 14:12:48 +0000
+++ src/pipe/gui/Pipe.java 2019-12-12 08:15:25 +0000
@@ -77,7 +77,7 @@
77 public static Color QUERY_NOT_SATISFIED_COLOR = new Color(204, 0, 0);77 public static Color QUERY_NOT_SATISFIED_COLOR = new Color(204, 0, 0);
78 public static Color QUERY_INCONCLUSIVE_COLOR = new Color(255, 153, 0);78 public static Color QUERY_INCONCLUSIVE_COLOR = new Color(255, 153, 0);
7979
80 public static final int verifytaMinRev = 5648;80 public static final String verifytaMinRev = "4.1.19";
81 public static final String verifytapnMinRev = "1.3.0";81 public static final String verifytapnMinRev = "1.3.0";
82 public static final String verifydtapnMinRev = "3.3.0";82 public static final String verifydtapnMinRev = "3.3.0";
83 public static final String verifypnMinRev = "3.0.2";83 public static final String verifypnMinRev = "3.0.2";

Subscribers

People subscribed via source and target branches