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

Move ostree:// parsing to daemon #3199

Merged
merged 1 commit into from
Nov 3, 2021
Merged

Conversation

jlebon
Copy link
Member

@jlebon jlebon commented Nov 2, 2021

Follow-up to df8315a (#3157).

GNOME Software uses the D-Bus API directly, not the CLI. So let's move
handling of ostree:// there.

The test added in #3157 will verify that we're parsing it correctly.

Closes: #3192

Follow-up to df8315a (coreos#3157).

GNOME Software uses the D-Bus API directly, not the CLI. So let's move
handling of `ostree://` there.

The test added in coreos#3157 will verify that we're parsing it correctly.

Closes: coreos#3192
@cgwalters
Copy link
Member

Ah, yes oops. My bad for not actually testing gnome-software here 😦

@lucab lucab enabled auto-merge November 2, 2021 17:52
@lucab lucab merged commit fc6e7e2 into coreos:main Nov 3, 2021
@tpopela
Copy link

tpopela commented Nov 3, 2021

@cgwalters @jlebon does this mean that the rebases from F34 to F35 through GNOME Software won't work? Becase we are getting some reports about it. If so, can you please do rpm-ostree hotfix build for F34?

@lucab
Copy link
Contributor

lucab commented Nov 3, 2021

@tpopela yes, specifically the combination of an ostree:// prefix in the refspec and the rebase going through DBus.
rpm-ostree-2021.13-2.fc34 is currently building, with this PR backported.

@tpopela
Copy link

tpopela commented Nov 3, 2021

thank you @lucab ! Once there will be a bodhi update created please put the link here.

@jlebon jlebon deleted the pr/fix-rebase-fsb branch April 23, 2023 23:31
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.

Can't upgrade to F35 via GUI
4 participants