Skip to content

feat: PKGBUILD 打包 - #15

Open
JohnChiao75 wants to merge 5 commits into
ILoveScratch2:mainfrom
JohnChiao75:aur
Open

feat: PKGBUILD 打包#15
JohnChiao75 wants to merge 5 commits into
ILoveScratch2:mainfrom
JohnChiao75:aur

Conversation

@JohnChiao75

Copy link
Copy Markdown
Contributor
  • fix: 将 mediakit 依赖编译到程序中
  • fix: CI 系统库污染产物 → 删除后 media_kit_libs_linux 自动接管捆绑
  • feat: 打包 AUR 包

Checklist / 检查清单

  • I have read and understood the contribution guidelines. / 我已阅读并理解贡献指南。

  • I have signed the CLA. / 我已签署 CLA。

Description / 描述

  • This is a breaking change. / 这是一个重大更改。

  • This change requires a server-side update. / 此更改需要服务器端更新。

  • This change is compatible with the current version of TouchFish Client. / 此更改与当前版本的 TouchFish Client 兼容。

  • This change causes a behavior change that may affect existing users. / 此更改会导致行为更改,可能会影响现有用户。

@ILoveScratch2

Copy link
Copy Markdown
Owner

删除依赖并不能迫使 Flutter 静态打包:

Run flutter build linux --release
Building Linux application...                                   
CMake Error at flutter/ephemeral/.plugin_symlinks/media_kit_video/linux/CMakeLists.txt:53 (target_link_libraries):
  Target "media_kit_video_plugin" links to:

    PkgConfig::mpv

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.



CMake Generate step failed.  Build files cannot be regenerated correctly.
Unable to generate build files
Error: Process completed with exit code 1.

@JohnChiao75

Copy link
Copy Markdown
Contributor Author

@ILoveScratch2 我已修改 CI:保留 libmpv-dev 确保 Linux 编译通过;在打包 tar.gz 前,将 libmpv.so.2 拷贝进 bundle 并设置 RPATH=$ORIGIN/lib,使普通用户解压即用,无需安装 mpv。AUR 的 PKGBUILD 会在打包时剔除该 .so 并清除 RPATH,依赖系统 libmpv,避免冲突。请重新审查。

needs: quality
# cannot VS 18.6/MSVC 14.51 windows-latest
runs-on: windows-2022
runs-on: windows-latest

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

这里为什么要改

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

windows-latest会爆警告吧,但是不改也行

- name: Install dependencies
run: flutter pub get


Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

build web和deploy pages都删了干嘛qwq

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

嘶,好像交错了awa

name: web-build
path: touchfish-web.zip

release:

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

为什么release也没了 qaq

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

同上

@JohnChiao75

Copy link
Copy Markdown
Contributor Author

@ILoveScratch2 什么?RC了?yui改的,你去问她(((

@ILoveScratch2

Copy link
Copy Markdown
Owner

@JohnChiao75 那你问啊,我感觉不行

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.

2 participants