Skip to content
This repository has been archived by the owner on Dec 19, 2022. It is now read-only.

fix(snap): Update dependencies and versioning #53

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

siggiskulason
Copy link
Contributor

This commit fixes two build issues with the snap.

  1. Source tags for iotech-c-utils and device-sdk-c updated
    to the latest version. Also update the source tag for mraa
    to use the long git commit hash instead of the short hash.

  2. A VERSION file is required by the cmake build so it is
    created if it doesn't exist.

Snap versioning was also updated to use "snapcraftctl set-version"
and to not use the git tag or date in the version string.

Signed-off-by: Siggi Skulason siggi.skulason@canonical.com

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

The snap doesn't build successfully

Issue Number:

What is the new behavior?

The snap build succeeds

Does this PR introduce a breaking change?

  • Yes
  • No

Are there any new imports or modules? If so, what are they used for and why?

Are there any specific instructions or things that should be known prior to reviewing?

Other information

This commit fixes two build issues with the snap.

1) Source tags for iotech-c-utils and device-sdk-c updated
to the latest version. Also update the source tag for mraa
to use the long git commit hash instead of the short hash.

2) A VERSION file is required by the cmake build so it is
created if it doesn't exist.

Snap versioning was also updated to use "snapcraftctl set-version"
and to not use the git tag or date in the version string.

Signed-off-by: Siggi Skulason <siggi.skulason@canonical.com>
Copy link
Member

@tonyespy tonyespy left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@iain-anderson iain-anderson left a comment

Choose a reason for hiding this comment

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

LGTM

@iain-anderson iain-anderson merged commit 59b1f73 into edgexfoundry:master Dec 7, 2020
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.

3 participants