arayq2 commented on PR #21: URL: https://github.com/apache/activemq-cpp/pull/21#issuecomment-2692395638
I've been looking at these various config and build related files, and I agree that they need work. For example, I've checked the two README files and determined that the one at the distro root is obsolete. We should probably nuke it lest it cause confusion, and convert the other one to .md format. There's stuff in configure.ac that looks iffy, such as the PLAT_CXXFLAGS setting for GCC outside the case statement. (Does that mean that GCC is not used at all on those platforms?) I think it should be removed as well. And then there is the business of cppunit-config being obsolete and missing on modern systems. pkg-config is neither a drop-in replacement, nor is it universal (I believe it is Debian/Ubuntu specific). My shell script is a terrible hack but right no i can't think of a good way to automate that part. -- 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: gitbox-unsubscr...@activemq.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@activemq.apache.org For additional commands, e-mail: gitbox-h...@activemq.apache.org For further information, visit: https://activemq.apache.org/contact
