Skip to content

libssh2: raise lowest supported version to 1.2.8 #16199

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

Closed
wants to merge 2 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Feb 6, 2025

Shipped on April 5 2011

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
Shipped on April 5 2011
@bagder bagder added the SCP/SFTP label Feb 6, 2025

Verified

This commit was signed with the committer’s verified signature.
bagder Daniel Stenberg
@testclutch
Copy link

Analysis of PR #16199 at 9d36d127:

Test 987 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 2 different CI jobs (the link just goes to one of them).

Generated by Testclutch

@bagder bagder closed this in 553248f Feb 6, 2025
vszakats added a commit to vszakats/curl that referenced this pull request Feb 10, 2025

Verified

This commit was signed with the committer’s verified signature.
vszakats Viktor Szakats
Seen in curl-for-win daily builds:
```
/home/runner/work/curl-for-win/curl-for-win/curl/lib/vssh/libssh2.c:2644:9: warning: 'libssh2_scp_send_ex' is deprecated: since libssh2 1.2.6. Use libssh2_scp_send64() [-Wdeprecated-declarations]
 2644 |         SCP_SEND(sshc->ssh_session, sshp->path, data->set.new_file_perms,
      |         ^
/home/runner/work/curl-for-win/curl-for-win/curl/lib/vssh/libssh2.c:417:27: note: expanded from macro 'SCP_SEND'
  417 | #define SCP_SEND(a,b,c,d) libssh2_scp_send_ex(a, b, (int)(c), (size_t)d, 0, 0)
      |                           ^
/home/runner/work/curl-for-win/curl-for-win/libssh2/_x64-win-ucrt-libressl/usr/include/libssh2.h:1063:1: note: 'libssh2_scp_send_ex' has been explicitly marked deprecated here
 1063 | LIBSSH2_DEPRECATED(1.2.6, "Use libssh2_scp_send64()")
      | ^
/home/runner/work/curl-for-win/curl-for-win/libssh2/_x64-win-ucrt-libressl/usr/include/libssh2.h:170:25: note: expanded from macro 'LIBSSH2_DEPRECATED'
  170 |          __attribute__((deprecated("since libssh2 " # version ". " message)))
      |                         ^
1 warning generated.
```
https://github.com/curl/curl-for-win/actions/runs/13229370277/job/36924363438#step:3:5805

Follow-up to 553248f curl#16199
vszakats added a commit that referenced this pull request Feb 11, 2025

Verified

This commit was signed with the committer’s verified signature.
vszakats Viktor Szakats
Seen in curl-for-win daily, building against libssh2 1.11.2_DEV:
```
curl-for-win/curl/lib/vssh/libssh2.c:2644:9: warning: 'libssh2_scp_send_ex' is deprecated:
  since libssh2 1.2.6. Use libssh2_scp_send64() [-Wdeprecated-declarations]
 2644 |         SCP_SEND(sshc->ssh_session, sshp->path, data->set.new_file_perms,
```
Ref: https://github.com/curl/curl-for-win/actions/runs/13229370277/job/36924363438#step:3:5805

Follow-up to 553248f #16199

Closes #16291
vszakats added a commit that referenced this pull request Feb 12, 2025

Verified

This commit was signed with the committer’s verified signature.
vszakats Viktor Szakats
Follow-up to 553248f #16199

Closes #16292
@bagder bagder deleted the bagder/libssh2-drop-oldies branch February 26, 2025 07:48
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Seen in curl-for-win daily, building against libssh2 1.11.2_DEV:
```
curl-for-win/curl/lib/vssh/libssh2.c:2644:9: warning: 'libssh2_scp_send_ex' is deprecated:
  since libssh2 1.2.6. Use libssh2_scp_send64() [-Wdeprecated-declarations]
 2644 |         SCP_SEND(sshc->ssh_session, sshp->path, data->set.new_file_perms,
```
Ref: https://github.com/curl/curl-for-win/actions/runs/13229370277/job/36924363438#step:3:5805

Follow-up to 553248f curl#16199

Closes curl#16291
pps83 pushed a commit to pps83/curl that referenced this pull request Apr 26, 2025

Unverified

This commit is not signed, but one or more authors requires that any commit attributed to them is signed.
Follow-up to 553248f curl#16199

Closes curl#16292
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants