Skip to content

introduce 'excludes' option for tests #395

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jacobmealey
Copy link
Contributor

Addresses hexadecimal changes from curl 8.15. I've also duplicated the tests that were failing with the new uppercase hex values.

Thanks @charles2910 for reporting this bug
fixes: #394

@jacobmealey
Copy link
Contributor Author

Not sure why the curl for win is failing. Will investigate tomorrow

@jacobmealey
Copy link
Contributor Author

The curl-for-win build is failing on my clone of trurl on master, which was synced this morning: https://github.com/jacobmealey/urler/actions/runs/16097961365/job/45423345923
Both failing builds have the same error

ld.lld-19: error: undefined symbol: if_nametoindex

The daily builds on the curl-for-win repo are passing.

cc @vszakats

vszakats added a commit to curl/curl-for-win that referenced this pull request Jul 6, 2025
Static libcurl on Windows, built for Vista and up requires the iphlpapi
system DLL since this curl commit:

curl/curl@0d71b18
curl/curl#17413

Reported-by: Jacob Mealey
Bug: curl/trurl#395 (comment)
@vszakats
Copy link
Member

vszakats commented Jul 6, 2025

@jacobmealey Thanks for reporting this. Indeed, static libcurl needs linking
an additional Windows system lib (when building for Vista or up) after a recent
update, and I missed to sync curl-for-win's trurl build to accommodate for that.

Is it better after this?:
curl/curl-for-win@b39be5a

(it's enough to retry the CI job or push any change to trigger a fresh run)

@jacobmealey
Copy link
Contributor Author

@vszakats that worked! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

trurl's tests fail with curl 8.15-rc2
3 participants