-
Notifications
You must be signed in to change notification settings - Fork 26
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
docs: Update guide for SCANOSS install on Ubuntu 24 #595
Changes from 3 commits
d1d1164
f9847a0
89768fe
bf1d042
50f0260
27fd724
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -131,7 +131,9 @@ You can now create a new pull request or update an existing one with commits. _L | |
|
||
Before building _LPVS_ from source code, ensure that you have the following prerequisites installed: | ||
|
||
- SCANOSS Python package by following the [guidelines](https://github.com/scanoss/scanoss.py#installation). Install it using the command: | ||
- SCANOSS Python package by following the [guidelines](https://github.com/scanoss/scanoss.py#installation). | ||
|
||
For installing SCANOSS on Ubuntu 2023.04, Fedora 38, Debian 11, etc. or later versions of mentioned OSes please follow the note at the end of the paragraph otherwise install it using the command: | ||
```bash | ||
pip3 install scanoss | ||
``` | ||
|
@@ -145,6 +147,29 @@ Before building _LPVS_ from source code, ensure that you have the following prer | |
sudo apt install mysql-server | ||
``` | ||
|
||
|
||
> [!NOTE] | ||
> For installing SCANOSS on Ubuntu 2023.04, Fedora 38, Debian 11, etc. or later versions of mentioned OSes, a few additional steps are required, as previously mentioned command : `pip3 install scanoss` will not work correctly. [More details](https://github.com/scanoss/scanoss.py/blob/main/PACKAGE.md#externally-managed-environments-on-linux) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same as above |
||
|
||
Installation steps: | ||
- Install `pipx`: | ||
```bash | ||
sudo apt install pipx | ||
``` | ||
- Install `scanoss` by using `pipx`: | ||
```bash | ||
pipx install scanoss | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why don't you use There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Unfortunately pipx does not support such syntax. Need additionally check how to use pipx with specified requirements defined in file. qa@qa-VirtualBox:~/Downloads/LPVS$ pipx install -help The install command is the preferred way to globally install apps The result: apps you can run from anywhere, located in packages pipx install PACKAGE_NAME ... The PACKAGE_SPEC argument is passed directly to The default virtual environment location is /home/qa/.local/share/pipx The default app location is /home/qa/.local/bin and can be The default manual pages location is /home/qa/.local/share/man and The default python executable used to install a package is positional arguments: options: There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please find the solution first. This is a known issue. Please refer to pypa/pipx#359 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Seems pipx still does not have official solution for installing from a requirement file. As confirmation of it- If this PR will be merged seems will be standard possibility to use requirement file as syntax is: Another way by using additional packet: There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Got it. Will be added to the guide after pipx feature official release. |
||
``` | ||
- Add to `PATH` `scanoss` installation: | ||
```bash | ||
pipx ensurepath | ||
``` | ||
- Close the terminal, open new terminal and check if the command below is available from any location: | ||
```bash | ||
scanoss-py | ||
``` | ||
|
||
|
||
### 2. Create Necessary MySQL Database and User (optional if not using a database) | ||
|
||
2.1 Start the MySQL server: | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please do not use such abbreviations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please clarify -is it related to names of OSes? I do not know how to inform an user if needed special steps if no information of OS name and version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mention the abbreviation
OS
here.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for comment. Is there a variant? And how to inform user that needed to use pipx instead pip if no OS reference? May be Python version? But on scanoss guide they referenced to name of OSes and versions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My suggestion is the following.
Could you try to install ScanOSS using such a command? In case of failure, please refer to the official documentation of ScanOSS.