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

Added pre-requisites for building native modules on linux and updated native-node-modules.md #336

Closed
wants to merge 12 commits into from

Conversation

ttheshreeyasingh
Copy link

@ttheshreeyasingh ttheshreeyasingh commented Apr 5, 2022

Added file linux-requirements.md in element-desktop/docs and updated native-node-modules.md, put the config part before the yarn start step for more clarity. Also added steps to install SQLCipher library in Adding Seshat for search in E2E encrypted rooms.

Signed-off-by: Shreeya [email protected]


This PR currently has no changelog labels, so will not be included in changelogs.

A reviewer can add one of: T-Deprecation, T-Enhancement, T-Defect, T-Task to indicate what type of change this is, or add Type: [enhancement/defect/task] to the description and I'll add them for you.

@ttheshreeyasingh ttheshreeyasingh requested a review from a team as a code owner April 5, 2022 18:33
Copy link
Member

@andybalaam andybalaam left a comment

Choose a reason for hiding this comment

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

Nice work, with some small improvements needed, thanks!

docs/linux-requirements.md Show resolved Hide resolved
docs/linux-requirements.md Show resolved Hide resolved
docs/linux-requirements.md Outdated Show resolved Hide resolved
docs/linux-requirements.md Outdated Show resolved Hide resolved
docs/native-node-modules.md Outdated Show resolved Hide resolved
docs/native-node-modules.md Outdated Show resolved Hide resolved
Copy link
Member

@andybalaam andybalaam left a comment

Choose a reason for hiding this comment

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

Sorry for the delay re-reviewing. Just one small change left and we can merge this. Thank you for working on it!

docs/linux-requirements.md Outdated Show resolved Hide resolved
Copy link
Member

@andybalaam andybalaam left a comment

Choose a reason for hiding this comment

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

Oops, I mean to click "Request changes" - just the Perl section needs changing.

@ttheshreeyasingh
Copy link
Author

Oops, I mean to click "Request changes" - just the Perl section needs changing.

Sure, will make the required changes

@ArchangeGabriel
Copy link

This change does not make sense. But more generally, all sections about sqlcipher should be removed since #334 has been merged.

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