If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Simplify objdir usage in mozharness

NEW
Unassigned

Status

Release Engineering
Mozharness
P5
normal
3 years ago
5 months ago

People

(Reporter: mshal, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

3 years ago
Somewhat similar to bug 1139413, we should simplify our usage of getting an objdir in the build scripts. For example, we have at least the following in various places:

 - self.query_abs_dirs()['abs_objdir']
 - self.query_abs_dirs()['abs_obj_dir']
 - self.objdir
 - self.config['objdir']
 - self._query_objdir()

Not to mention several other flavors of objdir that may or may not be redundant with these (shell_objdir, abs_analyzed_objdir, analysis-objdir). This would require some investigation to see if they can be joined with the others or not.

I think at least the first group can all be unified into a single thing in a one of the common script classes, so it is defined in one place and used the same throughout mozharness.

Updated

5 months ago
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.