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

Allow publishing of externally built tarballs #20

Open
juliusknorr opened this issue Dec 7, 2017 · 9 comments
Open

Allow publishing of externally built tarballs #20

juliusknorr opened this issue Dec 7, 2017 · 9 comments

Comments

@juliusknorr
Copy link

I just tried to release a nightly build of deck to the app store, but if fails with the following error message:

$  krankerl publish --nightly https://download.bitgrid.net/nextcloud/deck/nightly/deck_nightly_2017-12-07.tar.gz
an error occured: Other("No package found")

@ChristophWurst

@juliusknorr
Copy link
Author

juliusknorr commented Dec 7, 2017

Ah I needed to package the nightly build with krankerl as well. Maybe it should be possible to use a prebuild package url and just download it if it is not present in the build/artifacts/ directory.

@ChristophWurst
Copy link
Owner

Did you run krankerl package before? Does build/artifacts/<appid>.tar.gz exist?

The error originates from

return Err(error::Error::Other("No package found".to_string()));
.

@ChristophWurst
Copy link
Owner

Ah I needed to package the nightly build with krankerl as well. Maybe it should be possible to use a prebuild package url and just download it if it is not present in the build/artifacts/ directory.

Yup, right. What package URL do you mean?

@juliusknorr
Copy link
Author

Well, I run a remote cron to build nightly tarballs so i just tried to upload it as a release with krankerl without building it locally.

@ChristophWurst
Copy link
Owner

Right now you could trick krankerl by simply downloading the tarball to build/artifacts. I could be wrong but it should be possible to release a nightly build this way, although it's a bit hacky.

@ChristophWurst ChristophWurst changed the title Error when trying to upload a nightly release Allow publishing of externally built tarballs Dec 7, 2017
@juliusknorr
Copy link
Author

Yep, that worked fine. I guess it kind of makes sense from a security point of view to require a local build. Maybe it should not download by default but a possible download CLI parameter would be great, or at least showing an error message that gives a bit more detail on what's wrong.

@ChristophWurst
Copy link
Owner

not download by default but a possible download CLI parameter would be great

Something like krankerl publish --nightly --download <url>? Then, if the --download flag is set, it does not use the local build, but download the file from the given URL, sign it and register the release on the app store.

@juliusknorr
Copy link
Author

Exactly. That would give the possibility to do so but not download it automatically.

@ChristophWurst
Copy link
Owner

I'm considering making this the default. Because I noticed a little flaw in the current process: the tool uses the locally available artifact to generate the tarball signature.

This fails

  • Package tarball 1
  • Upload tarball 1 to github
  • Package another tarball 2
  • Publish with URL from github asset
  • 💥 app store can't verify signature

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

No branches or pull requests

2 participants