search

nginx / unit

NGINX Unit - universal web app server - a lightweight and versatile open source server that simplifies the application stack by natively executing application code across eight different programming language runtimes.
https://unit.nginx.org
Apache License 2.0
5.37k stars 323 forks source link

ci: Fix wasmtime paths in ci.yml #1419

Closed ac000 closed 2 weeks ago

ac000 commented 2 weeks ago

This comprises a couple of fixes to the CI.

1) Fix the wasmtime C API include/lib paths which recently changed with the update to wasmtime v24.

2) Run the CI when anything under src/contrib is changed (which would have caught the above).

thresheek commented 2 weeks ago

Hmm, right, I didnt think to check if we had any other users for wasmtime as packaged :/

thresheek commented 2 weeks ago

Commit log seems to have a typo:

Trigger ci.yml for changes under src/contrib

s,src,pkg,

ac000 commented 2 weeks ago

Oh, right, good catch!

ac000 commented 2 weeks ago

Fix commit subject

$ git range-diff c4906f9f...46ddb010
1:  c4906f9f ! 1:  46ddb010 ci: Trigger ci.yml for changes under src/contrib
    @@ Metadata
     Author: Andrew Clayton <a.clayton@nginx.com>

      ## Commit message ##
    -    ci: Trigger ci.yml for changes under src/contrib
    +    ci: Trigger ci.yml for changes under pkg/contrib

         This will catch changes to the likes of wasmtime and njs.