Skip to content
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

libssh2: raise lowest supported version to 1.2.8 #16199

Closed
wants to merge 2 commits into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Feb 6, 2025

Shipped on April 5 2011

@bagder bagder added the SCP/SFTP label Feb 6, 2025
@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
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
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
@bagder bagder deleted the bagder/libssh2-drop-oldies branch February 26, 2025 07:48
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.

3 participants