Skip to content

conan: experimental package support #588

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

conan: experimental package support #588

wants to merge 1 commit into from

Conversation

@Juff-Ma
Copy link
Contributor

@Juff-Ma Juff-Ma commented Apr 17, 2025

So, this adds the talked about package support for conan and in theory this should work.
However, when I tried using the package it spit out linker errors (tried with tls=True and shared=True, it couldn't find the used functions).
I'm not an expert in conan packaging so maybe the package itself is fine and I did something wrong.

I tried it using the following commands (files below):

  1. conan create . (in webui repo, everything else in the test folder)
  2. conan install . --build="webui/2.5.0-beta.4" -of conan
  3. cmake -B build -S . -DCMAKE_TOOLCHAIN_FILE=conan/conan_toolchain.cmake
  4. cmake --build build

main.c:

#include <webui.h>

int main() {

	size_t my_window = webui_new_window();
	webui_show(my_window, "<html><head><script src=\"webui.js\"></script></head> Hello World ! </html>");
	webui_wait();
	return 0;
}

CMakeLists.txt:

cmake_minimum_required(VERSION 3.15)

project(Test)

add_executable(Test
    main.c)

find_package(webui REQUIRED)

include_directories(${webui_INCLUDE_DIRS})

target_link_libraries(Test
    PUBLIC webui::webui)

install(TARGETS Test)

conanfile.txt:

[requires]
webui/2.5.0-beta.4

[generators]
CMakeToolchain
CMakeDeps

[options]
webui/*:shared=True
webui/*:tls=True

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Juff-Ma