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

Fixes Swift Package Manager Support #74

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

swhitty
Copy link
Contributor

@swhitty swhitty commented Feb 20, 2021

Updates Headers to only expose publicly;

  • Bagel.h
  • BagelCarrierDelegate.h
  • BagelConfiguration.h
  • BagelDeviceModel.h
  • BagelProjectModel.h

All other headers are project headers with forward declarations are added where required.

Adds support for macOS targets so that you can test the packages with with % swift build

@swhitty swhitty force-pushed the spm branch 2 times, most recently from 2495bf9 to b5cccc7 Compare February 20, 2021 05:25
@mran3
Copy link

mran3 commented Jul 14, 2021

I like Bagel, I want to be able to use it with Swift Package Manager. I hope this fixes it :)

@mran3
Copy link

mran3 commented Aug 5, 2021

I managed to make this work, If anyone needs it, as a workaround, it can be used via SPM from my repo:
https://github.com/mran3/Bagel
I basically did nothing, just merged @swhitty PR with the main branch of this project. All credits to him and @yagiz for this great project. I hope it gets active again.

@filimo
Copy link

filimo commented Aug 5, 2021

@mran3 You can find other fixes here https://github.com/ricobeck/Bagel.git

@Abanoub-Yassir
Copy link

@yagiz please accept this so we can use your library by SPM

@watanak
Copy link

watanak commented Oct 30, 2023

why I still got error with Bagel/BagelBaseModel.h' file not found when I try to build it? Please help me a bit.

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