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

fix(main/cvs): fix build without editor #21900

Merged
merged 1 commit into from
Oct 22, 2024

Conversation

robertkirkman
Copy link
Contributor

@robertkirkman robertkirkman commented Oct 20, 2024

Fixes "Failed to find a text file editor. CVS cannot be compiled without a default log message editor." during build (within the official termux builder docker image using scripts/run-docker.sh ./build-package.sh -f cvs)

Based on code copied from https://gitlab.com/buildroot.org/buildroot/-/commit/1455d5241b288770aeea7f5ba6dcd1ea31966ba1

packages/cvs/build.sh Outdated Show resolved Hide resolved
@fornwall
Copy link
Member

@robertkirkman Thanks! Can you squash the three commits into a single fix(main/cvs): fix build without editor one (see e.g. #21524 (review))?

Fixes "`Failed to find a text file editor. CVS cannot be compiled without a default log message editor.`" during build (within the official termux builder docker image using `scripts/run-docker.sh ./build-package.sh -f cvs`)

Based on code copied from https://gitlab.com/buildroot.org/buildroot/-/commit/1455d5241b288770aeea7f5ba6dcd1ea31966ba1
@robertkirkman robertkirkman force-pushed the cvs-fix-build-without-editor branch from 96202e6 to cf31c03 Compare October 22, 2024 18:51
@robertkirkman
Copy link
Contributor Author

Yes sure! I hope that it is changed correctly now.

@fornwall fornwall merged commit d47cc2b into termux:master Oct 22, 2024
7 checks passed
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.

4 participants