Skip to content

Commit fa93ff3

Browse files
committed
Bugfix: do not overwrite board properies during compile
Fix arduino#1614
1 parent 5332ffd commit fa93ff3

File tree

3 files changed

+12
-13
lines changed

3 files changed

+12
-13
lines changed

Diff for: legacy/builder/setup_build_properties.go

+1-2
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,11 @@ func (s *SetupBuildProperties) Run(ctx *types.Context) error {
3636

3737
targetPlatform := ctx.TargetPlatform
3838
actualPlatform := ctx.ActualPlatform
39-
targetBoard := ctx.TargetBoard
4039

4140
buildProperties := properties.NewMap()
4241
buildProperties.Merge(actualPlatform.Properties)
4342
buildProperties.Merge(targetPlatform.Properties)
44-
buildProperties.Merge(targetBoard.Properties)
43+
buildProperties.Merge(ctx.TargetBoardBuildProperties)
4544

4645
if ctx.BuildPath != nil {
4746
buildProperties.SetPath("build.path", ctx.BuildPath)

Diff for: legacy/builder/target_board_resolver.go

+1-2
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,6 @@ func (s *TargetBoardResolver) Run(ctx *types.Context) error {
3030
return fmt.Errorf("%s: %w", tr("Error resolving FQBN"), err)
3131
}
3232

33-
targetBoard.Properties = buildProperties // FIXME....
34-
3533
core := targetBoard.Properties.Get("build.core")
3634
if core == "" {
3735
core = "arduino"
@@ -46,6 +44,7 @@ func (s *TargetBoardResolver) Run(ctx *types.Context) error {
4644

4745
ctx.BuildCore = core
4846
ctx.TargetBoard = targetBoard
47+
ctx.TargetBoardBuildProperties = buildProperties
4948
ctx.TargetPlatform = targetPlatform
5049
ctx.TargetPackage = targetPackage
5150
ctx.ActualPlatform = actualPlatform

Diff for: legacy/builder/types/context.go

+10-9
Original file line numberDiff line numberDiff line change
@@ -82,15 +82,16 @@ type Context struct {
8282
BuildOptionsJson string
8383
BuildOptionsJsonPrevious string
8484

85-
PackageManager *packagemanager.PackageManager
86-
Hardware cores.Packages
87-
AllTools []*cores.ToolRelease
88-
RequiredTools []*cores.ToolRelease
89-
TargetBoard *cores.Board
90-
TargetPackage *cores.Package
91-
TargetPlatform *cores.PlatformRelease
92-
ActualPlatform *cores.PlatformRelease
93-
USBVidPid string
85+
PackageManager *packagemanager.PackageManager
86+
Hardware cores.Packages
87+
AllTools []*cores.ToolRelease
88+
RequiredTools []*cores.ToolRelease
89+
TargetBoard *cores.Board
90+
TargetBoardBuildProperties *properties.Map
91+
TargetPackage *cores.Package
92+
TargetPlatform *cores.PlatformRelease
93+
ActualPlatform *cores.PlatformRelease
94+
USBVidPid string
9495

9596
PlatformKeyRewrites PlatforKeysRewrite
9697
HardwareRewriteResults map[*cores.PlatformRelease][]PlatforKeyRewrite

0 commit comments

Comments
 (0)