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

Add Sparkle distribution engine #481

Closed
wants to merge 28 commits into from
Closed

Conversation

ezekg
Copy link
Member

@ezekg ezekg commented Oct 25, 2021

Related to #478. Requires #470 to be able to read a product.

@ezekg
Copy link
Member Author

ezekg commented Oct 26, 2021

What about deltas? Could eventually define a deltas relationship to releases...?

<item>
   <title>Version 2.0 </title>
   <link>https://myproductwebsite.com</link>
   <sparkle:version>2.0</sparkle:version>
   <description>foo bar baz</description>
   <pubDate>Wed, 09 Jan 2006 19:20:11 +0000</pubDate>
   <enclosure url="http://you.com/2.0_full.zip"
              length="123"
              type="application/octet-stream"
              sparkle:edSignature="..." />
   <sparkle:deltas>
       <enclosure url="http://you.com/1.5-2.0.delta"
                  sparkle:deltaFrom="1.5"
                  length="1485"
                  type="application/octet-stream"
                  sparkle:edSignature="..." />
       <enclosure url="http://you.com/1.6-2.0.delta"
                  sparkle:deltaFrom="1.6"
                  length="1428"
                  type="application/octet-stream"
                  sparkle:edSignature="..." />
   </sparkle:deltas>
</item>

@ezekg
Copy link
Member Author

ezekg commented Oct 26, 2021

I'm not sure if Sparkle will work with relative enclosure URLs.

@ezekg ezekg force-pushed the feature/sparkle-electron-engines branch from df5bfbe to d487163 Compare October 26, 2021 17:38

include Rails.application.routes.url_helpers

def initialize(account:, product:, releases:)
def initialize(account:, product:, releases:, host: 'api.keygen.sh')
Copy link
Member Author

Choose a reason for hiding this comment

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

Maybe we can eventually use Current.host here, if we get traction with custom domains.

@@ -4,19 +4,21 @@

class GenerateAppcastService < BaseService
SUPPORTED_FILETYPES = %i[zip dmg pkg mpkg tar.gz tar.bz2].freeze
SUPPORTED_PLATFORMS = %i[macos windows].freeze
SUPPORTED_PLATFORMS = %i[macos windows windows-x86 windows-x64].freeze
Copy link
Member Author

Choose a reason for hiding this comment

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

What if we just removed filtering platforms and filetypes? Unsupported items would simply be ignored, and it’d be more forward compatible.

@ezekg ezekg force-pushed the feature/sparkle-electron-engines branch 2 times, most recently from d4f2e94 to d146389 Compare October 27, 2021 13:50
@ezekg ezekg changed the title Add Sparkle and Electron distribution engines Add Sparkle distribution engine Oct 27, 2021
@ezekg
Copy link
Member Author

ezekg commented Dec 16, 2021

Add caching.

@ezekg ezekg force-pushed the master branch 7 times, most recently from ad595e0 to a5b0dc0 Compare February 4, 2022 03:39
@ezekg ezekg force-pushed the master branch 2 times, most recently from 3fc8952 to d515cb6 Compare March 14, 2022 20:38
@ezekg ezekg force-pushed the master branch 2 times, most recently from a78fcc6 to 1462020 Compare March 31, 2023 19:23
@ezekg ezekg force-pushed the master branch 4 times, most recently from b18f142 to ac4cdaf Compare May 3, 2023 22:09
@ezekg ezekg force-pushed the master branch 6 times, most recently from cab84a1 to a390331 Compare June 1, 2023 19:24
@ezekg ezekg marked this pull request as draft June 13, 2023 21:51
@ezekg ezekg closed this Sep 4, 2023
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.

1 participant