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.

Sign up for GitHub

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

Jump to bottom

Publish package for addon repo #3

Closed
rzr opened this issue May 6, 2020 · 10 comments
Closed

Publish package for addon repo #3

rzr opened this issue May 6, 2020 · 10 comments

Comments

@rzr
Copy link
Owner

rzr commented May 6, 2020

Installing dependencies on device using pip is currently failing on missing libs for numpy

The headers or library files could not be found for jpeg,
rzr added a commit to CrossStream/addon-builder that referenced this issue May 7, 2020
@rzr
toolchain: Add jpeg-dev to python images
d4eec3d 
To be used for building upcoming sense-hat-adapter

Relate-to: rzr/sense-hat-webthing#3
Change-Id: Ib3cb7fb8f2a52e70c71a5f779020a4570604874a
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr rzr mentioned this issue May 7, 2020
Merged
rzr added a commit to CrossStream/addon-builder that referenced this issue May 7, 2020
@rzr
toolchain: Add jpeg-dev to python images
8d68448 
To be used for building upcoming sense-hat-adapter

Relate-to: rzr/sense-hat-webthing#3
Forwarded: WebThingsIO#50
Change-Id: Ib3cb7fb8f2a52e70c71a5f779020a4570604874a
Signed-off-by: Philippe Coval <rzr@users.sf.net>
rzr added a commit to CrossStream/addon-builder that referenced this issue May 7, 2020
@rzr
toolchain: Add jpeg-dev to python images
3d13b7b 
To be used for building upcoming sense-hat-adapter
that is pulling RTIMULib which depends on it.

Relate-to: rzr/sense-hat-webthing#3
Forwarded: WebThingsIO#50
Change-Id: Ib3cb7fb8f2a52e70c71a5f779020a4570604874a
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr rzr mentioned this issue May 7, 2020
Closed
mrstegeman pushed a commit to WebThingsIO/addon-builder that referenced this issue May 7, 2020
@rzr
toolchain: Add jpeg-dev to python images (#50)
4732479 
To be used for building upcoming sense-hat-adapter
that is pulling RTIMULib which depends on it.

Relate-to: rzr/sense-hat-webthing#3
Forwarded: #50
Change-Id: Ib3cb7fb8f2a52e70c71a5f779020a4570604874a
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr
Copy link
Owner Author

rzr commented May 7, 2020

May action triggered by @mrstegeman again it failed on fetch:

https://github.com/mozilla-iot/addon-builder/runs/653694149

WebThingsIO/addon-builder#51

@mrstegeman
Copy link
Contributor

Yes, I can trigger the proper build manually if you're ready for release. Can you go ahead and open a PR for the addon-list with the new add-on? I can fill in the URLs and checksums.

rzr added a commit to CrossStream/addon-list that referenced this issue May 7, 2020
@rzr
sense-hat-adpter: Add version 0.0.3
aad8972 
Binary was build on rpi and uploaded to project releases.
Next version might use addon-builder.

Relate-to: rzr/sense-hat-webthing#3
Change-Id: I3e022f80c9417df1d27dba78336b47012558d6ce
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr rzr mentioned this issue May 7, 2020
Merged
@rzr
Copy link
Owner Author

rzr commented May 7, 2020

OK, let's rebuild next version I will make a new PR to update submodule in builder, will the list be updated automatically when using builder ?

@mrstegeman
Copy link
Contributor

You don't need to update the submodule, it's done automatically. You just need to submit a PR or issue to the addon-list when you want it updated and I'll manually kick off a build of the new version.

rzr added a commit to CrossStream/addon-list that referenced this issue May 7, 2020
@rzr
sense-hat-adpter: Add version 0.0.3
c3fdb1b 
Binary was build on rpi and uploaded to project releases.
Next version might use addon-builder.

Forwarded: WebThingsIO#822
Relate-to: rzr/sense-hat-webthing#3
Change-Id: I3e022f80c9417df1d27dba78336b47012558d6ce
Signed-off-by: Philippe Coval <rzr@users.sf.net>
mrstegeman pushed a commit to WebThingsIO/addon-list that referenced this issue May 7, 2020
@rzr
sense-hat-adpter: Add version 0.0.3 (#822)
8620f5a 
Binary was build on rpi and uploaded to project releases.
Next version might use addon-builder.

Forwarded: #822
Relate-to: rzr/sense-hat-webthing#3
Change-Id: I3e022f80c9417df1d27dba78336b47012558d6ce
Signed-off-by: Philippe Coval <rzr@users.sf.net>
rzr added a commit to CrossStream/addon-list that referenced this issue May 8, 2020
@rzr
sensehat: Update to 0.0.4 from 0.0.3
8803acf 
Change-Id: I7bca3477e10b4808c0c13358e553d734ab318301
Relate-to: rzr/sense-hat-webthing#3
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr rzr mentioned this issue May 8, 2020
Merged
rzr added a commit to CrossStream/addon-list that referenced this issue May 8, 2020
@rzr
sensehat: Update to 0.0.4 from 0.0.3
0693167 
Built on device using python 3.7 with pip 18.1

Change-Id: I7bca3477e10b4808c0c13358e553d734ab318301
Relate-to: rzr/sense-hat-webthing#3
Forwaded: WebThingsIO#825
Relate-to: rzr/sense-hat-webthing#6
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr
Copy link
Owner Author

rzr commented May 8, 2020 •
edited
Loading

For the record pip 10+ is needed to build package:
earlier versions will fail on numpy and log:

ImportError: No module named 'Cython'

Relate-to: #6

@rzr rzr mentioned this issue May 8, 2020
Closed
mrstegeman pushed a commit to WebThingsIO/addon-list that referenced this issue May 9, 2020
@rzr
sensehat: Update to 0.0.4 from 0.0.3 (#825)
08e6158 
Built on device using python 3.7 with pip 18.1

Change-Id: I7bca3477e10b4808c0c13358e553d734ab318301
Relate-to: rzr/sense-hat-webthing#3
Forwaded: #825
Relate-to: rzr/sense-hat-webthing#6
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@mrstegeman
Copy link
Contributor

This is finally building with the addon-builder, so we can do that for 0.0.5.

https://github.com/mozilla-iot/addon-builder/actions/runs/100116602

@rzr
Copy link
Owner Author

rzr commented May 9, 2020

Ok I am releasing

rzr added a commit to CrossStream/addon-list that referenced this issue May 9, 2020
@rzr
sense-hat: Update to 0.0.5 from 0.0.4
8cd92cc 
URL is expected one once published by mozilla-iot

While checksum is the one from upstream's release file
but could differ if rebuilt using addon-builder.

Bug: rzr/sense-hat-webthing#3
Relate-to: https://github.com/rzr/sense-hat-webthing/releases/tag/v0.0.5
Relate-to: https://github.com/rzr/sense-hat-webthing/releases/download/v0.0.5/sense-hat-adapter-0.0.5-linux-arm.tgz
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr rzr mentioned this issue May 9, 2020
Merged
@rzr
Copy link
Owner Author

rzr commented May 9, 2020

It built, According to builder log
https://github.com/mozilla-iot/addon-builder/runs/659606718?check_suite_focus=true

I expected it be published at:
https://s3-us-west-2.amazonaws.com/mozilla-gateway-addons/sense-hat-adapter-0.0.5-linux-arm-v3.7.tgz

@mrstegeman , Am I missing something to update:
WebThingsIO/addon-list#827

rzr added a commit to CrossStream/addon-list that referenced this issue May 9, 2020
@rzr
sense-hat: Update to 0.0.5 from 0.0.4
a690f5f 
sense-hat: Use builder binaries

Bug: rzr/sense-hat-webthing#3
Relate-to: https://github.com/rzr/sense-hat-webthing/releases/tag/v0.0.5
Relate-to: https://github.com/rzr/sense-hat-webthing/releases/download/v0.0.5/sense-hat-adapter-0.0.5-linux-arm.tgz
Signed-off-by: Philippe Coval <rzr@users.sf.net>
mrstegeman pushed a commit to WebThingsIO/addon-list that referenced this issue May 9, 2020
@rzr
sense-hat: Update to 0.0.5 from 0.0.4 (#827)
abcd17d 
sense-hat: Use builder binaries

Bug: rzr/sense-hat-webthing#3
Relate-to: https://github.com/rzr/sense-hat-webthing/releases/tag/v0.0.5
Relate-to: https://github.com/rzr/sense-hat-webthing/releases/download/v0.0.5/sense-hat-adapter-0.0.5-linux-arm.tgz
Signed-off-by: Philippe Coval <rzr@users.sf.net>
rzr added a commit that referenced this issue May 10, 2020
@rzr
build: Add rule/release/X.Y.Z
429512a 
Bug: #3
Signed-off-by: Philippe Coval <rzr@users.sf.net>
Change-Id: Icfe86219ab2364cd432bede53ce1c0fb4ad7830a
rzr added a commit to CrossStream/addon-list that referenced this issue May 10, 2020
@rzr
sense-hat-adapter: Update to 0.0.6
adb346b 
Forwarded: WebThingsIO#829
Bug: rzr/sense-hat-webthing#3
mrstegeman pushed a commit to WebThingsIO/addon-list that referenced this issue May 11, 2020
@rzr
sense-hat-adapter: Update to 0.0.6 (#829)
133ed59 
* sense-hat-adapter: Update to 0.0.6

Forwarded: #829
Bug: rzr/sense-hat-webthing#3

* sense-hat-adapter: Update checksum

Forwarded: #829
Signed-off-by: Philippe Coval <rzr@users.sf.net>
@rzr
Copy link
Owner Author

rzr commented May 11, 2020

Thanks @mrstegeman I think the flow is validated, Maybe I should make my PR editable or use a hook that would fetch binaries to update checksums and refresh PR, but it can be done manually for now, Probably github actions could do this on pushed tags too.

@rzr rzr closed this as completed May 11, 2020
@mrstegeman
Copy link
Contributor

Yeah... I’ve been trying to figure out a streamlined way of doing this, but it’s tricky to do without giving random people access to our S3 buckets and CI workflows.

rzr added a commit to CrossStream/addon-list that referenced this issue May 19, 2020
@rzr
sense-hat-adapter: Update to 0.0.7
0485ffb 
Bug: rzr/sense-hat-webthing#3
@rzr rzr mentioned this issue May 19, 2020
Merged
rzr added a commit that referenced this issue May 19, 2020
@rzr
build: Add checksums check
96d156c 
Bug: #3
Signed-off-by: Philippe Coval <rzr@users.sf.net>
Change-Id: Ibae8bbdd5abdf6f3cdc2decade0d90810f8aa39a
@rzr rzr mentioned this issue May 30, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rzr @mrstegeman