Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

build: use pkg-config to find curl header#1328

Closed
rsmarples wants to merge 24 commits intonodegit:masternodegit/nodegit:masterfrom
rsmarples:find-libcurlCopy head branch name to clipboard
Closed

build: use pkg-config to find curl header#1328
rsmarples wants to merge 24 commits intonodegit:masternodegit/nodegit:masterfrom
rsmarples:find-libcurlCopy head branch name to clipboard

Conversation

@rsmarples
Copy link
Contributor

pkgsrc (as used by NetBSD) installs package headers in non default
locations.
The solution is to use pkg-config to work out the correct location
via CFLAGS.

pkgsrc (as used by NetBSD) installs package headers in non default
locations.
The solution is to use pkg-config to work out the correct location
via CFLAGS.
@tbranyen
Copy link
Member

Looks good to me 👍

Carson Howard and others added 22 commits July 24, 2017 11:41
Bump libgit2 to maintenance v0.26
AsyncBatonWithNoResults

wip 2
Build + test register complete

filter init test wip
Adding subtract filter + refactoring template code to eliminate new self() in cc code

filter registry update + templating changes
working filters!! (for now)
Reached Async Land

adding error handling

testing commit + apply callback

fixed field_accessor template on windows
implementing checks in test suite

added more filter tests

lol CRLF

Added more unit tests
Add CI build config for node stable version
Adding git_filter support in nodegit
Exposed libgit2 git_branch_remote_name method
pkgsrc (as used by NetBSD) installs package headers in non default
locations.
The solution is to use pkg-config to work out the correct location
via CFLAGS.
@rsmarples rsmarples closed this Aug 5, 2017
@rsmarples
Copy link
Contributor Author

no idea what happened here, I'll redo

@valpackett
Copy link

The "<!(pkg-config libcurl --cflags 2>/dev/null || echo)" line is also required on FreeBSD.

@rsmarples
Copy link
Contributor Author

See PR #1237 for a better solution. Bump that one instead ;)

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.

5 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.