2008-6-21
| 08:23 | telenieko | morning all |
| 08:32 | thej1 | telenieko: good morning |
| 08:33 | thej1 | telenieko: regarding #7514...do the tests pass after your patch? |
| 08:33 | DjangoBot | |
| 09:03 | theju | telenieko: you there? |
| 09:07 | telenieko | theju: yes |
| 09:07 | telenieko | and yes also to the other q. ;) |
| 09:08 | telenieko | #7517 digs a bit more on the issue |
| 09:08 | DjangoBot | |
| 09:09 | theju | telenieko: it doesn't for me...actually i ain't doing syncdb...i am trying to run tests and it fails |
| 09:09 | telenieko | freakboy3742: you there? |
| 09:09 | telenieko | theju: how does it fail? |
| 09:09 | theju | let me paste |
| 09:09 | telenieko | dpaste |
| 09:11 | theju | |
| 09:12 | telenieko | I'd bet your problem is somewhere else ... mmm that happens without my patch? |
| 09:12 | theju | those tests never failed before...i applied your patch and it still fails |
| 09:12 | telenieko | if you revert the patch does it also fail? |
| 09:12 | theju | i saw something like syncdb...and thought it might have to do with syncdb |
| 09:12 | theju | it fails in both cases |
| 09:13 | theju | the tests start failing post-r7716 |
| 09:13 | DjangoBot | |
| 09:13 | telenieko | Nope, my patch clears a global var on syncdb but doesn't touch anything |
| 09:14 | telenieko | do you have the django/trunk/tests/urls.py changed on r7716 |
| 09:14 | DjangoBot | |
| 09:14 | telenieko | ? |
| 09:14 | theju | yes |
| 09:14 | telenieko | strange :o |
| 09:15 | theju | sorry...no |
| 09:15 | theju | it hasn't been changed |
| 09:15 | theju | should i change it? |
| 09:15 | telenieko | yes |
| 09:16 | telenieko | If you don't have tghe auth urls you'll always get 404 ;) |
| 09:16 | theju | i guess you meant django/tests/urls.py |
| 09:17 | theju | and not django/trunk/tests/urls.py |
| 09:17 | telenieko | nope, django/trunk/tests/urls.py (on svn) |
| 09:17 | telenieko | you'd have it in django/tests/urls.py |
| 09:18 | theju | it is available in django/tests/urls.py |
| 09:18 | telenieko | the last line from r7716, the include(django.contrib.auth.urls) |
| 09:18 | DjangoBot | |
| 09:21 | telenieko | it's dinner time in .au, bed time in the U.S. Where are Europeans? :o |
| 09:21 | telenieko | I've ran tests on a clean checkout and they run |
| 09:22 | telenieko | also theju would you mind on running test using tests/runtests.py ? |
| 09:23 | telenieko | I think that the urls.py is not used when running tests from modeltests/ |
| 09:25 | telenieko | did you conn die? |
| 09:25 | thej1 | yes |
| 09:25 | telenieko | http://oebfare.com/logger/django-dev/ < last lines (since last DjangoBot comment) |
| 09:25 | thej1 | saw that |
| 09:25 | telenieko | ;) |
| 09:25 | thej1 | let me re run the tests |
| 09:33 | thej1 | works |
| 09:33 | thej1 | it works now |
| 09:33 | thej1 | from tests/runtests.py |
| 09:34 | telenieko | ;)) |
| 09:35 | thej1 | oops...i used git-apply --check ../7514.diff |
| 09:35 | thej1 | no wonder it still failed :) |
| 10:32 | telenieko | theju: are tests working fine now? |
| 10:32 | theju | yes |
| 10:33 | theju | but why do you have two tickets open? any reason? |
| 10:33 | telenieko | nice. We only need some committer to wake up now :) |
| 10:33 | telenieko | they do not solve the same thing |
| 10:33 | telenieko | the first one solve the tests problem. The other one is about .delete() and .get_current() |
| 10:33 | telenieko | I put them separate so one commiter could fix the tests asap |
| 10:34 | theju | ok |
| 10:37 | telenieko | who maintains the buildbot? |