public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
* [gentoo-commits] proj/javatoolkit:master commit in: src/py/javatoolkit/, src/py/
@ 2017-08-08 20:39 Patrice Clement
  0 siblings, 0 replies; 3+ messages in thread
From: Patrice Clement @ 2017-08-08 20:39 UTC (permalink / raw
  To: gentoo-commits

commit:     7d1d602b655317b1f7996d2e88943b1924eca511
Author:     Patrice Clement <monsieurp <AT> gentoo <DOT> org>
AuthorDate: Tue Aug  8 20:13:48 2017 +0000
Commit:     Patrice Clement <monsieurp <AT> gentoo <DOT> org>
CommitDate: Tue Aug  8 20:38:57 2017 +0000
URL:        https://gitweb.gentoo.org/proj/javatoolkit.git/commit/?id=7d1d602b

change variable name

because cvv is indeed a magic class (except for James)

 src/py/class-version-verify.py | 22 ++++++++++------------
 src/py/javatoolkit/cvv.py      |  3 +--
 2 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/src/py/class-version-verify.py b/src/py/class-version-verify.py
index b46d99c..a53b10d 100755
--- a/src/py/class-version-verify.py
+++ b/src/py/class-version-verify.py
@@ -4,13 +4,11 @@
 # Copyright(c) 2005, Gentoo Foundation
 #
 # Licensed under the GNU General Public License, v2
-#
-# $Id$
 
 import os
 import sys
 from optparse import OptionParser, make_option
-from javatoolkit.cvv import *
+from javatoolkit.cvv import CVVMagic
 
 
 def main():
@@ -62,35 +60,35 @@ def main():
 
     options.version = int(options.version.split(".")[-1])
 
-    cvv = cvv(options.version)
+    cvv_magic = CVVMagic(options.version)
 
     for arg in args:
         if os.path.isfile(arg):
-            cvv.do_file(arg)
+            cvv_magic.do_file(arg)
 
         if options.deep and os.path.isdir(arg):
             for root, dirs, files in os.walk(arg):
                 for filename in files:
-                    cvv.do_file("%s/%s" % (root, filename))
+                    cvv_magic.do_file("%s/%s" % (root, filename))
 
     if options.file_only:
-        lst = set([set[1] for set in cvv.bad])
+        lst = set([set[1] for set in cvv_magic.bad])
         for i in lst:
             print(i)
     else:
         if options.verbose:
-            for set in cvv.good:
+            for set in cvv_magic.good:
                 print("Good: %s %s %s" % set)
 
         if not options.silent:
-            for set in cvv.bad:
+            for set in cvv_magic.bad:
                 print("Bad: %s %s %s" % set)
 
         print("CVV: %s\nChecked: %i Good: %i Bad: %i" %
-              (options.version, len(cvv.good) +
-               len(cvv.bad), len(cvv.good), len(cvv.bad)))
+              (options.version, len(cvv_magic.good) +
+               len(cvv_magic.bad), len(cvv_magic.good), len(cvv_magic.bad)))
 
-    if len(cvv.bad) > 0:
+    if len(cvv_magic.bad) > 0:
         sys.exit(1)
     else:
         sys.exit(0)

diff --git a/src/py/javatoolkit/cvv.py b/src/py/javatoolkit/cvv.py
index c72cfcc..b5f817e 100644
--- a/src/py/javatoolkit/cvv.py
+++ b/src/py/javatoolkit/cvv.py
@@ -1,7 +1,6 @@
 # Copyright 2005, Thomas Matthijs <axxo@gentoo.org>
 # Copyright 1999-2008 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: $
 
 import os
 import sys
@@ -9,7 +8,7 @@ from struct import unpack
 from zipfile import ZipFile
 
 
-class cvv:
+class CVVMagic:
     def __init__(self, target):
         self.target = target
         self.good = []


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [gentoo-commits] proj/javatoolkit:master commit in: src/py/javatoolkit/, src/py/
@ 2021-06-19 17:35 Miroslav Šulc
  0 siblings, 0 replies; 3+ messages in thread
From: Miroslav Šulc @ 2021-06-19 17:35 UTC (permalink / raw
  To: gentoo-commits

commit:     09739b1ae090fc1944a1267ed4c40cf58ce7d049
Author:     Miroslav Šulc <fordfrog <AT> gentoo <DOT> org>
AuthorDate: Sat Jun 19 17:34:56 2021 +0000
Commit:     Miroslav Šulc <fordfrog <AT> gentoo <DOT> org>
CommitDate: Sat Jun 19 17:35:25 2021 +0000
URL:        https://gitweb.gentoo.org/proj/javatoolkit.git/commit/?id=09739b1a

fixed handling of module-info.class

Closes: https://bugs.gentoo.org/742935
Signed-off-by: Miroslav Šulc <fordfrog <AT> gentoo.org>

 src/py/class-version-verify.py | 8 +++++---
 src/py/javatoolkit/cvv.py      | 5 ++++-
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/src/py/class-version-verify.py b/src/py/class-version-verify.py
index e4ee7c8..968618c 100755
--- a/src/py/class-version-verify.py
+++ b/src/py/class-version-verify.py
@@ -81,10 +81,12 @@ def main():
         if not options.silent:
             for set in cvv_magic.bad:
                 print("Bad: %s %s %s" % set)
+            for set in cvv_magic.skipped:
+                print("Skipped: %s %s %s" % set)
 
-        print("CVV: %s\nChecked: %i Good: %i Bad: %i" %
-              (options.version, len(cvv_magic.good) +
-               len(cvv_magic.bad), len(cvv_magic.good), len(cvv_magic.bad)))
+        print("CVV: %s\nChecked: %i Good: %i Bad: %i Skipped: %i" %
+              (options.version, len(cvv_magic.good) + len(cvv_magic.bad) + len(cvv_magic.skipped),
+              len(cvv_magic.good), len(cvv_magic.bad), len(cvv_magic.skipped)))
 
     if len(cvv_magic.bad) > 0:
         sys.exit(1)

diff --git a/src/py/javatoolkit/cvv.py b/src/py/javatoolkit/cvv.py
index 6ae2501..1f15415 100644
--- a/src/py/javatoolkit/cvv.py
+++ b/src/py/javatoolkit/cvv.py
@@ -13,9 +13,12 @@ class CVVMagic:
         self.target = target
         self.good = []
         self.bad = []
+        self.skipped = []
 
     def add(self, version, jar, file):
-        if version <= self.target:
+        if file == "module-info.class" and self.target < 9:
+            self.skipped.append(("1.%s" % (version), jar, file))
+        elif version <= self.target:
             self.good.append(("1.%s" % (version), jar, file))
         else:
             self.bad.append(("1.%s" % (version), jar, file))


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [gentoo-commits] proj/javatoolkit:master commit in: src/py/javatoolkit/, src/py/
@ 2021-06-19 19:12 Miroslav Šulc
  0 siblings, 0 replies; 3+ messages in thread
From: Miroslav Šulc @ 2021-06-19 19:12 UTC (permalink / raw
  To: gentoo-commits

commit:     916b92e8a987b403fa73f27dd0d2d7e3a44c44ad
Author:     Miroslav Šulc <fordfrog <AT> gentoo <DOT> org>
AuthorDate: Sat Jun 19 19:10:39 2021 +0000
Commit:     Miroslav Šulc <fordfrog <AT> gentoo <DOT> org>
CommitDate: Sat Jun 19 19:10:39 2021 +0000
URL:        https://gitweb.gentoo.org/proj/javatoolkit.git/commit/?id=916b92e8

correctly handling java versions in class-version-verify.py

Signed-off-by: Miroslav Šulc <fordfrog <AT> gentoo.org>

 src/py/class-version-verify.py |  2 --
 src/py/javatoolkit/cvv.py      | 18 ++++++++++++++----
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/py/class-version-verify.py b/src/py/class-version-verify.py
index 968618c..dc44d90 100755
--- a/src/py/class-version-verify.py
+++ b/src/py/class-version-verify.py
@@ -56,8 +56,6 @@ def main():
         print("-t is mandatory")
         sys.exit(2)
 
-    options.version = int(options.version.split(".")[-1])
-
     cvv_magic = CVVMagic(options.version)
 
     for arg in args:

diff --git a/src/py/javatoolkit/cvv.py b/src/py/javatoolkit/cvv.py
index 1f15415..b0fdc84 100644
--- a/src/py/javatoolkit/cvv.py
+++ b/src/py/javatoolkit/cvv.py
@@ -10,18 +10,28 @@ from zipfile import ZipFile
 
 class CVVMagic:
     def __init__(self, target):
-        self.target = target
+        # this is a number 8 9 10 11 etc, not including 1.
+        if '.' in target:
+            self.target = int(target.split(".")[-1])
+        else:
+            self.target = int(target)
         self.good = []
         self.bad = []
         self.skipped = []
 
     def add(self, version, jar, file):
         if file == "module-info.class" and self.target < 9:
-            self.skipped.append(("1.%s" % (version), jar, file))
+            self.skipped.append((version, jar, file))
         elif version <= self.target:
-            self.good.append(("1.%s" % (version), jar, file))
+            if version < 9:
+                self.good.append(("1.%s" % (version), jar, file))
+            else:
+                self.good.append((version, jar, file))
         else:
-            self.bad.append(("1.%s" % (version), jar, file))
+            if version < 9:
+                self.bad.append(("1.%s" % (version), jar, file))
+            else:
+                self.bad.append((version, jar, file))
 
     def do_class(self,filename):
         classFile = file(filename,"rb")


^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-06-19 19:12 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-06-19 19:12 [gentoo-commits] proj/javatoolkit:master commit in: src/py/javatoolkit/, src/py/ Miroslav Šulc
  -- strict thread matches above, loose matches on Subject: below --
2021-06-19 17:35 Miroslav Šulc
2017-08-08 20:39 Patrice Clement

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox