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 build failure in RISCV deepin #2

Merged
merged 1 commit into from
Nov 16, 2022
Merged

Fix build failure in RISCV deepin #2

merged 1 commit into from
Nov 16, 2022

Conversation

Gui-Yue
Copy link
Contributor

@Gui-Yue Gui-Yue commented Nov 12, 2022

ncurses is not linked with telnet by default, which triggered an undefined
symbol error. This patch fixes this by explicitly telling cmake about about
to do that.

@github-actions
Copy link

github-actions bot commented Nov 12, 2022

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Gui-Yue
Copy link
Contributor Author

Gui-Yue commented Nov 12, 2022

I have read the CLA Document and I hereby sign the CLA.

@Gui-Yue
Copy link
Contributor Author

Gui-Yue commented Nov 12, 2022

/check

ncurses is not linked with telnet by default, which triggered an undefined
symbol error. This patch fixes this by explicitly telling cmake about about
to do that.
@Gui-Yue Gui-Yue changed the title fix build failure caused missing include and wrong manpage path Fix build failure in RISCV deepin Nov 13, 2022
@Gui-Yue
Copy link
Contributor Author

Gui-Yue commented Nov 13, 2022

I have read the CLA Document and I hereby sign the CLA.

deepin-community-ci-bot bot added a commit to deepin-community/cla that referenced this pull request Nov 13, 2022
@Gui-Yue
Copy link
Contributor Author

Gui-Yue commented Nov 13, 2022

/check

@RevySR
Copy link

RevySR commented Nov 16, 2022

/merge

@deepin-community-ci-bot
Copy link

Permission denied

@Zeno-sole
Copy link

/merge

@deepin-community-ci-bot deepin-community-ci-bot bot merged commit f349181 into deepin-community:master Nov 16, 2022
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.

3 participants