-
Notifications
You must be signed in to change notification settings - Fork 281
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove EXIV2_EXT from code-base #1273
Comments
Thanks @clanmills, happy to help w/ testing this on msys2/mingw64 |
Ok, I will try to remove |
@LeoHsiao1. Please keep focus and get #1257 finished and merged. Removing EXIV2_EXT from system_tests is a different issue. I've asked GitHub to invite you to join Team Exiv2 as this will make it easier for me to assign issues for your attention. Because you have write access, you can re-assign issues to me if you don't want to deal with something assigned to you! |
I believe #1257 will be completed quickly if there are no new problems |
I also think we are almost 100% done on #1257. I think we only need to remove one line of code from tests/system_tests.py. That's it. 565 rmills@rmillsmbp:~/gnu/github/exiv2/leo/tests $ grep BT *.py
system_tests.py:from bash_tests import utils as BT
system_tests.py: BT.Config.bin_dir = os.path.abspath(config['ENV']['exiv2_path'])
system_tests.py: BT.Config.data_dir = os.path.abspath(config['paths']['data_path'])
system_tests.py: BT.Config.tmp_dir = os.path.abspath(config['paths']['tmp_path'])
system_tests.py: BT.Config.exiv2_ext = config['ENV']['binary_extension'] # <---- Remove this!
system_tests.py: BT.Config.exiv2_http = config['ENV']['exiv2_http']
system_tests.py: BT.Config.exiv2_port = int(config['ENV']['exiv2_port'])
566 rmills@rmillsmbp:~/gnu/github/exiv2/leo/tests $ I would like to see the bash wrappers for Keep the 'rocknroll' magic. I have no idea what it does. However it's not causing me any trouble. Keep it! Should it go into |
Always submit into 0.27-maintenance. I don't know if anybody is working on 'master' any more. Nothing much has been done since January. |
I think this is finished with #1289 |
Hi guys, I would like to know if this work (the whole work about migrating bash tests to python) is being ported to master or if you had any discussion about it. |
We're not done yet with #1215 on 0.27-maintenance. I would like to operate both Then we should "decommission" tests/*.so by updating test/makefile, test/CMakeList.txt, README.md and README-CONAN.md #1274 After that we can port them to 'master'. I hope porting will be painfree and only involve:
This will cause EXIV2_EXT to disappear from 'master'. The big win here is to simplify running the test suite on Visual Studio builds. Users will have to install python3 (with the lxml module) and not require bash/make and other MinGW/msys2 utilities to be on their PATH. The aim of #1215 is to extend the excellent tests/system_tests.py developed by Dan while making life for Visual Studio users much simpler. |
See #1269 (comment)
This environment string has caused difficulties on #1257 and should be removed from the code AND documentation.
I intend to invite @LeoHsiao1 to work on this.
The text was updated successfully, but these errors were encountered: