Skip to content

Conversation

xry111
Copy link
Contributor

@xry111 xry111 commented May 31, 2023

Co-authored-by: Viktor Szakats
Regression from 83853f8 #663
Fixes #1056

@xry111 xry111 force-pushed the xry111/issue1056 branch from 1c9f514 to b95e757 Compare May 31, 2023 08:58
@vszakats vszakats force-pushed the xry111/issue1056 branch from 81bae52 to 59cfd30 Compare May 31, 2023 09:50
@vszakats vszakats changed the title tests: Do not run tests accessing internal functions if --disable-static tests: skip tests requiring static lib if --disable-static May 31, 2023
@vszakats vszakats changed the title tests: skip tests requiring static lib if --disable-static autotools: skip tests requiring static lib if --disable-static May 31, 2023
@vszakats vszakats force-pushed the xry111/issue1056 branch 2 times, most recently from 6b4302f to 47b076f Compare May 31, 2023 11:52
- rename RUN_TESTS_WITH_LIB_STATIC to HAVE_LIB_STATIC

- shorten suffix _WITH_LIB_STATIC to _STATIC

- drop existing TESTS_WITH_LIB_STATIC variable now with a duplicate of
  exceptions and use the new lists in CMakeLists.txt and Makefile.mk.
  Also fixing the latter to not lose static tests.

- fix CMake builds to keep running static tests.

- restore and update comment in Makefile.inc

- update variable lists in comments
@vszakats vszakats force-pushed the xry111/issue1056 branch from 47b076f to 3990ff3 Compare May 31, 2023 12:01
@vszakats vszakats merged commit 572c57c into libssh2:master May 31, 2023
@vszakats
Copy link
Member

Thank you! Merged now.

vszakats added a commit that referenced this pull request May 31, 2023
Add a non-static autotools build to GitHub Actions. Make this build
target i386 and libgcrypt, to test a new build combination if we are at
it.

Also:
- GHA: add necessary generic bits for i386 autotools builds.
- AppVeyor CI: teach it to ignore commits updating our GHA config.

Follow-up to 572c57c #1072
Closes #1074
lampmanyao pushed a commit to lampmanyao/libssh2 that referenced this pull request Jul 16, 2023
lampmanyao pushed a commit to lampmanyao/libssh2 that referenced this pull request Jul 16, 2023
Add a non-static autotools build to GitHub Actions. Make this build
target i386 and libgcrypt, to test a new build combination if we are at
it.

Also:
- GHA: add necessary generic bits for i386 autotools builds.
- AppVeyor CI: teach it to ignore commits updating our GHA config.

Follow-up to 572c57c libssh2#1072
Closes libssh2#1074
agreppin pushed a commit to agreppin/libssh2 that referenced this pull request Jul 14, 2024
agreppin pushed a commit to agreppin/libssh2 that referenced this pull request Jul 14, 2024
Add a non-static autotools build to GitHub Actions. Make this build
target i386 and libgcrypt, to test a new build combination if we are at
it.

Also:
- GHA: add necessary generic bits for i386 autotools builds.
- AppVeyor CI: teach it to ignore commits updating our GHA config.

Follow-up to 572c57c libssh2#1072
Closes libssh2#1074
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

make check fails with --disable-static
2 participants