Skip to content

Set OSAL_SYSTEM_BSPTYPE for native builds#439

Merged
skliper merged 1 commit intonasa:merge-20200108from
jphickey:fix-438-set-osal-bsptype
Jan 10, 2020
Merged

Set OSAL_SYSTEM_BSPTYPE for native builds#439
skliper merged 1 commit intonasa:merge-20200108from
jphickey:fix-438-set-osal-bsptype

Conversation

@jphickey
Copy link
Contributor

Describe the contribution

Fix #438

This explicitly specifies the BSP to use when using SIMULATION=native flags to the build. All other example toolchain files already included this setting.

Testing performed
Rebuild code with and without SIMULATION=native flag. Confirmed no build issues/changes.

Expected behavior changes
No impact - build script change only.

System(s) tested on:
Ubuntu 18.04 LTS 64-bit

Additional context
This makes it compatible after a related OSAL build script cleanup is also merged (see nasa/osal#312, nasa/osal#261).

Contributor Info
Joseph Hickey, Vantage Systems, Inc.

Community contributors
You must attach a signed CLA (required for acceptance) or reference one already submitted

This explicitly specifies the BSP to use when using SIMULATION=native
flags to the build.  All other example toolchain files already included
this setting.
@skliper skliper added this to the 6.8.0 milestone Dec 30, 2019
@skliper skliper added the CCB:Approved Indicates code review and approval by community CCB label Jan 10, 2020
@skliper
Copy link
Contributor

skliper commented Jan 10, 2020

CCB 20190108 - Reviewed and approved for integration candidate

@skliper skliper changed the base branch from master to merge-20200108 January 10, 2020 15:12
@skliper skliper merged commit e3b90f2 into nasa:merge-20200108 Jan 10, 2020
skliper pushed a commit that referenced this pull request Jan 10, 2020
Fix #259 #425 #427 #435 #437 #438 #443 #445
Reviewed and approved at 2020-01-08 CCB
skliper pushed a commit that referenced this pull request Jan 14, 2020
Fix #438: Set OSAL_SYSTEM_BSPTYPE for native builds
@jphickey jphickey deleted the fix-438-set-osal-bsptype branch February 13, 2020 15:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates code review and approval by community CCB enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants