Closed
Bug 1158893
Opened 10 years ago
Closed 10 years ago
Delete the old Cython compiled log parser files as part of deploy
Categories
(Tree Management :: Treeherder: Infrastructure, defect, P3)
Tree Management
Treeherder: Infrastructure
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
Attachments
(1 file)
During deploy we delete the .pyc files, but don't currently remove the output from the Cython build of the log parser files:
./treeherder/log_parser/artifactbuildercollection.c
./treeherder/log_parser/parsers.c
./treeherder/log_parser/artifactbuilders.c
./treeherder/log_parser/artifactbuildercollection.so
./treeherder/log_parser/parsers.so
./treeherder/log_parser/artifactbuilders.so
This means that they don't get rebuild after eg changing the version of Cython (bug 1158395).
We should purge them too, and let the later existing build step re-create them.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8598105 -
Flags: review?(mdoglio)
Comment 2•10 years ago
|
||
Right, I didn't think about this case :)
Updated•10 years ago
|
Attachment #8598105 -
Flags: review?(mdoglio) → review+
Comment 3•10 years ago
|
||
Commit pushed to master at https://github.com/mozilla/treeherder
https://github.com/mozilla/treeherder/commit/343d112f930c21b1516f7a9353812e9ff99926d9
Bug 1158893 - Delete any existing Cython build output as part of deploy
The Cython build process creates .c and .so files in the log_parser
directory, which only get re-created if the original source changes.
We recently updated the version of Cython, however this doesn't trigger
re-building them. To avoid surprises, we should purge them like we do
for the .pyc files.
In addition, there is no need to delete .pyc files from the UI source
directory, since that does not contain any Python files.
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•10 years ago
|
Attachment #8598105 -
Flags: checkin-
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8598105 [details] [review]
Delete any existing Cython build output as part of deploy
Bah wrong bug.
Attachment #8598105 -
Flags: checkin-
You need to log in
before you can comment on or make changes to this bug.
Description
•