418896 - profileserver.py.in shouldn't need to be preprocessed.

Status: RESOLVED FIXED

Description

[Robert Sayre]

In bug 418772, Ted says
> Did your changes remove all of the preprocessor stuff?  Could this become a
> normal .py file instead of a .py.in now? 

And it's a good question. Unfortunately, getting relative imports to work with a mix of preprocessed and unpreprocessed files is tricky. We'll need to figure that out, or adjust the module search path somehow.

Comment 1

[Jeff Walden [:Waldo]]

Moving a bunch of Core :: Testing bugs to Testing :: Mochitest to clear out the former, which is obsolete now that we have more specialized categories for such bugs; filter on the string "MochitestMmMm" to delete all these notifications.

Comment 2

[Ted Mielczarek [:ted.mielczarek]]

Note that I worked around this with runreftest.py by just $(INSTALL)ing the script to _tests/reftest in the objdir, alongside automation.py. runreftest.py is not preprocessed, as it relies on automation.py for all that dirty work.

Comment 3

[Ted Mielczarek [:ted.mielczarek]]

bug 576026 fixed runtests.py.

Comment 4

[Edmund Wong (:ewong)]

Created attachment 492635 [details] [diff] [review]
Removed profileserver.py from being preprocessed.

Comment 5

[Ted Mielczarek [:ted.mielczarek]]

Comment on attachment 492635 [details] [diff] [review]
Removed profileserver.py from being preprocessed.

r=me, but we should probably fix this the same way as bug 614863 gets fixed to avoid problems.

Comment 6

[Justin Wood (:Callek)]

(In reply to comment #5)
> Comment on attachment 492635 [details] [diff] [review]
> Removed profileserver.py from being preprocessed.
> 
> r=me, but we should probably fix this the same way as bug 614863 gets fixed to
> avoid problems.

I just advocated for 614863 to be wontfix. do you have any suggestions/ideas on how to fix that issue easily?

Comment 7

[:Ehsan Akhgari]

http://hg.mozilla.org/mozilla-central/rev/ae5dcbff45df