Provide protobuf via conan

MSYS provided protobuf version 3.16 is quite old and causes issues during build of !78 (merged) (see also #128 (closed)). We should upgrade to a newer protobuf version (via conan).

changed milestone to %Release vX.X.X

added Enhancement label

assigned to @rbiegel

changed milestone to %Release v0.9

FYI - I found this Conan Center repo with recipes for protobuf and osi. Does it help? https://github.com/conan-io/conan-center-index/blob/master/recipes/protobuf/all/conanfile.py

For the local use of conan (building openPASS with MSYS and conan), Michael and I decided to disable the conan center, where conan tries to download artifacts instead of builing FMI and OSI. But may we could use conan as pacman alternative for the CI?

And another update.. - I just realized that this (the latest status of conan for openPASS) is on this branch !72 (merged) which was not merged end of 2021.

added Work In Progress label

mentioned in issue #128 (closed)

Solved via !78 (merged)

closed

removed Work In Progress label

I tried to locally build the openPASS v0.9 "release candidate" with Conan, but I get this error: ERROR: OSI/3.3.1: Invalid configuration: WORKSPACE environment variable has to be set

I figured that this WORKSPACE is set in the CI script 15_prepare_thirdParty.sh - which I don't need for building openPASS with Conan. Is there any way to set it manually?

Right now, the OSI recipe only works for the CI

reopened

I have to admit, I only had the shell script CI build in mind when applying recent changes. Never built openPass itself using conan. You should be able to build locally by mimicking the CI environment and using the shell script based build:

• Create a workspace folder (e.g. ~/op-workspace)
• Checkout of repository in repo subfolder (e.g. `~/op-workspace/repo)
• run ~/op-workspace/repo/utils/ci/scripts/run_all.sh

Keep in mind to stick to the folder structure above, when running the ci build scripts! They will wipe some folders adjacent to the repo subfolder (see https://gitlab.eclipse.org/eclipse/simopenpass/simopenpass/-/blob/develop/utils/ci/scripts/10_prepare.sh#L27)

Is there any way to set it manually?

Should work by just setting the WORKSPACE environment variable before running conan?

The WORKSPACE environment variable is currently required to be able to point the OSI conan build to the protobuf installation (outside of the conan profile/cache directories). Otherwise we didn't manage to use unmodified versions of the protobuf recipes. But maybe we have to look into that again.

Do you consider a working openPASS conan build as mandatory for 0.9?

No, a working Conan build is not necessary for 0.9. The supported / suggested way to build openPASS is with Cmake. In the WG, we discussed to list further options, if they are stable and useful.

I will test manually setting the variable, thx

changed milestone to %Release vX.X.X

closed