Skip to content

install_service.sh: run main script inside uv .venv environment#297

Merged
M03ED merged 1 commit intoPasarGuard:devfrom
qmel:fix-install-service
Feb 25, 2026
Merged

install_service.sh: run main script inside uv .venv environment#297
M03ED merged 1 commit intoPasarGuard:devfrom
qmel:fix-install-service

Conversation

@qmel
Copy link

@qmel qmel commented Feb 23, 2026

Expected behaviour:

After building PasarGuard Panel from source code using make commands and running install_service.sh script, the service would be successfully installed and started.

Actual behaviour:

During the service start, python cannot find any dependent packages, as ExecStart=/usr/bin/env python3 resolved to the global environment, not the environment installed by uv (checked on Ubuntu 24.04.3 LTS).

Fix:

ExecStart was changed from /usr/bin/env python3 to $PWD/.venv/bin/python3, so the main script would be called directly in the virtual environment installed by uv during previous make requirements command.

@coderabbitai
Copy link

coderabbitai bot commented Feb 23, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@M03ED M03ED merged commit 00e672d into PasarGuard:dev Feb 25, 2026
2 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