Re: Flickr export working with the new Auth system in Flickr (Yahoo!)



On Mon, 2005-10-17 at 23:10 +0200, Alvaro del Castillo wrote:
> Hi!
> 
> I have a working version of F-Spot that can export to Flickr using the
> new Auth system. It is based in the FlickrNet library. With this new
> system you can use the new Yahoo! accounts and also, the old Flickr
> accounts so I think we can use only this way.
> 
> You can see how it works in this screenshot tutorial with 9 steps:
> 
> http://acs.barrapunto.org/~acs/flickrSpot/

Nice work!  Are all the auth steps really required now? They really made
that harder than it used to be, what a pain.  Is there any way we could
add extra code to simplify things?

> 
> Any comments will be welcomed and if this feature is going to be in
> Flickr in the future, we need to define how to do it. In the past I have
> sent several patches, for example for the Duplicate detect feature, and
> finally they haven't reached upstream without a clear reason for it.
> Maybe it is just lack of time from Larry to review the patches. Larry, I
> have write permissions in GNOME CVS repository so I can do the work to
> commit the code to F-Spot repository but I suppose you want to review
> the code and be sure it is right to be integrated upstream. I could
> create a patch for this Flickr feature against current CVS but I would
> do it if finally is decided to integrate it upstream.
> 

The slow patch review is all me, and I apologize.  I'll send out some
comments today.  

> You can test it working right now in my unofficial F-Spot repository at:
> 
> https://acs.barrapunto.org/svn/f-spot/

Glancing over the changes there the first thing I notice is a lot of
whitespace changes and 4 space tabs, please use 8 space tabs and don't
align variable names at start of functions (it leads to too much
reformatting to be useful).  I've checked in a HACKING file with some of
the rules on codeing style.  Beyond that the import changes look pretty
cool, but I haven't looked deeply at them yet.

> Also, it will be great if the Duplicate detect feature is also
> integrated upstream. I feel people find it useful and the final design
> have followed the comments in the list.
> 

Like I said I'll follow up with some comments on duplicates later today.

--Larry




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]