BF: use --python-match minor option in new datalad-installer release to match outside version of Python#6827
Merged
Conversation
This commit adds a very rough ssh-version detection to deal with different filename quoting requirements in ssh version <= 8 and in ssh version 9. This fixes the ssh copy tests error that is described in issue datalad#6806. This is a rough first draft to verify the overall viability and to test the waters in CI. The version parsing will have to be improved.
This commit adds a "# nosec" comment to an application of 'md5' in order to skip insecure-hash checks by 'codeclimate'. The hash is not security critical, since it is only used as an "abbreviation" of the unique connection property string.
This commit adds '<' and '>' to the characters that can be put into an obscure filename.
Thie commit adds a regular expression to determine the version, sub-version and patch-number of the used ssh-client. It currently only recognizes "OpenSSH" versions.
This commit removes the custom ssh-version detection code from sshconnector.py in favor of the existing version detection code in external_versions.py
This commit refactors the external version detection for ssh to include the ssh type. Currently only OpenSSH is recognized, all other types are considered to be unknown.
This commit removes the ssh type signaling in the external versions object because it exhibited too many incompatibilities and we do not really have a strategy to deal with ssh types other than OpenSSH.
now we use external_versions to get version and it does not store implementation. So compare just against the [0]
Swapped the logic for version comparison so that if we have not figured out ssh version, we do not quote (since we do not know if we should).
…to match outside version of python we set in our matrix runs of travis CI setup. with datalad-installer 0.9.0 we can now match to minor (or try even more detailed match) portion of the version of system wide python. Again -- it would not be system wide python intallation, but conda installation of python matching in minor portion of the version, which is what we care in our matrix This one is positioned against master since maint we run into difficulty installing nose in env with python 3.10
3 tasks
Codecov Report
@@ Coverage Diff @@
## maint #6827 +/- ##
==========================================
+ Coverage 90.24% 91.23% +0.98%
==========================================
Files 354 354
Lines 46056 46078 +22
==========================================
+ Hits 41563 42037 +474
+ Misses 4493 4041 -452
Continue to review full report at Codecov.
|
Member
Author
|
Any brave soul to hit merge for two birds at once? |
Member
Author
|
#6826 is merged, so this is no brainer |
Contributor
|
🚀 PR was released in |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Yet another replacement now of #6803 and sits on top of #6826 -- I am aiming for a clean CI without needing to lookup what failed. #6826 should be merged first