Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug 1792152 #8

Merged
merged 1 commit into from Sep 13, 2018
Merged

Fix bug 1792152 #8

merged 1 commit into from Sep 13, 2018

Conversation

ediardo
Copy link
Contributor

@ediardo ediardo commented Sep 12, 2018

Fixed utility function is_stx_region

Closes-bug: https://bugs.launchpad.net/starlingx/+bug/1792152

Signed-off-by: Eddie Ramirez eddie.ramirez@intel.com

Fixed utility function is_stx_region

Closes-bug: https://bugs.launchpad.net/starlingx/+bug/1792152

Signed-off-by: Eddie Ramirez <eddie.ramirez@intel.com>
@tylersmith22
Copy link
Contributor

Looks good Eddie, thanks!

@dtroyer
Copy link
Contributor

dtroyer commented Sep 13, 2018

For future upstream purposes this should be merged into the earlier patch.
LGTM

@dtroyer dtroyer merged commit 80c6bed into starlingx-staging:master Sep 13, 2018
@kbujold
Copy link
Contributor

kbujold commented Sep 13, 2018

This fails to build due to a syntax error. The change should be reverted or fixed.

@ediardo
Copy link
Contributor Author

ediardo commented Sep 13, 2018

@kbujold @tylersmith22 @dtroyer please see fix to the syntax error here: #9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants