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

find libimobiledevice-glue in e.g. /opt/homebrew/ #1532

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tallenintegsys
Copy link

This allows one to build under OSX using xcode 14.2.

@nikias
Copy link
Member

nikias commented Apr 1, 2024

It compiles for me even without the change. The change for common makes sense, since it actually uses something from libimobiledevice-glue, but for tools I added the dependency for every tool that uses it. What are you warnings/errors when you try to compile?

@tallenintegsys
Copy link
Author

tallenintegsys commented Apr 1, 2024 via email

@tallenintegsys
Copy link
Author

tallenintegsys commented Apr 1, 2024 via email

@tallenintegsys
Copy link
Author

It's worth noting that while I have been an open source user of a very long time (like when Linux booted off of two floppies) I've not contributed much. As such I don't really know what's expected of me.

@tallenintegsys
Copy link
Author

Some random notes (sorry, should have done this in one message). In the README I only saw instructions of Debian/Ubuntu so I assumed you were only testing on same but upon closer reading I see that you test with Mac OS too. Some notes: On Ubuntu 22.04 using the libplist-dev (from apt) I get Requested 'libplist-2.0 >= 2.3.0' but version of libplist is 2.2.0 from autogen. As for Mac OS here is some background: Ventura 13.6.6, automake-1.16, Apple clang version 14.0.0, on an M1 (arm64), my brew install list is a bit difficult as I have to hand type. Apologies for the noise but trying to make helpful noises.

@tallenintegsys
Copy link
Author

This will just be ignored forever? Not sure what else I could have done.

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.

None yet

2 participants