Skip to content

Conversation

@bbhtt
Copy link
Contributor

@bbhtt bbhtt commented Jul 17, 2025

A new v6 tag should be created if this is merged.

@bbhtt bbhtt force-pushed the bbhtt/bundle-sources branch 2 times, most recently from b6c3682 to 6e850f7 Compare July 17, 2025 08:36
@bbhtt
Copy link
Contributor Author

bbhtt commented Aug 18, 2025

ping @bilelmoussaoui

`--default-branch=${branch}`,
`--arch=${config.arch}`
]
if (config.arch === 'x86_64' && config.bundleSources !== false) {
Copy link
Member

Choose a reason for hiding this comment

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

Nothing says in the readme that this would only be enabled for x86_64

Copy link
Member

Choose a reason for hiding this comment

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

I feel like this arch condition should be removed and you could set bundle-source: ${arch} == 'x86_64'` or so?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No we don't want everyone uploading sources from both arches to Flathub.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added to readme.

Copy link
Member

Choose a reason for hiding this comment

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

But what if someone wants to build an aarch64 only app? We can potentially add a flathub mode that enforces specific configs for it or something like that

Copy link
Contributor Author

Choose a reason for hiding this comment

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

A flathub mode won't prevent if people can tweak some action options and do it anyway.

@bbhtt bbhtt force-pushed the bbhtt/bundle-sources branch from 6e850f7 to caba4a6 Compare August 18, 2025 11:28
@bbhtt
Copy link
Contributor Author

bbhtt commented Aug 18, 2025

Patched flatpak-builder, should've done that earlier actually...

@bbhtt bbhtt closed this Aug 18, 2025
@bbhtt bbhtt deleted the bbhtt/bundle-sources branch August 18, 2025 15: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