Skip to content

Commit

Permalink
Merge pull request #40 from brentos/fix-make-check
Browse files Browse the repository at this point in the history
Fix make check and make install on MacOS
  • Loading branch information
jptosso authored Sep 25, 2024
2 parents d6ebdaa + 5787737 commit b23b070
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 3 deletions.
6 changes: 5 additions & 1 deletion .github/workflows/make-install.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ jobs:
strategy:
matrix:
go-version: [1.18.x, 1.19.x]
os: [ubuntu-latest]
os: [ubuntu-latest, macos-latest]
runs-on: ${{ matrix.os }}
steps:
- name: Install Go
Expand All @@ -20,6 +20,10 @@ jobs:
with:
lfs: true
fetch-depth: 0 #for better blame info
- name: Install macOS dependencies
if: runner.os == 'macOS'
run: |
brew install autoconf automake libtool
- name: Setup
run: |
./build.sh
Expand Down
2 changes: 1 addition & 1 deletion Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ docs:
check_PROGRAMS = tests/simple_get

if OSX
AM_LDFLAGS = -framework CoreFoundation
AM_LDFLAGS = -framework CoreFoundation -framework Security
endif

if LINUX
Expand Down
2 changes: 1 addition & 1 deletion tests/simple_get.c
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

#include "coraza/coraza.h"

void logcb(void *log, const void *data)
void logcb(const void *data)
{
printf("%s\n", (const char *)data);
}
Expand Down

0 comments on commit b23b070

Please sign in to comment.