Skip to content

Use CMake when building the vendored libgit2. - #261

Merged
arthurschreiber merged 9 commits into
libgit2:developmentfrom
arthurschreiber:use_cmake
Oct 14, 2013
Merged

arthurschreiber merged 9 commits into
libgit2:developmentfrom
arthurschreiber:use_cmake

Conversation

@arthurschreiber

Copy link
Copy Markdown
Member

@arthurschreiber

Copy link
Copy Markdown
Member Author

I'll see if I can also make cross compilation work again, because this PR breaks it.

@johnbellone

Copy link
Copy Markdown
Contributor

I followed these patches and was able to successfully build the library using CMake on Linux. I got a failure attempting to build on OS X 10.8.

@arthurschreiber

Copy link
Copy Markdown
Member Author

@johnbellone Can you post the failure you get? I do all my development on OS X, and have not seen any issue.

@johnbellone

Copy link
Copy Markdown
Contributor

Ah it was probably something I screwed up. The build worked fine this time around.

@johnbellone

Copy link
Copy Markdown
Contributor

Are we waiting on anything here? I'd be great to not have to maintain my own fork :).

@arthurschreiber

Copy link
Copy Markdown
Member Author

Yeah. I think I won't tackle cross compilation in this PR (so that will be broken, but I guess I'm the only one who cared about that in the first place, anyway). The other is that if this PR gets merged in it's current state, gems will be missing needed files. It should be an easy fix, I'll look into it later today.

@arthurschreiber

Copy link
Copy Markdown
Member Author

@vmg I just pushed some changes to the gemspec to build valid gems and rebased against the latest upstream changes. Can you take a look?

@arthurschreiber

Copy link
Copy Markdown
Member Author

Merging this now. If you have cmake installed on your system, installing rugged will automatically include https support. We don't yet expose the needed functionality to work with https or ssh remotes that require authentication, but anonymous access for https should work just fine.

arthurschreiber added a commit that referenced this pull request Oct 14, 2013
Use CMake when building the vendored libgit2.
@arthurschreiber
arthurschreiber merged commit 4dcf573 into libgit2:development Oct 14, 2013
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.

2 participants