-
Notifications
You must be signed in to change notification settings - Fork 36
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
Rename D-Installer to Agama #507
Conversation
97042bc
to
f1ed0b3
Compare
@@ -54,7 +54,7 @@ def commit | |||
end | |||
|
|||
def current_installation_phase | |||
dbus_phase = dbus_object["org.opensuse.DInstaller.Manager1"]["CurrentInstallationPhase"] | |||
dbus_phase = dbus_object["org.opensuse.Agama1.Manager"]["CurrentInstallationPhase"] |
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.
watch out, the autorename did not work right here
it should be Agama.Manager1
for consistency with the other interfaces
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.
Not exactly. We have a root service, which is called org.opensuse.Agama1. Manager belongs to that service. We might consider renaming the service in the future, but I would keep it as It is by now.
Thanks for checking the changes!
.DInstaller.Storage1.DASD.Format | ||
.Agama1.ServiceStatus | ||
.Agama1.Progress | ||
.Agama1.Validation |
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.
also these 3, I believe we want .Agama.Foo1
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.
Not really, the idea is to have the version number in the service name.
@@ -138,7 +138,7 @@ def safe_run(&block) | |||
end | |||
|
|||
def busy_error | |||
::DBus.error("org.opensuse.DInstaller.Error.Busy") | |||
::DBus.error("org.opensuse.Agama1.Error.Busy") |
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.
::DBus.error("org.opensuse.Agama1.Error.Busy") | |
::DBus.error("org.opensuse.Agama.Error1.Busy") |
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.
The same that the rest.
Closed in favor of #509. |
D-Installer is now Agama. So let's try to rename everything to the new name.