1026238 - "make clean" should clear .pyc files in ./

Status: RESOLVED FIXED

(Socorro :: General, task)

Description

[Robert Helmer [:rhelmer]]

Our "make clean" target only removes .pyc files from ./socorro and ./alembic/versions right now - this is because it pre-dates webapp-django being merged into the main socorro repo.

I think we should include ./webapp-django on the list of directories to clear .pyc files, that would avoid hitting things like bug 1026158 (which is mighty painful to debug)

Comment 1

[Robert Helmer [:rhelmer]]

Actually is there any reason we shouldn't just clear all .pyc files in ./ ?

Here's where I see them in my local checkout:

$ find . -name "*.pyc" | awk -F \/ '{print $2}' | sort | uniq
alembic
socorro
socorro-virtualenv
tools
webapp-django

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/ed7e7e86c9518bbb0f2acdc744dc0d91f13edfd5
fix bug 1026238 - clean .pyc files anywhere in checkout

https://github.com/mozilla/socorro/commit/d46bd636d3488c46f6b45b4f84567f05134f9e16
Merge pull request #2124 from rhelmer/bug1026238-clear-webapp-django-pyc

fix bug 1026238 - clean .pyc files anywhere in checkout