Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Removing sample.bb, replaced with some basic information about how to… #132

Merged
merged 2 commits into from
Feb 14, 2017

Conversation

darobs
Copy link
Contributor

@darobs darobs commented Feb 13, 2017

… build using bitbake on Yocto.

Added some Intel Edison specific instruction.

… build using bitbake on Yocto.

Added some Intel Edison specific instruction.
Copy link
Member

@damonbarry damonbarry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good...I made a couple minor comments that should be addressed. Thanks!

to your `conf/local.conf` file.

```
PACKAGECONFIG_pn-azure-iot-gateway-sdk = "bluetooth"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see how adding something about bluetooth to a conf file will prevent the java target from being built?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's not obvious, sorry. There are 3 packages in the azure-iot-gateway-sdk recipe: the main package, azure-iot-gateway-sdk-java and azure-iot-gateway-sdk-bluetooth. This statement will limit the packages to the main and bluetooth packages. I'll try to make this more clear.


- Add the azure-iot-gateway-sdk recipe to the target.

Edit the Edison image receipe to include the "azure-iot-gateway-sdk" target. For example:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: receipe --> recipe

@@ -1,32 +0,0 @@
DESCRIPTION = ""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay! Look at all the red!

Copy link
Member

@damonbarry damonbarry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥇

@darobs darobs merged commit aa580c1 into Azure:master Feb 14, 2017
@darobs darobs deleted the better-bitbake branch April 27, 2017 15:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants