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

Draft - update dependencies to support building development environment on OS X #489

Closed
wants to merge 5 commits into from

Conversation

mrchrisadams
Copy link
Member

@mrchrisadams mrchrisadams commented Jul 11, 2023

This PR tightens up the specified version of mysqlclient that was causing failing builds on OS X, which meant working without gitpod locally was impossible.

It also moves the rich logging library into the packages group. It's useful in management commands, as well as logging generally.

@gitpod-io
Copy link

gitpod-io bot commented Jul 11, 2023

@mrchrisadams mrchrisadams changed the title Lock the version of mysql client Draft - update dependencies to support building development environment on OS X Jul 11, 2023
This time being explicit about python version, like so

python -m pipenv lock --dev --clear --python 3.8.10
python -m pipenv install --dev --clear --python 3.8.10
@mrchrisadams
Copy link
Member Author

mrchrisadams commented Jul 11, 2023

At least.

With pipenv, I think you need to be really careful when setting up a virtualenv. This didn't give me the virtual env using python 3.8.10, no matter what I did with switching local pythons, and even making it explicit in the pipfile doesn't seemt to work.

PIPENV_VENV_IN_PROJECT=True python -m pipenv lock --dev

I had to do this when creating the virtual environment instead:

PIPENV_VENV_IN_PROJECT=True python -m pipenv lock --dev --clear --python 3.8.10
PIPENV_VENV_IN_PROJECT=True python -m pipenv install --dev --clear --python 3.8.10

@mrchrisadams
Copy link
Member Author

This is no longer needed - we addressed the problematic mysqlclient dependency issue in a recent PR instead linked below:

#496

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.

1 participant