> var native_path = true;
> try {
> var client = GConf.Client.get_default();
> var backend_name = client.get_string(DejaDup.BACKEND_KEY);
>
> if (backend_name == "s3")
> native_path = false;
> else if (backend_name == "file")
> {
> var path = DejaDup.BackendFile.get_location_from_gconf();
> var backend_file = File.parse_name(path);
> native_path = backend_file.is_native();
> }
> }
>
> What do you think of this?
Note that while get_default() does take a Window argument, it is "Gtk.Window?
win" -- the question mark means it's optional. You can just pass "null".
But truthfully, though I don't like the duplication of code, it's probably
worth it here if it means dropping the libdeja-dup dependency. Backends don't
get added everyday. We can either separate out parts of libdeja-dup later or
just deal with the duplication. This would let us merely add dbus as a
dependency to the monitor.
You can leave the translation changes in, just as long as I can drop them when
I merge (i.e. I don't have to pick out meaningful changes from non-meaningful).
I don't think you made any changes to user strings. And the whitespace stuff
is fine too.
--
https://code.edge.launchpad.net/~temposs/deja-dup/network-manager/+merge/11123
Your team Déjà Dup Maintainers is subscribed to branch lp:deja-dup.
_______________________________________________
Mailing list: https://launchpad.net/~deja-dup-team
Post to : [email protected]
Unsubscribe : https://launchpad.net/~deja-dup-team
More help : https://help.launchpad.net/ListHelp
