Skip to content

Conversation

@guanzi008
Copy link
Contributor

Implement loading of extensions from config files
实现从配置文件加载扩展

@deepin-ci-robot
Copy link
Collaborator

Hi @guanzi008. Thanks for your PR.

I'm waiting for a OpenAtom-Linyaps member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@codecov
Copy link

codecov bot commented Oct 16, 2025

@guanzi008 guanzi008 reopened this Oct 21, 2025
@deepin-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: guanzi008

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

1 similar comment
@deepin-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: guanzi008

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@guanzi008
Copy link
Contributor Author

1) 全局扩展

ll-cli config set-extensions --global org.demo.ExtA,org.demo.ExtB
cat ~/.config/linglong/config.json

2) 应用扩展覆盖

ll-cli config set-extensions org.deepin.AppX org.demo.AppXExt
cat ~/.config/linglong/apps/org.deepin.AppX/config.json

3) 环境变量

ll-cli config set-env org.deepin.AppX LANG=zh_CN.UTF-8 PATH+=/opt/appx/bin

4) 文件系统映射

ll-cli config add-fs org.deepin.AppX --host ~/Downloads --target /data --mode rw --persist
ll-cli config rm-fs org.deepin.AppX --target /data

5) 命令设置

ll-cli config set-command org.deepin.AppX myapp --entrypoint /opt/my/bin/myapp --cwd /work --args-prefix "--safe -v" --args-suffix "--log debug" QT_QPA_PLATFORM=wayland

ll-cli config 支持维护沙箱 allowlist:新增 add-fs-allow、rm-fs-allow、clear-fs-allow 子命令,

Copy link
Collaborator

Choose a reason for hiding this comment

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

翻译文件不要更新

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好,回退了。

Copy link
Collaborator

@myml myml Oct 30, 2025

Choose a reason for hiding this comment

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

建议整理一下,命令不要加Ll这种奇怪的前缀

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的。

description: filesystem entries applied to command
items:
$ref: '#/$defs/LlCliFilesystemEntry'
args_prefix:
Copy link
Collaborator

Choose a reason for hiding this comment

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

有一些配置项好像没用到,如果还没有实现功能,就先把配置项去掉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

这个是准备把配置写到应用的linglong.yaml
就是这里也可以写配置。
这只是我的一种尝试。
都没验证有没有效果。。

@guanzi008 guanzi008 requested a review from myml October 30, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants