Skip to content
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

Add 0.5.1 zip to release #441

Closed
pietbrauer opened this issue Feb 15, 2015 · 12 comments
Closed

Add 0.5.1 zip to release #441

pietbrauer opened this issue Feb 15, 2015 · 12 comments

Comments

@pietbrauer
Copy link
Member

In the most recent carthage release it is possible to attach zips. I tried to build it for 0.6.0 but it is not building because of a libssh2 error.

However I was able to build 0.5.1 and tried to attach it to this ticket, but it is 12,3 MB and Github only allows 10 MB. It would be so cool if I could attach it to the release 🐈

Many Thanks!

@jspahrsummers
Copy link
Contributor

I've just invited you to be a collaborator. 😊

@pietbrauer
Copy link
Member Author

I uploaded the binary and since removed it again because the header search path is not working.

So everyone including this project via carthage will get the git2/filter.h file not found errors. Because obviously "$(SRCROOT)/Carthage/Checkouts/objective-git/External/libgit2/include/" does no longer exist.

Any idea?

@jspahrsummers
Copy link
Contributor

Bleh, this is the problem that the module map is intended to solve, but that only came in 0.6.

@pietbrauer
Copy link
Member Author

Even with the module map it is not solved (tried with 0.6) and the mac build needs to set HEADER_SEARCH_PATHS too so it can't be an iOS configuration error. Any ideas?

@jspahrsummers
Copy link
Contributor

Sorry, I don't have a whole lot of free time to investigate this, especially because ObjectiveGit is successfully building in our subproject setup. 😕

@pietbrauer
Copy link
Member Author

Sure, maybe @phatblat has an idea. He already had a fix every time I opened an issue 😃

@jspahrsummers
Copy link
Contributor

Now that #442 is fixed, I was able to build and upload a binary for 0.6.1. Does that one work?

@pietbrauer
Copy link
Member Author

Worked but had to add $(SRCROOT)/Carthage/Build/iOS/ObjectiveGit.framework/Headers/ to HEADER_SEARCH_PATHS

@jspahrsummers
Copy link
Contributor

I wonder if we could fix that by using angle-bracket imports in the module map? Anyways, glad to hear it's working now.

@pietbrauer
Copy link
Member Author

You mean like this? That doesn't work.

@jspahrsummers
Copy link
Contributor

Ah, alright. 😞

Thanks for trying!

@zetachang
Copy link

Add HEADER_SEARCH_PATHS do the trick. thanks @pietbrauer !

aral added a commit to aral/objective-git that referenced this issue Oct 15, 2015
I had to manually add a header search path as per libgit2#441 in order to compile. Added that as a step to the instructions. Only tested on iOS so added to end of that list. Not sure if it also applies for Mac targets (assuming so). If so, please feel free to move to the top where it might make more sense and apply to both.
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

No branches or pull requests

3 participants