Actions
action #78240
closedprevent circular dependencies in bmwqemu.pm and autotest.pm to be able to use "strictures"
Status:
Resolved
Priority:
Low
Assignee:
-
Category:
Feature requests
Target version:
QA (public, currently private due to #173521) - future
Start date:
2020-11-19
Due date:
% Done:
0%
Estimated time:
Description
Motivation¶
To be able to make all perl warnings fatal in test we should use use strictures;
, see #10704#note-16 , which okurz tried in https://github.com/os-autoinst/os-autoinst/pull/1548 where we see that we already have circular dependencies which use strictures;
makes obvious now. This needs to be fixed first
Acceptance criteria¶
- AC1: No more circular dependencies in bmwqemu.pm and autotest.pm as reported by
use strictures
Suggestions¶
- See https://github.com/os-autoinst/os-autoinst/pull/1548#issuecomment-704841925 first
- Checkout the the PR and temporarily add
use strictures
locally, then try to fix the circular dependencies by pulling out common dependencies into a separate module
Updated by okurz about 4 years ago
- Related to action #10704: Make warnings in tests fatal added
Updated by tinita over 2 years ago
- Has duplicate action #104670: Fix circular dependency of autotest <-> bmwqemu added
Updated by tinita over 2 years ago
- Status changed from New to Resolved
This was actually solved in #104670
I didn't see this ticket when I created the other one.
Actions