doc: add import from conan #277
Conversation
doc/conan.md
Outdated
|
|
||
| ### 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"
)
func main() {
names := make(map[string][]string)
dirs, err := os.ReadDir("./recipes")
if err != nil {
panic(err)
}
for _, pkg := range dirs {
if pkg.IsDir() {
pkgnoLib := strings.TrimPrefix(pkg.Name(), "lib")
names[pkgnoLib] = append(names[pkgnoLib], pkg.Name())
}
}
for name, pkgs := range names {
if len(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.
MeteorsLiu
force-pushed
the
doc/import-from-conan
branch
from
4f3d04a to
bad5f31
Compare
|
|
||
| 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. |
|
order by popular |
|
[Git-flow] Hi @MeteorsLiu, There are some suggestions for your information: Rebase suggestions
Which seems insignificant, recommend to use For other If you have any questions about this comment, feel free to raise an issue here: |
2 participants
No description provided.