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

[libboinc] fix build when integrator tries to link app with libboinc #5232

Merged

Conversation

AenBleidd
Copy link
Member

Header files that are exposed should not contain any 'config.h' file reference.
By this PR next issue is fixed:

In file included from /home/runner/work/boinc-apps/boinc-apps/build/boinc-autodock-vina/x64-linux-static/vcpkg_installed/x64-linux-static/include/boinc/str_util.h:25,
                 from /home/runner/work/boinc-apps/boinc-apps/build/boinc-autodock-vina/x64-linux-static/vcpkg_installed/x64-linux-static/include/boinc/parse.h:31,
                 from /home/runner/work/boinc-apps/boinc-apps/build/boinc-autodock-vina/x64-linux-static/vcpkg_installed/x64-linux-static/include/boinc/coproc.h:83,
                 from /home/runner/work/boinc-apps/boinc-apps/build/boinc-autodock-vina/x64-linux-static/vcpkg_installed/x64-linux-static/include/boinc/hostinfo.h:32,
                 from /home/runner/work/boinc-apps/boinc-apps/build/boinc-autodock-vina/x64-linux-static/vcpkg_installed/x64-linux-static/include/boinc/app_ipc.h:28,
                 from /home/runner/work/boinc-apps/boinc-apps/build/boinc-autodock-vina/x64-linux-static/vcpkg_installed/x64-linux-static/include/boinc/boinc_api.h:26,
                 from /home/runner/work/boinc-apps/boinc-apps/boinc-autodock-vina/src/boinc-autodock-vina/boinc-autodock-vina.cpp:24:
/home/runner/work/boinc-apps/boinc-apps/build/boinc-autodock-vina/x64-linux-static/vcpkg_installed/x64-linux-static/include/boinc/str_replace.h:28:10: fatal error: config.h: No such file or directory
   28 | #include "config.h"
      |          ^~~~~~~~~~
compilation terminated.

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
AenBleidd added a commit to AenBleidd/vcpkg that referenced this pull request May 10, 2023
Ref: BOINC/boinc#5232
Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
@AenBleidd AenBleidd added this to the Client/Manager 7.24.0 milestone May 10, 2023
@AenBleidd AenBleidd merged commit 68df98e into BOINC:master May 10, 2023
@AenBleidd AenBleidd deleted the vko_fix_build_when_linking_to_libboinc branch May 10, 2023 13:38
JavierMatosD pushed a commit to microsoft/vcpkg that referenced this pull request May 11, 2023
Ref: BOINC/boinc#5232

Signed-off-by: Vitalii Koshura <lestat.de.lionkur@gmail.com>
AenBleidd added a commit that referenced this pull request May 21, 2023
…_libboinc

[libboinc] fix build when integrator tries to link app with libboinc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant