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

Update and integrate appstream file #730

Merged
merged 1 commit into from
Mar 18, 2019
Merged

Update and integrate appstream file #730

merged 1 commit into from
Mar 18, 2019

Conversation

flub
Copy link
Member

@flub flub commented Mar 17, 2019

The appstream/appdata file is required for linux destop appliciations
and is not packaging-format specific so makes sense to include
upstream.

  • Update existing appdata file.

  • Add .desktop file since this is referred to by the appdata.

  • Update the travis config to test this appdata and .desktop files
    making sure they remain correct. We have to add linux back to the
    job matrix for this, but we only want to publish a binary release
    using electron-builder on OSX.

@flub flub force-pushed the appstream branch 10 times, most recently from 11b051f to 4a48ac6 Compare March 17, 2019 20:21
The appstream/appdata file is required for linux destop appliciations
and is not packaging-format specific so makes sense to include
upstream.

- Update existing appdata file.

- Add .desktop file since this is referred to by the appdata.

- Update the travis config to test this appdata and .desktop files
  making sure they remain correct.  We have to add linux back to the
  job matrix for this, but we only want to publish a binary release
  using electron-builder on OSX.

- Skip running the integration test on Linux, it doesn't want to work
  and I'm not in the business of fixing that and life is too short.

Fixes #729.
@flub
Copy link
Member Author

flub commented Mar 17, 2019

I experimented with multiple jobs instead of the default matrix as well, but the main downside is there is no data shared between jobs so i stuck with the not overly nice main matrix thing instead. The ugly part of that is how i select things which are only for one platform, but it works.

@flub flub merged commit 0c98ccf into master Mar 18, 2019
@flub flub mentioned this pull request Mar 18, 2019
@flub flub deleted the appstream branch March 18, 2019 18:54
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.

3 participants