Use pipenv instead of requirements.txt #203
Conversation
|
Note: dette ændrer venv dir'et på prodekanus fra Jeg ved ikke om det er vigtigt at venv'et ligger et bestemt sted, da vi altid bare kan bruge |
|
Det skal opdateres i Apache configen (som burde være i web-repoet, men er det ikke - relateret til #144), men det er ikke noget problem så længe vi husker det når vi deployer. |
There was a problem hiding this comment.
Dvs. django-cleanup og django-model-utils er enten direct dependencies eller bliver slet ikke brugt.
Jeg har undersøgt det nærmere, og vi bruger dem rent faktisk: django-cleanup indsætter hooks der automatisk sletter filer på disk når de tilhørende modelinstanser med FileField slettes; django-model-utils bliver brugt i tkweb/apps/gallery/.
Følgende ser ikke ud til at være direct dependencies (medmindre vi selvfølgelig bruger dem direkte):
Pillow (krævet af wiki og django-versatileimagefield)
Direct dependency, da den bruges i gallery og regnskab.
sorl-thumbnail (krævet af wiki)
Direct dependency: Importeres i gallery.
(numpy (krævet af scipy, men lad os beholde den som explicit dependency))
Direct dependency: Importeres i regnskab.
Vil du tilføje dem til Pipfile?
|
Fixed |
cron.sh
Outdated
| python $DIR/manage.py updateical --settings=tkweb.settings.prod | ||
| python $DIR/manage.py delete_marked_images --settings=tkweb.settings.prod | ||
| pipenv run ./manage.py updateical --settings=tkweb.settings.prod | ||
| python run ./manage.py delete_marked_images --settings=tkweb.settings.prod |
readme-prodekanus.md
Outdated
| python3 manage.py migrate --settings=tkweb.settings.prod # Migrer databasen til en evt. ny model. | ||
| python3 manage.py collectstatic --settings=tkweb.settings.prod # Saml statiske filer så apache kan finde dem. | ||
| export PIPENV_VENV_IN_PROJECT=1 # Sig til pipenv at den skal bruge .venv mappe inde i projektet | ||
| pipenv install --three # Installer og opdater alle python pakker i virtualenv. Det kan være at den skal køres flere gange. |
There was a problem hiding this comment.
Hm, hvorfor skal pip install -U -r requirements.txt eller pipenv install --three køres flere gange?
There was a problem hiding this comment.
Jeg ved ikke med pipenv, men pip install -U pip -r requirements.txt (med -U pip) skal køre to gange hvis pip selv bliver opdateret.
There was a problem hiding this comment.
Jeg kan ikke se at nogen af dem køres flere gange, kan du uddybe?
There was a problem hiding this comment.
Hvis man kører pip install -U pip -r requirements.txt og ens pip-pakke er outdated bliver pip-pakken og kun pip-pakken opdateret. Man skal derfor køre kommandoen igen for at opdatere det der er i requirements.txt. Det var derfor jeg skrev kommentaren for lang tid siden. Jeg tror ikke at pipenv install har brug for at køre flere gange, så kommentaren kan slettes.
There was a problem hiding this comment.
Ah, havde ikke læst kommentaren. Sletter den...
readme.md
Outdated
| @@ -43,7 +43,7 @@ flere moduler der nedarver fra hinanden. Se også under | |||
| ## Udviklingsmiljø | |||
|
|
|||
| Det er en forudsætning at maskinen har en fungerende python installation med | |||
| `pip`. | |||
| [`pipenv`](https://docs.pipenv.org/), som kan installeres med `pip install pipenv`. | |||
There was a problem hiding this comment.
Evt. pip install --user pipenv?
There was a problem hiding this comment.
Det kræver at man har $HOME/.local/bin i sin PATH, hvilket jeg f.eks. ikke har på nogen af mine setups.
|
Det ser godt ud! |
|
Aha. I så fald vil jeg formode at kommentaren blot kan slettes.
|
|
Aha. Hvad med laveste fællesnævner og blot skrive at pipenv kan installeres med pip?
|
Avoids "ValueError: Invalid Specifiers 5.0.0" upon `pipenv install`
See TK-IT/meta#26