8000 [luajit] Fix build failure that resulted in no output. by BillyONeal · Pull Request #26360 · microsoft/vcpkg · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[luajit] Fix build failure that resulted in no output. #26360

New issue
8000

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 16, 2022
Merged

[luajit] Fix build failure that resulted in no output. #26360

merged 1 commit into from
Aug 16, 2022

Conversation

BillyONeal
Copy link
Member

This fixes the problem reported in:

#25856 (comment)

@BillyONeal BillyONeal added category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team. labels Aug 15, 2022
@BillyONeal BillyONeal marked this pull request as ready for review August 15, 2022 20:54
@BillyONeal BillyONeal mentioned this pull request Aug 15, 2022
Merged
BillyONeal
BillyONeal commented Aug 15, 2022
View reviewed changes
ports/luajit/portfile.cmake
@@ -5,9 +5,8 @@ vcpkg_from_github(
SHA512 0a1d79ab7d2de6894bcff33309e015fdba0ea67cf0425d75b9301a30006039e81b527178dbb3485e1adea177ffe062e6fcef74307f8e725678e70562d57d1a5b
HEAD_REF master
PATCHES
001-fix-build-path.patch
Copy link
Member Author

Choose a reason for hiding this comment

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

I merged these patches because they didn't apply cleanly independently and patching patches is evil

@ghost
Copy link
ghost commented Aug 15, 2022

fixes #26348 ?

8000

@BillyONeal
Copy link
Member Author

fixes #26348 ?

Indeed!

@BillyONeal BillyONeal mentioned this pull request Aug 15, 2022
Closed
beats-dh
beats-dh approved these changes Aug 16, 2022
View reviewed changes
Copy link
@beats-dh beats-dh left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Perfect!

image

@JonLiu1993 JonLiu1993 self-assigned this Aug 16, 2022
@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Aug 16, 2022
@BillyONeal BillyONeal merged commit acc3bcf into microsoft:master Aug 16, 2022
Jimmy-Hu added a commit to Jimmy-Hu/vcpkg that referenced this pull request Aug 16, 2022
@Jimmy-Hu
Merge pull request #1117 from microsoft/master
1306255 
[luajit] Fix build failure that resulted in no output. (microsoft#26360)
@JackBoosY JackBoosY mentioned this pull request Aug 17, 2022
Closed
@BillyONeal BillyONeal mentioned this pull request Aug 17, 2022
Merged
@BillyONeal BillyONeal deleted the luajit branch October 14, 2022 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@github-actions github-actions[bot] github-actions[bot] approved these changes

@beats-dh beats-dh beats-dh approved these changes

@JonLiu1993 JonLiu1993 JonLiu1993 approved these changes

Assignees

@JonLiu1993 JonLiu1993

Labels
category:port-bug The issue is with a library, which is something the port should already support info:internal This PR or Issue was filed by the vcpkg team. info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BillyONeal @beats-dh @JonLiu1993
0