Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-22 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  aRkadeFR
 Type:  Cleanup/optimization  |   Status:  closed
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:  fixed
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+

Comment (by Tim Graham ):

 In [changeset:"252867ed2aaec3e57dda08243a4603c8bffa7bb2" 252867e]:
 {{{
 #!CommitTicketReference repository=""
 revision="252867ed2aaec3e57dda08243a4603c8bffa7bb2"
 [1.8.x] Fixed #25001 -- Doc'd caveat about collectstatic and removing
 INSTALLED_APPS.

 Thanks aRkadeFR for the initial patch.

 Backport of 5ae0dd6abf8f93b90c962e62b04c675fdd6ca755 from master
 }}}

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.81e5fb39d817a75be1c8f1ef7f23055d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-22 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  aRkadeFR
 Type:  Cleanup/optimization  |   Status:  closed
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:  fixed
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by Tim Graham ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"5ae0dd6abf8f93b90c962e62b04c675fdd6ca755" 5ae0dd6a]:
 {{{
 #!CommitTicketReference repository=""
 revision="5ae0dd6abf8f93b90c962e62b04c675fdd6ca755"
 Fixed #25001 -- Doc'd caveat about collectstatic and removing
 INSTALLED_APPS.

 Thanks aRkadeFR for the initial patch.
 }}}

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.732f550a945933078066c2a5b2dc1263%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-22 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  aRkadeFR
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+

Comment (by aRkadeFR):

 seems good to me, thanks for the review.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.953a76ff9a02316f6184da02b2a07e37%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-22 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  aRkadeFR
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+

Comment (by timgraham):

 Thanks for the initial patch. For future reference, please use spaces
 instead of tabs for indentation. I tweaked the patch slightly in a
 [https://github.com/django/django/pull/4907 pull request]. Let me know if
 it's clear to you.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.aaaece0539a9488496b52d6b4110f272%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-22 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  aRkadeFR
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by aRkadeFR):

 * Attachment "25001.diff" added.

 Patch documentation for staticfiles stale content

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.b4eb8ee1ebe9fd17bbb42b0a6eb3fac0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-22 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  aRkadeFR
 Type:  Cleanup/optimization  |   Status:  assigned
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  1 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by aRkadeFR):

 * owner:  nobody => aRkadeFR
 * status:  new => assigned
 * has_patch:  0 => 1


Comment:

 First contribution, I followed the contribution documentation.
 I ran make spelling & make html, made the diff out of git diff.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.5d70935bba761eb121e0e43152d5f97f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-22 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by aRkadeFR):

 * Attachment "25001.diff" added.

 Patch documentation for staticfiles stale content

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.c9d0454211a1e468fb071c7c240e7cbc%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-20 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
--+
 Reporter:  JulianAndrews |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Documentation |  Version:  1.8
 Severity:  Normal|   Resolution:
 Keywords:| Triage Stage:  Accepted
Has patch:  0 |  Needs documentation:  0
  Needs tests:  0 |  Patch needs improvement:  0
Easy pickings:  1 |UI/UX:  0
--+
Changes (by timgraham):

 * type:  Bug => Cleanup/optimization
 * component:  contrib.staticfiles => Documentation
 * easy:  0 => 1
 * stage:  Unreviewed => Accepted


--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.df32727369ea2d8189f7822283666c9e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


Re: [Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-19 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
-+-
 Reporter:  JulianAndrews|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.8
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by timgraham):

 * needs_better_patch:   => 0
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 The issue is that `collectstatic` doesn't have any knowledge of which app
 a static file was collected from. It only looks at a file's modification
 time when determining whether to recopy a file. Unless there's an
 alternate proposal, I'd suggest to add a documentation warning about this
 and recommend the
 [https://docs.djangoproject.com/en/stable/ref/contrib/staticfiles/#django-
 admin-option---clear collectstatic --clear] option in this case.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/071.ca9e99fe978bd6f4662799d5fb719908%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.


[Django] #25001: collectstatic does not correctly handle uninstallation of apps which write the same file as remaining apps.

2015-06-18 Thread Django
#25001: collectstatic does not correctly handle uninstallation of apps which 
write
the same file as remaining apps.
-+
 Reporter:  JulianAndrews|  Owner:  nobody
 Type:  Bug  | Status:  new
Component:  contrib.staticfiles  |Version:  1.8
 Severity:  Normal   |   Keywords:
 Triage Stage:  Unreviewed   |  Has patch:  0
Easy pickings:  0|  UI/UX:  0
-+
 Steps to reproduce:

 1. Create a project with two apps which write the same static file.
 2. Run `python manage.py collectstatic`.
 3. Uninstall the first app.
 4. Re-run `python manage.py collectstatic`.

 Expected behavior:
 The file from the second (still installed) app should be installed.

 Observed Behavior:
 The file from the first (now uninstalled) app remains installed.

 See [https://github.com/fusionbox/collectstatic_bug] for a demonstration
 of the bug.

--
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/056.4f866ce96ac211688868a09e518b0c24%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.