-
Notifications
You must be signed in to change notification settings - Fork 191
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
Add Flatpak support #2526
Add Flatpak support #2526
Conversation
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 will fix the replace()
for you, so no further action needed from you. Thanks for the patch!
bodhi/server/buildsys.py
Outdated
@@ -261,6 +261,16 @@ def getBuild(self, build='TurboGears-1.0.2.2-2.fc17', other=False, testing=False | |||
} | |||
break | |||
|
|||
if token.endswith("flatpak"): | |||
tag = "f%s-updates-testing" % token.replace("fc", "").replace("", "") |
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.
Let's drop the replace()
call if it's not needed.
class FlatpakComposerThread(ContainerComposerThread): | ||
"""Use skopeo to copy and tag flatpak images.""" | ||
|
||
ctype = ContentType.flatpak |
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.
Heh, looks like patching this file was pretty easy ☺
@@ -1075,9 +1080,11 @@ def get_pkg_committers_from_pagure(self): | |||
""" | |||
pagure_url = config.get('pagure_url') | |||
# Pagure uses plural names for its namespaces such as "rpms" except for | |||
# container | |||
# container. Flatpaks build directly from the 'modules' namespace |
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.
Ah, so I take this to mean that ACLs on a flatpak are identical to having ACLs to the module of the same name?
Add a new content type for flatpaks, which is inferred from the Koji metadata. Flatpaks are mashed exactly the same as containers - the content type is different mostly to get a non-confusing display of content type in the user interface and to allow future enhancements like Flatpak-specific testing instructions. Signed-off-by: Owen W. Taylor <otaylor@fishsoup.net>
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.
Thanks for the patch!
These patches are planned to be included in the upcoming 3.10.0 release: #2556. |
These patches have been deployed to Fedora's staging Bodhi instance: |
Add a new content type for flatpaks, which is inferred from the Koji metadata.
Flatpaks are mashed exactly the same as containers - the content type is
different mostly to get a non-confusing display of content type in the user
interface and to allow future enhancements like Flatpak-specific testing
instructions.
Signed-off-by: Owen W. Taylor otaylor@fishsoup.net