PragmaTwice commented on issue #575:
URL: 
https://github.com/apache/incubator-kvrocks/issues/575#issuecomment-1125220948

   > > > The problem is caused by that `-static-libstdc++` is added as a flag 
to the compiler in 
[CMakeLists.txt#L72](https://github.com/apache/incubator-kvrocks/blob/unstable/CMakeLists.txt#L72),
 which is not added in makefiles, so a static library of libstdc++ is needed.
   > > 
   > > 
   > > Maybe some tips should be added to README to let users know that kvrocks 
need libstdc++.a for build. @git-hulk
   > 
   > Yes, you are right. Do you think it's a good idea to search library first 
before linking?
   
   I think it is a good idea, there are two solutions for searching library:
   1. find library, and give an error or warning if not found; give an option 
to user of whether `-static-libstdc++` is enabled
   2. find library, do not enable `-static-libstdc++` if not found
   
   I prefer the first solution, because I think the second is more implicit, 
users may not know whether `-static-libstdc++` is enabled while they are 
building the project (it is totally depend on the environment).


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to