Skip to content
This repository has been archived by the owner on May 30, 2020. It is now read-only.

Make it easier to run local version of PyPI. #536

Closed
wants to merge 1 commit into from

Conversation

Carreau
Copy link
Contributor

@Carreau Carreau commented Oct 6, 2016

Handle gracefully when one can't connect to s3.

Allow custom port and to deactivate TLS


This is just a cleanup of the local patch @michaelpacer and I are using on top of #506 to test/run locally.

import psycopg2
except ImportError:
pass
import psycopg2
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed the except-pass anyway if fails later if one does not have psycopg2 working.

validate=False,
)
else:
print("AWS database not fully configured. Getting the package files will be unavailable.", file=sys.stderr)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this is mostly indenting, so viewing the diff with ?w=1 might make things easier to review.

@@ -33,6 +33,19 @@ Then you can launch the server using the pypi.wsgi script::

PyPI will be available in your browser at http://localhost:8000

You can use the `--port` option to change the port, in particular some browser

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

browsers

@Carreau Carreau force-pushed the simple-run branch 2 times, most recently from 4626636 to 1f639ff Compare December 1, 2016 19:32
@Carreau
Copy link
Contributor Author

Carreau commented Dec 1, 2016

Rebased on master. And fixed spelling.

Handle gracefully when one can't connect to s3.

Allow custom port by using the --port option on the command line. This
is useful for browsers that only allow TLS on some ports.

Add a environment variable to deactivate TlS when running

Document both options in the readme.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants