Skip to content

heimdall: update to 2.2.2#60116

Merged
Duncaen merged 1 commit into
void-linux:masterfrom
r0man1an:heimdall-update
Apr 26, 2026
Merged

heimdall: update to 2.2.2#60116
Duncaen merged 1 commit into
void-linux:masterfrom
r0man1an:heimdall-update

Conversation

@r0man1an
Copy link
Copy Markdown
Contributor

@r0man1an r0man1an commented Apr 23, 2026

Testing the changes

  • I tested the changes in this PR: YES (tested both the cli version and heimdall-frontend)

Local build testing

  • I built this PR locally for my native architecture, x86_64-glibc
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • aarch64-musl
    • aarch64-glibc
    • armv7l
    • x86_64-musl
    • i686-glibc

@r0man1an
Copy link
Copy Markdown
Contributor Author

I was wondering if it would make sense to split this into two packages — heimdall for the CLI tool and heimdall-frontend as a subpackage for the Qt6 GU. Fedora and Debian do this already. Most users only need the CLI, so making the GUI optional would avoid unnecessary bloat. Happy to implement it if that's the preferred approach.

@Duncaen
Copy link
Copy Markdown
Member

Duncaen commented Apr 24, 2026

I was wondering if it would make sense to split this into two packages — heimdall for the CLI tool and heimdall-frontend as a subpackage for the Qt6 GU. Fedora and Debian do this already. Most users only need the CLI, so making the GUI optional would avoid unnecessary bloat. Happy to implement it if that's the preferred approach.

If you think that makes sense, I don't think anyone would have a problem with it.

@r0man1an
Copy link
Copy Markdown
Contributor Author

I was wondering if it would make sense to split this into two packages — heimdall for the CLI tool and heimdall-frontend as a subpackage for the Qt6 GU. Fedora and Debian do this already. Most users only need the CLI, so making the GUI optional would avoid unnecessary bloat. Happy to implement it if that's the preferred approach.

If you think that makes sense, I don't think anyone would have a problem with it.

Got it! Should I add borh into this PR or split it into two separate PRs?

@r0man1an r0man1an force-pushed the heimdall-update branch 2 times, most recently from 5946da7 to 8e49a8c Compare April 26, 2026 00:07
@r0man1an
Copy link
Copy Markdown
Contributor Author

Great! The heimdall and heimdall-frontend packages are ready to go!

@Duncaen
Copy link
Copy Markdown
Member

Duncaen commented Apr 26, 2026

Isn't is possible to just add a sub package that moves the heimdall-frontend binary?

@r0man1an
Copy link
Copy Markdown
Contributor Author

Isn't is possible to just add a sub package that moves the heimdall-frontend binary?

Done, thanks for your input!

Comment thread srcpkgs/heimdall/template Outdated
hostmakedepends+=" qt5-tools-devel"
fi
heimdall-frontend_package() {
short_desc="Qt6 GUI for heimdall (Samsung firmware flashing tool)"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
short_desc="Qt6 GUI for heimdall (Samsung firmware flashing tool)"
short_desc+=" - Qt6 GUI"

Is usually what we do in cases like this.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

how can I commit this without triggering the Ci to re-run?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think that would be possible.

@r0man1an
Copy link
Copy Markdown
Contributor Author

I'm glad it worked!

@Duncaen Duncaen merged commit 61f802e into void-linux:master Apr 26, 2026
8 checks passed
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.

2 participants