Opened 4 years ago

Closed 3 years ago

#17578 closed Bug (fixed)

Explicitly invoke python for setup.py for Windows users

Reported by: pariksheet.nanda@… Owned by: zsiciarz
Component: Documentation Version: 1.4-alpha-1
Severity: Normal Keywords:
Cc: Gandi Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: yes UI/UX: no

Description

Details of fault seen:

http://forums.degreez.net/viewtopic.php?f=3&t=7084

Suggested fix:

The last words of step 4 should be "python setup.py install" instead of just "setup.py install" in this page
https://docs.djangoproject.com/en/dev/topics/install/#installing-an-official-release

Attachments (1)

patch_17578.diff (1.1 KB) - added by zsiciarz 4 years ago.
Note about PATH modification on Windows.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 4 years ago by claudep

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

Well, the forum post you are linking to is not supporting your request. The user did type "python setup.py" instead of "python setup.py install". But I guess a "setup.py install" would have worked the same way. I'm not so much familiar with Windows to finally decide if explicitely calling python is recommended or not.

comment:2 Changed 4 years ago by jezdez

  • Summary changed from Explicitly involke python for setup.py for Windows users to Explicitly invoke python for setup.py for Windows users

comment:3 Changed 4 years ago by jezdez

  • Triage Stage changed from Unreviewed to Accepted
  • Type changed from Uncategorized to Bug

comment:4 Changed 4 years ago by zsiciarz

The "python" prefix is neccessary if you install Django in virtualenv (after executing activate.bat). Otherwise it installs Django to systemwide site-packages.

(django_install) C:\Users\USER\v\django_install\dist\Django-1.3.1>setup.py install
... loads of stuff printed ...
Writing D:\Python27\Lib\site-packages\Django-1.3.1-py2.7.egg-info

(django_install) C:\Users\USER\v\django_install\dist\Django-1.3.1>python setup.py install
... loads of stuff printed again...
Writing C:\Users\USER\v\django_install\Lib\site-packages\Django-1.3.1-py2.7.egg-info

So it would make sense to mention "python setup.py install" explicitely.

Changed 4 years ago by zsiciarz

Note about PATH modification on Windows.

comment:5 Changed 4 years ago by zsiciarz

As jezdez pointed out at the Django sprint, there is an issue when the path to Python executable is not on PATH. An old ticket #4878 mentioned this issue, but it was discarded in favor of a new ticket. The patch adds a note about neccessary modifications to the PATH.

comment:6 Changed 4 years ago by zsiciarz

  • Owner changed from nobody to zsiciarz

comment:7 Changed 4 years ago by zsiciarz

  • Has patch set

comment:8 Changed 4 years ago by Gandi

  • Cc Gandi added
  • Triage Stage changed from Accepted to Ready for checkin
  • Version changed from 1.3 to 1.4-alpha-1

comment:9 Changed 3 years ago by PaulM

  • Resolution set to fixed
  • Status changed from new to closed

In [17619]:

Fixes #17578. Improve setup instructions for Windows users.

Thanks Pariksheet Nanda for the report and zsiciarz for the patch.

Note: See TracTickets for help on using tickets.
Back to Top