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

memmem issue on Apple's 10.6 SDK (was: Package the code to vcpkg) #652

Open
msclock opened this issue Mar 2, 2023 · 8 comments
Open

memmem issue on Apple's 10.6 SDK (was: Package the code to vcpkg) #652

msclock opened this issue Mar 2, 2023 · 8 comments

Comments

@msclock
Copy link

msclock commented Mar 2, 2023

Can we package the code to vcpkg?

Are there any guidelines or permits I should follow?

@msclock msclock changed the title Packaging to vcpkg? Package the code to vcpkg? Mar 2, 2023
@rurban
Copy link
Contributor

rurban commented Mar 2, 2023

Sure you can. See the various cmake or mingw autotools recipes.

@msclock
Copy link
Author

msclock commented Mar 3, 2023

I have submit a pr to vcpkg. microsoft/vcpkg#30005.

This is the usage:
https://github.com/msclock/vcpkg/blob/bfa07fd074023fe0b4eb835363d280c80256572e/ports/libredwg/usage#L1-L4

All scripts I configured are based on the cmake build system.

I came across a configuration item that needed to be lowered.

@rurban
Copy link
Contributor

rurban commented Mar 4, 2023

The memmem issue on Apple's 10.6 SDK is already handled by the 2nd case in common.h, just the declaration is missing. There I assume HAVE_MEMMEM is 1, but the decl in string.h is missing.

@msclock
Copy link
Author

msclock commented Mar 4, 2023

The memmem issue on Apple's 10.6 SDK is already handled by the 2nd case in common.h, just the declaration is missing. There I assume HAVE_MEMMEM is 1, but the decl in string.h is missing.

Well, it is. HAVE_MEMMEM was evalated as 1 but it reports implicitly decl error. The assumption is not valid so that I have to set an eternal true condition on Apple.

@rurban
Copy link
Contributor

rurban commented Mar 4, 2023

So I'll try to come up with a better fix

@rurban rurban changed the title Package the code to vcpkg? memmem issue on Apple's 10.6 SDK (was: Package the code to vcpkg) Mar 30, 2023
@rurban
Copy link
Contributor

rurban commented Aug 3, 2023

I've added now a vcpkg.json

@msclock
Copy link
Author

msclock commented Aug 4, 2023

@rurban Got it. I've noticed that with the current quick iteration, maybe we can create our own registry for quick iterative updates, and then push it to the official VCPKG registry when it stabilizes. Please refer to:

I will create a pr implement it a bit later.

@rurban
Copy link
Contributor

rurban commented Aug 4, 2023 via email

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

2 participants