Uh oh!
There was an error while loading. Please reload this page.
doc: add import from conan - #277
Conversation
| ### 1. All `cflags` Included by Default | ||
| The automatically generated `llcpp.cfg` includes all header files from dependencies listed under `cflags`. For example, `libxml2` includes header files from `zlib`, and `libxslt` includes headers from both `libxml2` and `zlib` as part of its include paths. This behavior is unintended — for example, in certain cases, headers from `libxml2` and `zlib` need to be removed. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@## main #277 +/- ##
=======================================
Coverage 80.07% 80.07% =======================================
Files 28 28 Lines 2951 2951 =======================================
Hits 2363 2363 Misses 571 571 Partials 17 17 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| Remove `lib` prefix. | ||
| However, this may cause naming conflicts and needs to be verified. |
There was a problem hiding this comment.
recipes from https://github.com/conan-io/conan-center-index with these verify program,found these package will have conflict
package main
import (
"fmt""os""strings"
)
funcmain() {
names:=make(map[string][]string)
dirs, err:=os.ReadDir("./recipes")
iferr!=nil {
panic(err)
}
for_, pkg:=rangedirs {
ifpkg.IsDir() {
pkgnoLib:=strings.TrimPrefix(pkg.Name(), "lib")
names[pkgnoLib] =append(names[pkgnoLib], pkg.Name())
}
}
forname, pkgs:=rangenames {
iflen(pkgs) >1 {
fmt.Println(name, pkgs)
}
}
}❯ go run main.go
tar [libtar tar]
serial [libserial serial]
vdpau [libvdpau vdpau]
b2 [b2 libb2]
gettext [gettext libgettext]There was a problem hiding this comment.
@MeteorsLiu i open a pr for this verification MeteorsLiu#1, need review
There was a problem hiding this comment.
conanfile.py package_type to filter binary or libary
There was a problem hiding this comment.
There was a problem hiding this comment.
There was a problem hiding this comment.
There was a problem hiding this comment.
There was a problem hiding this comment.
4f3d04a to
bad5f31Compare| The automatically generated `llcppg.cfg` includes all header files from dependencies listed under `cflags`. For example, `libxml2` includes header files from `zlib`, and `libxslt` includes headers from both `libxml2` and `zlib` as part of its include paths. This behavior is unintended — for example, in certain cases, headers from `libxml2` and `zlib` need to be removed. | ||
| **Preliminary solution:** Automatically remove them during dependency population. |
There was a problem hiding this comment.
note:need support custom llpkg
| Batch conversion cannot perform demo validation. | ||
| **Preliminary solution:** Use basic `llgo build` validation. |
luoliwoshang
commented
Apr 28, 2025
order by popular |
[Git-flow] Hi @MeteorsLiu, There are some suggestions for your information: Rebase suggestions
Which seems insignificant, recommend to use For other DetailsIf you have any questions about this comment, feel free to raise an issue here: |
No description provided.