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

[idevicerestore] Fix libgen.h cannot be found #11074

Merged

Conversation

NancyLi1013
Copy link
Contributor

idevicerestore build error on x86-windows due to libgen.h cannot be found.

1>C:\agent\_work\1\s\buildtrees\idevicerestore\x86-windows-rel\1.0.12-cd5b5777d8\src\common.c(32,10): fatal error C1083: Cannot open include file: 'libgen.h': No such file or directory [C:\agent\_work\1\s\buildtrees\idevicerestore\x86-windows-rel\1.0.12-cd5b5777d8\idevicerestore.vcxproj]
         dfu.c
         download.c
 

Related PR #11003

Note: No feature needs to test.

@NancyLi1013 NancyLi1013 added the info:internal This PR or Issue was filed by the vcpkg team. label Apr 29, 2020
@NancyLi1013 NancyLi1013 marked this pull request as ready for review April 29, 2020 06:45
@LilyWangL
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@LilyWangL LilyWangL added the info:reviewed Pull Request changes follow basic guidelines label Apr 30, 2020
@ras0219-msft
Copy link
Contributor

Why does this PR cause libidevicerestore to fail on linux and osx?

@BillyONeal
Copy link
Member

Indeed, it seems most important for this port to work on MacOS (given that it is a firmware restoration bit for iOS development).

@strega-nil strega-nil self-requested a review May 1, 2020 18:44
Copy link
Contributor

@strega-nil strega-nil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given the concerns pointed out by @ras0219-msft @BillyONeal, this needs more work; could you look at it after your holiday? Thanks!

@NancyLi1013 NancyLi1013 removed the info:reviewed Pull Request changes follow basic guidelines label May 6, 2020
@NancyLi1013
Copy link
Contributor Author

Hi all,

Since idevicerestore depends on libimobiledevice to build. And libimobiledevice depends on libusbmuxd which currently can only build on windows.

idevicerestore should support all platforms. But currently, it cannot build on Linux and OSX. It is built via msbuild.

If we want to support Linux and OSX, we should fix the issue for libusbmuxd first. Then update the way to build idevicerestore.

@strega-nil
Copy link
Contributor

strega-nil commented May 6, 2020

how was it working before, then, on the CI?

@NancyLi1013
Copy link
Contributor Author

I just added vcpkg_fail_port_install(ON_TARGET "linux" "osx") and

idevicerestore:x64-linux=fail
idevicerestore:x64-osx=fail

It should not be working on linux and osx before this, which will be blocked by libusbmuxd.

If necessary, I will remove the messages I added and try to build it again.

@strega-nil
Copy link
Contributor

mmh, okay, I see. Alright, I think this is good then :)

@LilyWangL LilyWangL added the info:reviewed Pull Request changes follow basic guidelines label May 12, 2020
@strega-nil
Copy link
Contributor

Thanks @NancyLi1013 for pushing this forward!

@strega-nil strega-nil merged commit ba7450a into microsoft:master May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:internal This PR or Issue was filed by the vcpkg team. info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants