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

Do not source venv from inside script #340

Open
wants to merge 1 commit into
base: bano_v3
Choose a base branch
from

Conversation

frodrigo
Copy link
Member

Warning: breaking change.

The source command should be added to the crontab file.

@vdct
Copy link
Member

vdct commented Feb 26, 2023

My first try sourcing the conf file in CRON line failed :

source </path/to/config> && /path/to/cron_bano.sh

That's why I rolled it back to the current behavior. Can you suggest the correct command line to put in the crontab ?

@frodrigo
Copy link
Member Author

Je pense qu'il faut passer par un commande bash: bash -c "source </path/to/config> && /path/to/cron_bano.sh"

Mais je vois qu'il y a d'autres façon de faire
https://stackoverflow.com/questions/3287038/cron-and-virtualenv

@frodrigo frodrigo force-pushed the no_source_venv branch 2 times, most recently from c17650a to 7b17ecd Compare February 13, 2024 23:19
@frodrigo frodrigo changed the base branch from master to bano_v3 February 13, 2024 23:19
@frodrigo
Copy link
Member Author

Rebase on bano_v3 branch.

@frodrigo frodrigo marked this pull request as ready for review August 8, 2024 12:09
@frodrigo
Copy link
Member Author

frodrigo commented Aug 8, 2024

Je reactive ce PR. Et rebsé.

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