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

bump 1.2.0 #14

Closed
wants to merge 24 commits into from
Closed

bump 1.2.0 #14

wants to merge 24 commits into from

Conversation

xxchan
Copy link
Member

@xxchan xxchan commented Sep 19, 2023

No description provided.

@xxchan xxchan added the pr-pull label Sep 19, 2023
@xxchan xxchan changed the title bump bump 1.2.0 Sep 19, 2023
@xxchan xxchan removed the pr-pull label Sep 19, 2023
@xxchan
Copy link
Member Author

xxchan commented Oct 11, 2023

wtf

image

@BugenZhao
Copy link
Member

@BugenZhao
Copy link
Member

OpenSSL and rdkafka is really a nightmare...

@xxchan
Copy link
Member Author

xxchan commented Oct 11, 2023

OpenSSL and rdkafka is really a nightmare...

I think this isn't their fault. More probably apple's. 🤡

@xxchan
Copy link
Member Author

xxchan commented Oct 11, 2023

Because of this?

image

@xxchan
Copy link
Member Author

xxchan commented Oct 11, 2023

(x86, failed to build cxx) There's no "-std=c++11" here 🤔️

image image

@xxchan
Copy link
Member Author

xxchan commented Oct 11, 2023

OpenSSL and rdkafka is really a nightmare...

I think this isn't their fault. More probably apple's. 🤡

Or homebrew's 🤡

@github-actions
Copy link

@xxchan bottle request failed.

@github-actions
Copy link

@xxchan bottle request failed.

1 similar comment
@github-actions
Copy link

@xxchan bottle request failed.

@xxchan
Copy link
Member Author

xxchan commented Oct 14, 2023

Homebrew/brew#7248 This looks very similar

because compiling #include <stdlib.h> produces many warnings.

PostgreSQL's configure script adds Xcode SDK root -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX.sdk/usr/include
Xcode SDK root is printed by xml2-config --cflags, which runs xcrun --show-sdk-path
Homebrew adds CLT SDK root -isysroot/Library/Developer/CommandLineTools/SDKs/MacOSX10.15.sdk

@xxchan
Copy link
Member Author

xxchan commented Oct 14, 2023

Homebrew/homebrew-core#51949 (comment)

This sounds exactly the root of the problem

You might have issues if is CMake build is not set up to use std_cmake_args.

CMake is somewhat an unique build tool when it comes to the macOS SDK. In particular, if CMAKE_OSX_SYSROOT is not set then CMake makes its own choice of a SDK to use rather than rely on the compiler to handle it (which Homebrew shims). This can generally lead to bad things as Homebrew will still try to use its SDK choice through the compiler shims which means there is now a couple of different SDKs in the include path, and the ordering is likely all wrong too.

If passing std_cmake_args is not possible for some reason, then you can try just -DCMAKE_OSX_SYSROOT, or perhaps setting the SDKROOT env. If the bootstrap doesn’t cope well with either (which would warrant a bug report IMO) then unsetting HOMEBREW_SDKROOT should be safe to do here. It wouldn’t be for something like Python which has a python3_config binary.

Comment on lines 45 to 70

- name: Failures Summary and Bottle Result
if: always()
uses: Homebrew/actions/failures-summary-and-bottle-result@master
with:
workdir: ${{ github.workspace }}
result_path: bottles/steps_output.txt
step_name: 'Build failure summary on ${{ matrix.os }}'
collapse: 'true'

- name: Upload bottles as artifact
if: always() && github.event_name == 'pull_request'
uses: actions/upload-artifact@main
with:
name: bottles
path: '*.bottle.*'

- name: Post comment
uses: Homebrew/actions/post-comment@master
with:
token: ${{ github.token }}
issue: ${{ github.event.number }}
body: "@${{github.actor}} bottle request for ${{ runner.os }} (${{github.event.repository.html_url}}/actions/runs/${{github.run_id}})."
bot_body: "Bottle request for ${{ runner.os }} (${{github.event.repository.html_url}}/actions/runs/${{github.run_id}})."
bot: BrewTestBot

Copy link
Member Author

Choose a reason for hiding this comment

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

These are a little bit useful, but I still don't know how to let the CI fail

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

@BugenZhao
Copy link
Member

What an acute insight you have!

@github-actions
Copy link

@github-actions
Copy link

@github-actions
Copy link

1 similar comment
@github-actions
Copy link

@xxchan
Copy link
Member Author

xxchan commented Oct 15, 2023

macos-12 failed again... too much headache 🤡

image

@xxchan xxchan added the pr-pull label Oct 15, 2023
@github-actions github-actions bot closed this in ad7b028 Oct 15, 2023
@github-actions github-actions bot deleted the xxchan/relieved-baboon branch October 15, 2023 03:46
@BugenZhao
Copy link
Member

Why we get no Monterey 🤔

image

@xxchan
Copy link
Member Author

xxchan commented Oct 15, 2023

Because macos-12 failed and I’m too lazy to fix it. 🤡

@BugenZhao
Copy link
Member

Because macos-12 failed and I’m too lazy to fix it. 🤡

Only Ventura and Sonoma users can take benefits from pre-built bottles and others will suffer from build time of 30 minutes+. I'm okay with that though. 🤣

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants