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

luci-app-socat: fix file conflicts #205

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jjm2473
Copy link
Contributor

@jjm2473 jjm2473 commented Aug 16, 2022

修复luci-app-socat与socat冲突的文件

Signed-off-by: jjm2473 [email protected]

@@ -5,11 +5,15 @@

include $(TOPDIR)/rules.mk

PKG_VERSION:=20220816-1
Copy link
Member

Choose a reason for hiding this comment

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

I don't like this hack anyway.

Simply comment out this line and your issue should be solved:
https://github.com/immortalwrt/packages/blob/e896f65ee49be10b5c748e65c73b0db86ca6fc8f/net/socat/Makefile#L88

Copy link
Contributor Author

Choose a reason for hiding this comment

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

PKG_VERSION如果要兼容luci17到luci21,就只能这样hack。

之所以不改上游或者第一方的软件包,是因为我打算依赖官方的软件仓库,如果三方的软件包文件跟一方的软件包冲突,自然是改三方的,而且改了也没任何副作用。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

如果要让用户方便地使用多个软件仓库,自然是要解决这些文件冲突的。总之我们的仓库已经更新了没冲突的版本,如果你们不需要这个patch就关闭吧

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