python-sutime icon indicating copy to clipboard operation
python-sutime copied to clipboard

error in sutime.py

Open agrawal-b opened this issue 4 years ago • 2 comments

In sutime.py, there are certain jar files which are marked as required, and in jars those files have been updated and therefore now sutime.py is showing error, so to resolve this, i manually changed the names of those 'required jar files' in sutime.py. can you please fix this issue, so that I do not need to manually change the names of jar files.

agrawal-b avatar Jun 22 '20 10:06 agrawal-b

Yeah for some reason the sutime.py file in the repo and the one after doing pip install don't match even though they are the same version. The code in the repo has the correct version of the java files

Utkichaps avatar Jul 19 '20 12:07 Utkichaps

Yeah for some reason the sutime.py file in the repo and the one after doing pip install don't match even though they are the same version. The code in the repo has the correct version of the java files

Version of java dependencies have been changed in this commit on 27th Oct 2018: https://github.com/FraBle/python-sutime/commit/67c47980932f1b32e02bd50ae071deaf003e7b4d

https://pypi.org/project/sutime/#history pip install version=0.2.0 is from Jan 11, 2017 (Though there have been few more releases, but the default version is 0.2.0)

Hence when we download dependencies using the updated pom.xml, there are mismatches regarding dependency jars.

Solution: Either take the latest version from pypi or install from the local directory.

Regarding pom.xml In few threads(e.g. https://github.com/FraBle/python-sutime/issues/33), people have mentioned to copy pom.xml into sutime folder. But in my opinion, that's not required. Instead of that change: mvn dependency:copy-dependencies -DoutputDirectory=./jars to mvn dependency:copy-dependencies -DoutputDirectory=./sutime/jars

Would request @FraBle to make this change.

@Utkichaps Thanks for a. https://github.com/FraBle/python-sutime/issues/41#issuecomment-664778673 Showing visually the version differences in jars. b. https://github.com/FraBle/python-sutime/issues/41#issuecomment-665793864 Mentioning the changes needed to convert java.lang.String to python str.

These helped in making the code run.

kaushikacharya avatar Aug 13 '20 10:08 kaushikacharya