conan-center-index icon indicating copy to clipboard operation
conan-center-index copied to clipboard

odpi: new recipe

Open inPhraZ opened this issue 11 months ago • 2 comments

Summary

New recipe: odpi/5.4.1

Motivation

Oracle Database Programming Interface for C (ODPI-C) is an open source library of C code that simplifies access to Oracle Database for applications written in C or C++

Upstream repository: https://github.com/oracle/odpi


Maintainer changes

The original PR had only support for Linux, it still requires extra changes in the Makefile in order to use Conan build flags, because it does not allow overriding the compiler and compiler flags. The upstream does not allow PR directly, so I sent an issue with the proposed patch: https://github.com/oracle/odpi/issues/196. Plus, Debian uses a similar patch https://salsa.debian.org/debian/odpic/-/blob/debian/latest/debian/patches/do_not_overwrite_build_env_flags.patch too.


inPhraZ avatar Dec 15 '24 19:12 inPhraZ

Short summary:

  • Shared-library "A sample Makefile is provided if you wish to build ODPI-C as a shared library" https://github.com/oracle/odpi/blob/d97327db237d2efd10905f4339be63ca20f12e38/doc/src/user_guide/installation.rst#L14
  • Add the missing layout folder
  • Validate is no longer required
  • New build_requiremets for Windows support
  • PkgConfigDeps and AutotoolsDeps are not required because the recipe doesn't have any dependencies
  • Windows uses a special Makefile.win32
  • Copy libs dlls and headers on Windows
  • Remove self.cpp_info.set_property("pkg_config_name", "odpi") we can take the recipe name and upstream don't define the pkg_config_name
  • Small change on the test to get some console output
  • Use fix_apple_shared_install_name
  • ~New patch from: https://github.com/oracle/odpi/issues/196~ - Fixed on version 5.6.0

ErniGH avatar Jun 23 '25 10:06 ErniGH

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Jun 23 '25 14:06 CLAassistant