-
Notifications
You must be signed in to change notification settings - Fork 2.8k
local_file_system.cpp: minor fix for macOS libproc code #10758
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
Conversation
Recent commit has broken duckdb build on macOS < 10.7: duckdb@23112e9 Add a fallback to fix it. See also get_ppid_of.c in gettext’s libtextstyle.
16acae0 to
46d2fc5
Compare
|
Thanks for the PR! Hardcoding |
|
@Mytherin If you or someone is sure that gonna work, we could. Looks like it was originally introduced that way: coreutils/gnulib@015fe7c P. S. Provided this is correct for specific macOS versions (< 10.7), it is safe – or can be made so by extra macros specifying macOS versions explicitly. As long as the fallback sits inside, it cannot affect anything else (if that is your concern). |
|
That's fair - but since we are not running tests on older MacOS versions I would be more comfortable just returning |
@Mytherin Okay, sure, let’s do that. Could you either add a commit here or make a code suggestion for me to commit? |
|
Done - could you check if this resolves the issue for you on the older MacOS box? I don't have access to one. |
|
@Mytherin Thank you! Will confirm tomorrow once back to the machine. |
|
@Mytherin I cannot run tests at the moment, but yes, this works to fix the build, so all good for now. |
Merge pull request duckdb/duckdb#10758 from barracuda156/fix_macos
Recent 23112e9 commit has broken
duckdbbuild on macOS < 10.7.Add a fallback to fix it.
As an example of a similar code see get_ppid_of.c in gettext’s libtextstyle.