Open GoogleCodeExporter opened 9 years ago
Hi,
I like your enhancements. Unfortunately, I was not able to apply the patches. I
used the following command in terminal and received the attached error messages:
patch -p0 -i ./folders2flickr.patch
Cheers
Original comment by tue...@gmail.com
on 18 Jul 2013 at 2:50
Attachments:
Hi,
Thank you!
I have generated the svn patch by using Eclipse Subclipse plugin. I'll generate
it again and post it here. I actually did more changes to this, some minor
changes but still in the same philosophy (more logs, error handler when EXIF is
not present in files, etc). I don't have the code here so I'd post it later
Today or Tomorrow.
Regards
Original comment by gsfernan...@gmail.com
on 18 Jul 2013 at 2:54
Great to have these enhancements, and good that you are at least able to
execute the code.
There are a lot of people struggling to use this software because of issue 24.
Can you help with that one as well?
Original comment by pet...@gmail.com
on 19 Jul 2013 at 10:49
Thank you, I'm still fixing some issues that I found in the enhancements added.
As soon as this code changes are stable I would like to upgrade the code to
Python 3.x
Just watch for my comment on this ticket to get news about the upgrade,
hopefully I would change it in the next weeks.
Regards
Original comment by gsfernan...@gmail.com
on 19 Jul 2013 at 12:24
Hi,
I have fixed a few issues that I found in these features.
- While appending photo ids to the "locallyDeleted" set it was using the photo
object to build the list of ids instead of id values
- Improved logic to manage number of uploads per execution, now it will check
upload by upload if the limit has been reached instead of a folder basis. This
will avoid to process the whole directory and then check for number of uploaded
files.
- Improved error messages to show the whole array of messages instead of first
element only, this will allow users to understand the errors
With respect to the patch file, let me know if you can apply it to your working
copy.
Regards,
Guillermo
Original comment by gsfernan...@gmail.com
on 22 Jul 2013 at 5:09
Attachments:
Hi,
thanks for the update. I never worked with eclipse before, therefore I failed
to patch the files. I used Eclipse 3.8, which comes with the latest Linux mint
distribution 15.
I am not sure how to go forward.
Regards
Original comment by tue...@gmail.com
on 22 Jul 2013 at 9:39
Hi,
It doesn't matter which tool I used to generate the patch, it should work in
your environment too.
Try this, go to your folders2flickr checkout directory, copy the patch file
that I attached to the same folder and then run the command:
patch -p0 -i folders2flickr.issue29.v2.patch
Let me know if it works, if not I will attach the .zip file with the sources to
the ticket so you could download it.
Regards
Original comment by gsfernan...@gmail.com
on 22 Jul 2013 at 9:59
You could try with svn patch instead:
http://svnbook.red-bean.com/en/1.7/svn.ref.svn.c.patch.html
svn patch PATCHFILE [WCPATH]
Regards,
Guillermo
Original comment by gsfernan...@gmail.com
on 22 Jul 2013 at 10:00
Hi,
I downloaded a new copy of folders2flickr and applied the patch command above.
But it did not work out, a lot errors appeared (only tagging.py worked). Also
svn did not do the trick. The sources would be very helpful.
Cheers
Original comment by tue...@gmail.com
on 23 Jul 2013 at 6:23
Hi,
That's strange, I'll take a look to it later. Meantime, here is the source code
with the changes applied so you could use it.
Regards
Original comment by gsfernan...@gmail.com
on 23 Jul 2013 at 11:02
Attachments:
Hi,
I reverted my working copy to the latest version in trunk and applied the patch
without any problems by using svn patch command.
Regards,
Guillermo
Original comment by gsfernan...@gmail.com
on 23 Jul 2013 at 12:57
Hi pet...@gmail.com
I have tried to migrate the code to python 3 but it is too complex for me, I
don't have experience in python and seems that many changes are needed in order
to make this code work on python 3.
Regards,
Guillermo
Original comment by gsfernan...@gmail.com
on 24 Jul 2013 at 8:39
Original issue reported on code.google.com by
gsfernan...@gmail.com
on 8 Jul 2013 at 3:00Attachments: