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

[package] open62541/1.4.6: Unable to build package with option discovery=multicast (pro-mdnsd) #26471

Open
steinerthomas opened this issue Jan 27, 2025 · 4 comments · May be fixed by #26484
Open
Assignees
Labels
bug Something isn't working

Comments

@steinerthomas
Copy link
Contributor

Description

With PR #20892 (commit 606f757) the patches for finding cmake deps where removed and no longer maintained for new versions.

This change prevents building open62541 with option discovery=multicast:

CMake Error: File /home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/src/deps/mdnsd/libmdnsd/mdnsd_config_extra.in does not exist.
CMake Error at CMakeLists.txt:764 (configure_file):
  configure_file Problem configuring file


CMake Error at CMakeLists.txt:766 (file):
  file failed to open for reading (No such file or directory):

    /home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/src_generated/mdnsd_config_extra

@Kischy what was the reason to remove these patches? Am I missing something?
Our currently forked version has patches for fixing the mdnsd dependecy applied.

Package and Environment Details

  • Package Name/Version: open62541/1.4.6
  • Operating System+version: Debian GNU/Linux 12
  • Compiler+version: GCC 12
  • Conan version: conan 1.64.0
  • Python version: Python 3.11.2

Conan profile

[settings]
os=Linux
os_build=Linux
arch=x86_64
arch_build=x86_64
compiler=gcc
compiler.version=12
compiler.libcxx=libstdc++
build_type=Release
[options]
[conf]
[build_requires]
[env]

Steps to reproduce

conan create recipes/open62541/all open62541/1.4.6@ -tf None -o discovery=multicast --build=missing

Logs

Click to expand log
WARN: *** Conan 1 is legacy and on a deprecation path ***
WARN: *** Please upgrade to Conan 2 ***
Exporting package recipe
open62541/1.4.6 exports: File 'conandata.yml' found. Exporting it...
open62541/1.4.6 exports: Copied 2 '.yml' files: conandata.yml, submoduledata.yml
open62541/1.4.6: Calling export_sources()
open62541/1.4.6: The stored package has not changed
open62541/1.4.6: Using the exported files summary hash as the recipe revision: c284ac2f6b872cc6c03d3319cd9a70e7
open62541/1.4.6: Exported revision: c284ac2f6b872cc6c03d3319cd9a70e7
Configuration:
[settings]
arch=x86_64
arch_build=x86_64
build_type=Release
compiler=gcc
compiler.libcxx=libstdc++
compiler.version=12
os=Linux
os_build=Linux
[options]
discovery=multicast
[build_requires]
[env]

Installing package: open62541/1.4.6
Requirements
    open62541/1.4.6 from local cache - Cache
    pro-mdnsd/0.8.4 from 'cc-conan-local' - Cache
Packages
    open62541/1.4.6:c624e68eacba1cb65f630ac4aa4b9b2bfd04284b - Build
    pro-mdnsd/0.8.4:2a19826344ff00be1c04403f2f8e7008ed3a7cc6 - Build

Installing (downloading, building) binaries...
pro-mdnsd/0.8.4: Copying sources to build folder
pro-mdnsd/0.8.4: Building your package in /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6
pro-mdnsd/0.8.4: Generator txt created conanbuildinfo.txt
pro-mdnsd/0.8.4: Calling generate()
pro-mdnsd/0.8.4: WARN: Using the new toolchains and generators without specifying a build profile (e.g: -pr:b=default) is discouraged and might cause failures and unexpected behavior
pro-mdnsd/0.8.4: Preset 'release' added to CMakePresets.json. Invoke it manually using 'cmake --preset release'
pro-mdnsd/0.8.4: If your CMake version is not compatible with CMakePresets (<3.19) call cmake like: 'cmake <path> -G "Unix Makefiles" -DCMAKE_TOOLCHAIN_FILE=/home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/build/Release/generators/conan_toolchain.cmake -DCMAKE_POLICY_DEFAULT_CMP0091=NEW -DCMAKE_BUILD_TYPE=Release'
pro-mdnsd/0.8.4: Aggregating env generators
pro-mdnsd/0.8.4: Calling build()
pro-mdnsd/0.8.4: CMake command: cmake -G "Unix Makefiles" -DCMAKE_TOOLCHAIN_FILE="/home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/build/Release/generators/conan_toolchain.cmake" -DCMAKE_INSTALL_PREFIX="/home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6" -DCMAKE_POLICY_DEFAULT_CMP0091="NEW" -DCMAKE_BUILD_TYPE="Release" "/home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/src"
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 2.8.12 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.


-- Using Conan toolchain: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/build/Release/generators/conan_toolchain.cmake
-- Conan toolchain: Setting CMAKE_POSITION_INDEPENDENT_CODE=ON (options.fPIC)
-- Conan toolchain: Setting BUILD_SHARED_LIBS = OFF
-- The C compiler identification is GNU 12.2.0
-- The CXX compiler identification is GNU 12.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Deprecation Warning at libmdnsd/CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 2.8.12 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.


-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY - Success
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY - Success
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR - Success
-- Configuring done
CMake Warning (dev) at libmdnsd/CMakeLists.txt:77 (add_library):
  Policy CMP0063 is not set: Honor visibility properties for all target
  types.  Run "cmake --help-policy CMP0063" for policy details.  Use the
  cmake_policy command to set the policy and suppress this warning.

  Target "libmdnsd" of type "STATIC_LIBRARY" has the following visibility
  properties set for C:

    C_VISIBILITY_PRESET

  For compatibility CMake is not honoring them for this target.
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Generating done
-- Build files have been written to: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/build/Release
pro-mdnsd/0.8.4: CMake command: cmake --build "/home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/build/Release" '--' '-j28'
Scanning dependencies of target libmdnsd
[ 11%] Building C object libmdnsd/CMakeFiles/libmdnsd.dir/mdnsd.c.o
[ 22%] Building C object libmdnsd/CMakeFiles/libmdnsd.dir/xht.c.o
[ 33%] Building C object libmdnsd/CMakeFiles/libmdnsd.dir/1035.c.o
[ 44%] Building C object libmdnsd/CMakeFiles/libmdnsd.dir/sdtxt.c.o
[ 55%] Linking C static library libmdnsd.a
[ 55%] Built target libmdnsd
Scanning dependencies of target mquery
Scanning dependencies of target mdnsd
[ 66%] Building C object CMakeFiles/mquery.dir/mquery.c.o
[ 77%] Building C object CMakeFiles/mdnsd.dir/mdnsd.c.o
[ 88%] Linking C executable mquery
[100%] Linking C executable mdnsd
[100%] Built target mquery
[100%] Built target mdnsd
pro-mdnsd/0.8.4: Package '2a19826344ff00be1c04403f2f8e7008ed3a7cc6' built
pro-mdnsd/0.8.4: Build folder /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/build/Release
pro-mdnsd/0.8.4: Generated conaninfo.txt
pro-mdnsd/0.8.4: Generated conanbuildinfo.txt
pro-mdnsd/0.8.4: Generating the package
pro-mdnsd/0.8.4: Package folder /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6
pro-mdnsd/0.8.4: Calling package()
pro-mdnsd/0.8.4: Copied 1 file: LICENSE
pro-mdnsd/0.8.4: CMake command: cmake --install "/home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/build/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/build/Release" --prefix "/home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6"
-- Install configuration: "Release"
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/bin/mdnsd
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/bin/mquery
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/share/licenses/mdnsd/LICENSE
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/lib/cmake/mdnsd/mdnsdConfigTargets.cmake
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/lib/cmake/mdnsd/mdnsdConfigTargets-release.cmake
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/lib/cmake/mdnsd/mdnsdConfig.cmake
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/lib/libmdnsd.a
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/include/libmdnsd/mdnsd.h
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/include/libmdnsd/1035.h
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/include/libmdnsd/sdtxt.h
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/include/libmdnsd/xht.h
-- Installing: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/include/libmdnsd/mdnsd_config.h
-- Up-to-date: /home/thomas/.conan/data/pro-mdnsd/0.8.4/_/_/package/2a19826344ff00be1c04403f2f8e7008ed3a7cc6/include/libmdnsd/mdnsd_config.h
pro-mdnsd/0.8.4 package(): Packaged 1 '.a' file: libmdnsd.a
pro-mdnsd/0.8.4 package(): Packaged 1 '.cmake' file: conan-official-pro-mdnsd-targets.cmake
pro-mdnsd/0.8.4 package(): Packaged 3 files: LICENSE, mdnsd, mquery
pro-mdnsd/0.8.4 package(): Packaged 5 '.h' files
pro-mdnsd/0.8.4: Package '2a19826344ff00be1c04403f2f8e7008ed3a7cc6' created
pro-mdnsd/0.8.4: Created package revision 593aa5ee26416afb994d9d93b07765d1
open62541/1.4.6: Copying sources to build folder
open62541/1.4.6: Building your package in /home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b
open62541/1.4.6: Generator txt created conanbuildinfo.txt
open62541/1.4.6: Calling generate()
open62541/1.4.6: WARN: Using the new toolchains and generators without specifying a build profile (e.g: -pr:b=default) is discouraged and might cause failures and unexpected behavior
open62541/1.4.6: Preset 'release' added to CMakePresets.json. Invoke it manually using 'cmake --preset release'
open62541/1.4.6: If your CMake version is not compatible with CMakePresets (<3.19) call cmake like: 'cmake <path> -G "Unix Makefiles" -DCMAKE_TOOLCHAIN_FILE=/home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/generators/conan_toolchain.cmake -DCMAKE_POLICY_DEFAULT_CMP0077=NEW -DCMAKE_POLICY_DEFAULT_CMP0091=NEW -DCMAKE_BUILD_TYPE=Release'
open62541/1.4.6: WARN: Using the new toolchains and generators without specifying a build profile (e.g: -pr:b=default) is discouraged and might cause failures and unexpected behavior
open62541/1.4.6: Aggregating env generators
open62541/1.4.6: Calling build()
open62541/1.4.6: Apply patch (conan): Disable static code analysis
open62541/1.4.6: Apply patch (conan): Include iphlpapi.h for SecureZeroMemory
open62541/1.4.6: CMake command: cmake -G "Unix Makefiles" -DCMAKE_TOOLCHAIN_FILE="/home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/generators/conan_toolchain.cmake" -DCMAKE_INSTALL_PREFIX="/home/thomas/.conan/data/open62541/1.4.6/_/_/package/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b" -DCMAKE_POLICY_DEFAULT_CMP0077="NEW" -DCMAKE_POLICY_DEFAULT_CMP0091="NEW" -DCMAKE_BUILD_TYPE="Release" "/home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/src"
-- Using Conan toolchain: /home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/generators/conan_toolchain.cmake
-- Conan toolchain: Setting CMAKE_POSITION_INDEPENDENT_CODE=ON (options.fPIC)
-- Conan toolchain: Setting BUILD_SHARED_LIBS = OFF
-- The C compiler identification is GNU 12.2.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Found Python3: /usr/bin/python3 (found version "3.11.2") found components: Interpreter
-- Found Git: /usr/bin/git (found version "2.39.2")
fatal: not a git repository (or any of the parent directories): .git
CMake Warning at tools/cmake/SetGitBasedVersion.cmake:23 (message):
  Failed to determine the version from git information.  Using defaults.
Call Stack (most recent call first):
  CMakeLists.txt:49 (set_open62541_version)


-- open62541 Version: v1.4.6-undefined
-- The selected architecture is: posix
-- Test CC flag -std=c99
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -pipe
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wall
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wextra
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wpedantic
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wno-static-in-inline
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wno-overlength-strings
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wno-unused-parameter
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wno-maybe-uninitialized
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wc++-compat
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wformat
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wformat-security
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wformat-nonliteral
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wmissing-prototypes
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wstrict-prototypes
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wredundant-decls
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wuninitialized
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Winit-self
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wcast-qual
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wstrict-overflow
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wnested-externs
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wmultichar
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -Wundef
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fno-strict-aliasing
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fexceptions
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fstack-protector-strong
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fstack-clash-protection
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -mcet
-- Performing Test flag_supported
-- Performing Test flag_supported - Failed
-- Test CC flag -fcf-protection
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -D_FORTIFY_SOURCE=2
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -ffunction-sections
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fdata-sections
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fno-unwind-tables
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fno-asynchronous-unwind-tables
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
-- Test CC flag -fno-math-errno
-- Performing Test flag_supported
-- Performing Test flag_supported - Success
CMake Error: File /home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/src/deps/mdnsd/libmdnsd/mdnsd_config_extra.in does not exist.
CMake Error at CMakeLists.txt:764 (configure_file):
  configure_file Problem configuring file


CMake Error at CMakeLists.txt:766 (file):
  file failed to open for reading (No such file or directory):

    /home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/src_generated/mdnsd_config_extra


-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_VISIBILITY - Success
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY
-- Performing Test COMPILER_HAS_HIDDEN_INLINE_VISIBILITY - Failed
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR
-- Performing Test COMPILER_HAS_DEPRECATED_ATTR - Success
-- Could NOT find Sphinx (missing: SPHINX_EXECUTABLE)
-- Could NOT find LATEX (missing: LATEX_COMPILER)
-- Configuring incomplete, errors occurred!
See also "/home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/CMakeFiles/CMakeOutput.log".
See also "/home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/CMakeFiles/CMakeError.log".
open62541/1.4.6:
open62541/1.4.6: ERROR: Package 'c624e68eacba1cb65f630ac4aa4b9b2bfd04284b' build failed
open62541/1.4.6: WARN: Build folder /home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release
ERROR: open62541/1.4.6: Error in build() method, line 372
        cmake.configure()
        ConanException: Error 1 while executing cmake -G "Unix Makefiles" -DCMAKE_TOOLCHAIN_FILE="/home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/build/Release/generators/conan_toolchain.cmake" -DCMAKE_INSTALL_PREFIX="/home/thomas/.conan/data/open62541/1.4.6/_/_/package/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b" -DCMAKE_POLICY_DEFAULT_CMP0077="NEW" -DCMAKE_POLICY_DEFAULT_CMP0091="NEW" -DCMAKE_BUILD_TYPE="Release" "/home/thomas/.conan/data/open62541/1.4.6/_/_/build/c624e68eacba1cb65f630ac4aa4b9b2bfd04284b/src"
@steinerthomas steinerthomas added the bug Something isn't working label Jan 27, 2025
@Kischy
Copy link
Contributor

Kischy commented Jan 27, 2025

@steinerthomas I don't remember the reason I deleted this. I can't think of a reason from the top of my hat. If you fix this maybe add a test using option discovery=multicast, so it does not break in the future

@steinerthomas
Copy link
Contributor Author

steinerthomas commented Jan 30, 2025

Improvement in open62541 for future versions? open62541/open62541#7060

@uilianries
Copy link
Member

For a better understanding of this case:

  • Open62541 has the CMake option UA_ENABLE_DISCOVERY_MULTICAST which reflects the Conan option discovery=multicast;
  • Once UA_ENABLE_DISCOVERY_MULTICAST is active, Open62541 expects mdnsd (pro-mdnsd) to be installed, via git submodules, in the folder deps/mdnsd;
  • No patch was added for 1.4.6, so the version 1.4.6 never worked with the multicast option;
  • The PR Open62541 - Allow more clang versions #20892 only affected 1.3.4 and older versions;

The previous patch can be re-added in order to use mdnsd from Conan as external dependency. In case it was a vendorized dependency (whole copy of the project in deps/ instead of gitmodules) would not be a problem and I would not suggest splitting by using Conan (that's not the case).

The same previous patch is adapting some dependency variables like OPENSSL_LIBRARIES to OpenSSL_LIBRARIES. Those variables can now be mitigated by using the CMakeDeps property cmake_additional_variables_prefixes, and it's already in the recipe, see MBEDTLS fix.

@steinerthomas I see you are still using Conan 1.x and only few CMakeDeps properties are available, not including cmake_additional_variables_prefixes. You may not be broken now by luck, because the encryption option is False by default, but in case using mbedtls, it will break for you. I strongly recommend you move to Conan 2.x asap, because Conan 1.x is no longer supported in CCI.

Still, thank you for reporting this case and taking your time by asking the upstream for an alternative regarding usage.

@steinerthomas
Copy link
Contributor Author

Thank you for improving my pr and describing the problem in detail!

I'm going to move to conan2 soon. Bumping all dependecies to conan1-frozen and cleanup manual patches in recipes was the first step for the migration.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants