Closed Bug 1478338 Opened 3 years ago Closed 2 years ago

Configure scripts don't pass flake8


(Firefox Build System :: General, defect)

Not set


(Not tracked)



(Reporter: ahal, Assigned: Sylvestre)


(Blocks 1 open bug)



(1 file)

Bug 1406668 attempted to enable flake8 on build/moz.configure/*.configure files, however due to a bug in our flake8 integration those files were not actually being linted unless you explicitly specified the file path.

The flake8 integration issue was since fixed by bug 1473308, but I had to skip most of the *.configure files (they were either never flake8 compliant, or bustage slipped in after the fact).

To re-enable them, remove this line:

Note there other *.configure scripts scattered around the tree that are skipped in there as well.
Sigh, does that mean we'll have even more rules to deal with?  The rules for whitespace lines already make writing *.configure files annoying.
build/moz.configure/*.configure is almost green and almost didn't regress in 10 months.
What are your concerns? (these checks are enabled at review phase btw)
Keeping it open for the other configure files
Assignee: nobody → sledru
Keywords: leave-open
Comment on attachment 8994829 [details]
bug 1478338 - Fix some flake8 regression in moz.configure/ r?froydnj

Nathan Froyd [:froydnj] has approved the revision.
Attachment #8994829 - Flags: review+
Pushed by
Fix some flake8 regression in moz.configure/ r=froydnj
I will have a look, thanks!
Flags: needinfo?(sledru)

The leave-open keyword is there and there is no activity for 6 months.
:sylvestre, maybe it's time to close this bug?

Flags: needinfo?(sledru)

Closing, we should open a new bug to continue the work

Closed: 2 years ago
Flags: needinfo?(sledru)
Keywords: leave-open
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.