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

bmf: add package #4286

Draft
wants to merge 8 commits into
base: dev
Choose a base branch
from
Draft

bmf: add package #4286

wants to merge 8 commits into from

Conversation

star-hengxing
Copy link
Contributor

No description provided.

@KkemChen
Copy link
Contributor

插眼

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


insert eye

@KkemChen
Copy link
Contributor

_ No description provided. _

催更!😏

@KkemChen
Copy link
Contributor

KkemChen commented Jul 2, 2024

怠惰的⭐️🐎

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


Lazy ⭐️🐎

@star-hengxing
Copy link
Contributor Author

我猜 ci 用了最新 spdlog 自带的 fmt,所以挂了。
目前因为 bug xmake-io/xmake#4486 ,下面写法会直接报错。

add_deps("spdlog", {configs = {header_only = false, fmt_external = true}})
add_deps("fmt <=8.0.0", "dlpack", "backward-cpp")

这边你自己本地改包描述调试比较好,或者考虑用它的 submodule,不进行拆依赖

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


I guess ci used the fmt that comes with the latest spdlog, so it hung up.
Currently, due to bug xmake-io/xmake#4486, the following writing method will directly report an error.

add_deps("spdlog", {configs = {header_only = false, fmt_external = true}})
add_deps("fmt <=8.0.0", "dlpack", "backward-cpp")

It is better to modify the package description and debug locally, or consider using its submodule without removing dependencies.

@KkemChen
Copy link
Contributor

KkemChen commented Jul 3, 2024

我猜 ci 用了最新 spdlog 自带的 fmt,所以挂了。 目前因为 bug xmake-io/xmake#4486 ,下面写法会直接报错。

add_deps("spdlog", {configs = {header_only = false, fmt_external = true}})
add_deps("fmt <=8.0.0", "dlpack", "backward-cpp")

这边你自己本地改包描述调试比较好,或者考虑用它的 submodule,不进行拆依赖

👌ok

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


I guess ci uses the fmt that comes with the latest spdlog, so it hangs. Currently, due to the bug xmake-io/xmake#4486, the following writing method will directly report an error.

``lua
add_deps("spdlog", {configs = {header_only = false, fmt_external = true}})
add_deps("fmt <=8.0.0", "dlpack", "backward-cpp")


It is better to modify the package description and debug locally, or consider using its submodule without removing dependencies.

👌ok

@KkemChen
Copy link
Contributor

我猜 ci 用了最新 spdlog 自带的 fmt,所以挂了。 目前因为 bug xmake-io/xmake#4486 ,下面写法会直接报错。

add_deps("spdlog", {configs = {header_only = false, fmt_external = true}})
add_deps("fmt <=8.0.0", "dlpack", "backward-cpp")

这边你自己本地改包描述调试比较好,或者考虑用它的 submodule,不进行拆依赖

👌ok

话说这个bug,近期内会修复🐎

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


I guess ci used the fmt that comes with the latest spdlog, so it hung up. Currently, due to the bug xmake-io/xmake#4486, the following writing method will directly report an error.

add_deps("spdlog", {configs = {header_only = false, fmt_external = true}})
add_deps("fmt <=8.0.0", "dlpack", "backward-cpp")

It is better to modify the package description and debug locally, or consider using its submodule without removing dependencies.

👌ok

By the way, this bug will be fixed in the near future🐎

@star-hengxing
Copy link
Contributor Author

没时间修。。不过刚刚搞了个 patch,Windows 上跑通编译了
bmf-xmake.patch

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


No time to fix it. . But I just made a patch and it ran smoothly and compiled on Windows.
bmf-xmake.patch

@KkemChen
Copy link
Contributor

image
难蚌,我连linux都编不过💦

@star-hengxing
Copy link
Contributor Author

我试一下适配最新 fmt 版本

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


I'll try to adapt to the latest fmt version

@KkemChen
Copy link
Contributor

KkemChen commented Jul 23, 2024

🌟🐎 yyds, 大半夜的还在pr, 给您磕头了,砰砰砰

@star-hengxing
Copy link
Contributor Author

基于我的 patch,linux 和 windows 平台编译没问题了,你可以本地测试 ffmpeg/cuda configs 有没有问题

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


Based on my patch, there is no problem compiling on Linux and Windows platforms. You can test ffmpeg/cuda configs locally to see if there are any problems.

@KkemChen
Copy link
Contributor

基于我的 patch,linux 和 windows 平台编译没问题了,你可以本地测试 ffmpeg/cuda configs 有没有问题

👌 🌟🐎yyds

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


Based on my patch, there is no problem compiling on Linux and Windows platforms. You can test ffmpeg/cuda configs locally to see if there are any problems.

👌 🌟🐎yyds

@KkemChen
Copy link
Contributor

image

我介是嘛情况 🤔

@KkemChen
Copy link
Contributor

image

我把这个check注掉试了一下 好像还是fmt 不太对劲

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


image

I checked out the check and tried it, but it still seems that something is not right with fmt.

@star-hengxing
Copy link
Contributor Author

check

这个是新 api,更新 xmake 就行

fmt

把 bmf 所有 deps 全删了,然后再跑一遍看看,我估计是沿用之前的缓存了。反正 ci 上已经过了,应该没啥大问题

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


check

This is a new API, just update xmake

fmt

Delete all deps in bmf and run it again. I guess the previous cache is still used. Anyway, it has been passed on ci, so there should be no big problem.

@KkemChen
Copy link
Contributor

check

这个是新 api,更新 xmake 就行

fmt

把 bmf 所有 deps 全删了,然后再跑一遍看看,我估计是沿用之前的缓存了。反正 ci 上已经过了,应该没啥大问题

image

我这玩意是 system的 怎么删🤔

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


check

This is a new API, just update xmake

fmt

Delete all deps in bmf and run it again. I guess the previous cache is used. Anyway, it has been passed on ci, so there should be no big problem.

image

This thing belongs to system. How do I delete it?

@KkemChen
Copy link
Contributor

我都改成 system=false了,还是在这里报错
image

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


I have changed it to system=false, but I still get an error here.
image

@star-hengxing
Copy link
Contributor Author

里面的 fmt 依然没有禁用系统包
试一下用 xmake g 设置禁用系统库 policy

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


The fmt inside still does not disable system packages.
Try using xmake g to set the disable system library policy

@KkemChen
Copy link
Contributor

image

奶奶滴,开了 ffmpeg,这破壁fmt又双叒叕报错了

@KkemChen
Copy link
Contributor

🌟🐎,这个开了ffmpeg的config 它fmt又双叒叕寄了 ,该咋整

@Issues-translate-bot
Copy link

Bot detected the issue body's language is not English, translate it automatically.


🌟🐎, I opened the ffmpeg config and fmt was sent twice. What should I do?

@waruqi
Copy link
Member

waruqi commented Jul 31, 2024

note: the following packages are unsupported on windows/arm64:
-> backward-cpp v1.6

提示不是很明显么。。要么改 backward-cpp 要么禁 arm64

set_description("Cross-platform, customizable multimedia/video processing framework. With strong GPU acceleration, heterogeneous design, multi-language support, easy to use, multi-framework compatible and high performance, the framework is ideal for transcoding, AI inference, algorithm integration, live video streaming, and more.")
set_license("Apache-2.0")

add_urls("https://github.com/star-hengxing/bmf/archive/refs/heads/windows-support.zip")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这 url 咋这么怪,后面不会自动更新 break hash 么?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个库目前很多问题,需要手动改大量代码

@star-hengxing star-hengxing marked this pull request as draft July 31, 2024 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants