deja-dup-team team mailing list archive
-
deja-dup-team team
-
Mailing list archive
-
Message #00034
Re: [Merge] lp:~temposs/deja-dup/network-manager into lp:deja-dup
The code is ready for another looking over :-)
-Andrew
On Sun, Sep 6, 2009 at 10:26 PM, Michael Terry
<michael.terry@xxxxxxxxxxxxx>wrote:
> > 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<https://code.edge.launchpad.net/%7Etemposs/deja-dup/network-manager/+merge/11123>
> You are the owner of lp:~temposs/deja-dup/network-manager.
>
--
https://code.launchpad.net/~temposs/deja-dup/network-manager/+merge/11123
Your team Déjà Dup Maintainers is subscribed to branch lp:deja-dup.
References