Skip to content

Command by Command documentation for WSL setup of LPython #1501

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.

Sign up for GitHub

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

Jump to bottom
Merged
Thirumalai-Shaktivel merged 1 commit into from
Feb 27, 2023
Merged

Command by Command documentation for WSL setup of LPython #1501

Thirumalai-Shaktivel merged 1 commit into from
Feb 27, 2023

Conversation

@faze-geek
Copy link
Contributor

@faze-geek faze-geek commented Feb 5, 2023

I setup Lpython on WSL2. I faced some issues, so I am documenting the exact commands for smooth setup in the future. I have referred a corresponding pr from the LFortran repo.

Thirumalai-Shaktivel
Thirumalai-Shaktivel reviewed Feb 5, 2023
View reviewed changes
czgdp1807
czgdp1807 reviewed Feb 6, 2023
View reviewed changes
@faze-geek
Copy link
Contributor Author

faze-geek commented Feb 9, 2023

@Thirumalai-Shaktivel Could you check this out? I have made suggested changes.

@Thirumalai-Shaktivel
Copy link
Collaborator

Thirumalai-Shaktivel commented Feb 10, 2023 •
edited
Loading

I'm sorry, I didn't notice this earlier.
@certik @czgdp1807, is this installation.md hosted or used as a guide anywhere?
I think we use this page in LFortran, right?

@faze-geek
Copy link
Contributor Author

faze-geek commented Feb 10, 2023

I think we use this page in LFortran, right?

Yes ,I was just thinking to make the following changes suggested there too .

@faze-geek faze-geek requested a review from czgdp1807 February 16, 2023 04:22
@Pranavchiku
Copy link
Collaborator

Pranavchiku commented Feb 26, 2023

Referring changed files of this PR to a new contributor asking for command by command documentation. Thanks for this :)

@faze-geek
Copy link
Contributor Author

faze-geek commented Feb 26, 2023

Referring changed files of this PR to a new contributor asking for command by command documentation. Thanks for this :)

Sure :) ...@Thirumalai-Shaktivel is this ready to go in btw . Works perfectly for me !

certik
certik approved these changes Feb 26, 2023
View reviewed changes
Copy link
Contributor

@certik certik left a comment

Choose a reason for hiding this comment

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

If this works, then it is fine with me.

@Thirumalai-Shaktivel @faze-geek
Add LPython installation procedure in Docs
18c1d1c 
co-authored-by: Anurag Bhat <bhat.1@iitj.ac.in>
@Thirumalai-Shaktivel Thirumalai-Shaktivel changed the title Command by Command documentation for WSL setup of Lpython Command by Command documentation for WSL setup of LPython Feb 27, 2023
@Thirumalai-Shaktivel Thirumalai-Shaktivel enabled auto-merge (squash) February 27, 2023 03:34
@Thirumalai-Shaktivel Thirumalai-Shaktivel merged commit c803bef into lcompilers:main Feb 27, 2023
@kmr-srbh kmr-srbh mentioned this pull request Feb 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@certik certik certik approved these changes

@Thirumalai-Shaktivel Thirumalai-Shaktivel Thirumalai-Shaktivel approved these changes

@czgdp1807 czgdp1807 Awaiting requested review from czgdp1807

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@faze-geek @Thirumalai-Shaktivel @Pranavchiku @certik @czgdp1807