Skip to content

Commit

Permalink
remove outdated comments / messages about venv
Browse files Browse the repository at this point in the history
  • Loading branch information
AlvinSchiller committed Nov 15, 2023
1 parent ff042d5 commit f14c66a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 9 deletions.
3 changes: 1 addition & 2 deletions documentation/content/developers/development-environment.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ developing.

The jukebox also runs on any Linux machine. The Raspberry Pi specific
stuff will not work of course. That is no issue depending our your
development area. USB RFID Readers, however, will work. You may setup a
Python virtual environment or a conda virtual environment. You will have
development area. USB RFID Readers, however, will work. You will have
to install and configure [MPD (Music Player
Daemon)](https://www.musicpd.org/).

Expand Down
7 changes: 0 additions & 7 deletions src/jukebox/components/rfid/configure/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -26,13 +26,6 @@ def reader_install_dependencies(reader_path: str, dependency_install: str) -> No
if os.path.exists(reader_path + '/requirements.txt'):
# The python dependencies (if any)
print("\nInstalling/Checking Python dependencies ...\n")
print("IMPORTANT for developers: Python dependencies will be installed using "
" $ pip install --upgrade -r requirements.txt'\n"
" i.e. on system level. This is target for the default RPI setup. "
"If you do not want that, but rather have them in a local or virtual environment, "
"hit No here and manually install the dependencies from your virtual environment\n"
" $ pip install --upgrade -r requirements.txt'\n"
"It is no problem to install them after running this script.\n\n")
if dependency_install == 'auto' or pyil.input_yesno("Install Python dependencies?", blank=True,
prompt_color=Colors.lightgreen, prompt_hint=True):
print(f"{'=' * 80}")
Expand Down

0 comments on commit f14c66a

Please sign in to comment.