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 basic documentation #28

Merged
merged 6 commits into from
Sep 4, 2024
Merged

Added basic documentation #28

merged 6 commits into from
Sep 4, 2024

Conversation

Hahihula
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@kolipakakondal kolipakakondal left a comment

Choose a reason for hiding this comment

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

Thanks @Hahihula for the PR. Could you please check the review comments.

docs/en/index.rst Outdated Show resolved Hide resolved
Comment on lines 1 to 5
Installation of IDF
===============================
Installing the ESP-IDF using EIM is straightforward. Simply open your preferred shell (we recommend PowerShell for Windows users) and run EIM, providing your installation preferences using any of the configuration <configuration> methods you choose. If you haven't provided all the necessary options, an interactive wizard will guide you through the remaining steps.

After Installation
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please describe the installation process of the eim tool in detail, including screenshots.

Also, explain what artifacts it will generate, where they will be stored, and how users should get started once the installation is complete.

Comment on lines 3 to 12
Below are the minimum requirements for running the ESP-IDF Installation Manager.

Windows
----------------

- git
- cmake
- ninja
- python with pip capable of creating virtual environment and doing SSL requests

Copy link
Collaborator

Choose a reason for hiding this comment

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

For Windows, Git and CMake are not prerequisites for installing eim, right? They are prerequisites for IDF, they were installed by eim.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

the prerequisites are always for the IDF as eim does not have any and as it's stated in the note, on windows we can take care of these if user wants to

Copy link
Collaborator

Choose a reason for hiding this comment

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

IMO, the better way to present this is -

To get started with ESP-IDF, you need Git, CMake, Ninja, and Python. The ESP-IDF Installation Manager will verify the required prerequisites on your system and install any that are missing.

For more details about ESP-IDF prerequisites, please refer to the ESP-IDF prerequisites documentation.

docs/en/prerequisites.rst Show resolved Hide resolved
Comment on lines +20 to +23
- dfu-util
- cmake
- ninja
- python with pip capable of creating virtual environment and doing SSL requests
Copy link
Collaborator

Choose a reason for hiding this comment

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

Mention only the prerequisites that are not installed by eim but are needed for IDF.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

we do not install any prerequisities on posix systems so all this are needed for some version of IDF

docs/en/prerequisites.rst Show resolved Hide resolved
@Hahihula Hahihula merged commit 8202c09 into master Sep 4, 2024
5 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.

2 participants