-
Notifications
You must be signed in to change notification settings - Fork 52
builder: Export sources extension to repo #226
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
Conversation
b6c3682 to
6e850f7
Compare
|
ping @bilelmoussaoui |
| `--default-branch=${branch}`, | ||
| `--arch=${config.arch}` | ||
| ] | ||
| if (config.arch === 'x86_64' && config.bundleSources !== false) { |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added to readme.
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
6e850f7 to
caba4a6
Compare
|
Patched flatpak-builder, should've done that earlier actually... |
A new v6 tag should be created if this is merged.