Opened 18 years ago

Closed 18 years ago

#5307 closed (fixed)

django should make files created by startproject/startapp writeable

Reported by: Thomas Stromberg Owned by: Adrian Holovaty
Component: Core (Management commands) Version: dev
Severity: Keywords: permissions startproject startapp
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no
Pull Requests:How to create a pull request

Description

I noticed today that my previous fix (see Ticket #4688) only changed permissions for settings.py. We actually have the same problem with the files created by startapp from our read-only Django repository. This patch moves the permissions manipulation away from startproject.py to base.py, and is run for anything that uses copy_helper(). It also improves upon the old patch by calculating the proper permissions to achieve user-writeability rather than hardcoding 0600.

The attached patch is against today's trunk checkout.

Change History (2)

by Thomas Stromberg <tstromberg@…>, 18 years ago

Attachment: patch.trunk​ added

Patch to implement said changes, including addition of _make_writeable()

comment:1 by Adrian Holovaty, 18 years ago

Resolution: → fixed
Status: new → closed

(In [6028]) Fixed #5307 -- startproject/startapp now makes sure all files it creates are writeable. Thanks, Thomas Stromberg

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