Closed Bug 1026238 Opened 7 years ago Closed 7 years ago

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

Categories

(Socorro :: General, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rhelmer, Assigned: rhelmer)

Details

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)
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
Summary: "make clean" should clear .pyc files in ./webapp-django too → "make clean" should clear .pyc files in ./
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
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90
Target Milestone: 90 → 91
You need to log in before you can comment on or make changes to this bug.