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

ARROW-3860: [WIP][Gandiva][C++] Link std c++ statically. #3020

Closed
wants to merge 1 commit into from

Conversation

praveenbingo
Copy link
Contributor

Restores linking statically to std c++, so that it can be safely packaged for other
similar environments.

Restores linking statically to std c++, so that it can be safely packaged for other
similar environments.
@praveenbingo praveenbingo changed the title ARROW-3860: [Gandiva][C++] Link std c++ statically. ARROW-3860: [WIP][Gandiva][C++] Link std c++ statically. Nov 23, 2018
@pitrou
Copy link
Member

pitrou commented Nov 23, 2018

I'm not sure we want to do that by default. @wesm

@wesm
Copy link
Member

wesm commented Nov 23, 2018

-1. Use CMAKE_SHARED_LINKER_FLAGS when invoking CMake

@wesm wesm closed this Nov 23, 2018
@jacques-n
Copy link
Contributor

Hey @wesm, apparently the changes these are trying to revert to Gandiva mean we've broken the downstream Dremio builds. We should try to avoid making changes that break downstream builds without alternative paths already in place. It seems like the original change this is reverting broke a pattern without a proper alternative available.

@wesm
Copy link
Member

wesm commented Nov 26, 2018

I hear you -- this issue is a bit distressing as the hard-coding of linker flags specific to Dremio builds was causing problems for other Arrow developers. I asked multiple times and waited over a month before fixing the problem myself

Here is the fix on November 19: ba2b2ea#diff-55d595f15416efb8fd2b661f37cb55b7

We have a problem of process right now in that people like me are not able to verify changes that are affecting Dremio builds -- had I been able to, I would have been glad to sort out a remediation. Short of that, I am not sure how else I would have been able to force action on this issue. As you can see in the discussion on https://issues.apache.org/jira/browse/ARROW-3860, there is a lack of transparency into your process that makes it difficult for me to help.

@wesm
Copy link
Member

wesm commented Nov 26, 2018

If you want to revert ba2b2ea temporarily until you figure out the packaging stuff, it's fine with me, but let's please not let it linger much longer since I would like to get the project into release-ready shape in the next 2-3 weeks

@wesm
Copy link
Member

wesm commented Nov 26, 2018

I think #3035 will address the issue

@jacques-n
Copy link
Contributor

Hey @wesm , I appreciate your attention and detailed response. I've opened up https://issues.apache.org/jira/browse/ARROW-3887 and will have our team work on adding that. There is no desire to be opaque. Will work with the team on becoming more transparent as well.

@praveenbingo
Copy link
Contributor Author

Thanks a ton Wes and Jacques for taking this to closure.

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

Successfully merging this pull request may close these issues.

4 participants