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

Fix AppImageUpdater build #11971

Merged
merged 1 commit into from
Nov 12, 2024
Merged

Fix AppImageUpdater build #11971

merged 1 commit into from
Nov 12, 2024

Conversation

erikjv
Copy link
Collaborator

@erikjv erikjv commented Nov 7, 2024

No description provided.

@erikjv erikjv added the AppImage label Nov 7, 2024
@erikjv erikjv added this to the Desktop 6.0 milestone Nov 7, 2024
@erikjv erikjv self-assigned this Nov 7, 2024
@DeepDiver1975
Copy link
Member

Can we add a build step to the GitHub workflows to build the app image so that we prevent similar issues in the future?

@TheOneRing
Copy link
Contributor

BTW this is wrong

@DeepDiver1975
Copy link
Member

BTW this is wrong

brilliant - this is why I want this to be compiled on ci ....

Copy link
Member

@DeepDiver1975 DeepDiver1975 left a comment

Choose a reason for hiding this comment

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

as per comment from @TheOneRing - wrong ....

@erikjv erikjv force-pushed the work/fix-linux-build branch from 481264a to 6042420 Compare November 8, 2024 11:25
@erikjv erikjv requested a review from DeepDiver1975 November 8, 2024 12:56
@erikjv
Copy link
Collaborator Author

erikjv commented Nov 8, 2024

as per comment from @TheOneRing - wrong ....

I removed the method declaration, as I wrote in the new commit message:

The method handleStartup was renamed to validateUpdate, and is now
handled in the Updater base class. The AppImageUpdater does not need
to override it.

@erikjv erikjv force-pushed the work/fix-linux-build branch from 6042420 to 78fc97d Compare November 8, 2024 13:30
@erikjv
Copy link
Collaborator Author

erikjv commented Nov 8, 2024

(I fixed the commit message in the last push)

@DeepDiver1975
Copy link
Member

cherry-picked ci change from #11975 to see it truly compiling it now ...

@DeepDiver1975
Copy link
Member

compiler issue gone - but the package step is now failing .... Monday 👋

The method `handleStartup` was renamed to `validateUpdate`, and is now
handled in the `Updater` base class. The `AppImageUpdater` does not need
to override it.
@erikjv erikjv enabled auto-merge (rebase) November 12, 2024 16:55
@erikjv erikjv merged commit 9a5ab45 into master Nov 12, 2024
6 checks passed
@erikjv erikjv deleted the work/fix-linux-build branch November 12, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants