这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@twaik
Copy link
Member

@twaik twaik commented May 12, 2025

Allow commands like cat ${TERMUX_PKG_TMPDIR} to fail inside subpackages buildscripts.
Usually it does not directly affect target packages

@twaik twaik linked an issue May 12, 2025 that may be closed by this pull request
@TomJo2000
Copy link
Member

I dunno about this one.
It's not solving the problem, it just hopes it isn't serious.

@twaik
Copy link
Member Author

twaik commented May 12, 2025

But we used to use this behaviour before 622d796

@TomJo2000
Copy link
Member

But we used to use this behaviour before 622d796

I'm not sure what you mean.
There's no set +e in there

@twaik
Copy link
Member Author

twaik commented May 12, 2025

There was no such line, but behaviour is same when you add set +e line.

@licy183
Copy link
Member

licy183 commented May 12, 2025

The build failure in the linked action may be related to this.

https://github.com/termux/termux-packages/actions/runs/14975001019

@twaik
Copy link
Member Author

twaik commented May 12, 2025

Ok, so since old code allowed this and masked failures from this function I will merge this.

@twaik twaik merged commit ded5a54 into master May 12, 2025
11 checks passed
@twaik twaik deleted the issue-24689 branch May 13, 2025 03:40
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.

Auto update failing for jackett

3 participants