Bug 213353 - net/py-s3transfer: Update to 0.1.9
Summary: net/py-s3transfer: Update to 0.1.9
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Torsten Zuehlsdorff
URL: https://github.com/boto/s3transfer/bl...
Keywords: easy
Depends on:
Blocks: 213755
  Show dependency treegraph
 
Reported: 2016-10-10 07:38 UTC by Bradley T. Hughes
Modified: 2016-11-04 11:46 UTC (History)
2 users (show)

See Also:


Attachments
patch from a git commit (1.11 KB, patch)
2016-10-10 07:38 UTC, Bradley T. Hughes
bhughes: maintainer-approval+
Details | Diff
0.1.8 (1.13 KB, patch)
2016-10-13 11:47 UTC, Bradley T. Hughes
bhughes: maintainer-approval+
Details | Diff
0.1.9 (1.11 KB, patch)
2016-10-24 19:57 UTC, Bradley T. Hughes
bhughes: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bradley T. Hughes freebsd_committer freebsd_triage 2016-10-10 07:38:32 UTC
Created attachment 175597 [details]
patch from a git commit

Bump to the latest upstream release.

https://github.com/boto/s3transfer/blob/0.1.7/CHANGELOG.rst
Comment 1 Bradley T. Hughes freebsd_committer freebsd_triage 2016-10-10 10:36:59 UTC
portlint: OK (-C, looks fine.)
testport: OK (poudriere: 9.3-R, 10.[123]-R, 11.0-RC3, amd64/i386, default options)
Comment 2 Bradley T. Hughes freebsd_committer freebsd_triage 2016-10-13 11:47:44 UTC
Created attachment 175701 [details]
0.1.8

New upstream release, bumping to that version.
Comment 3 Bradley T. Hughes freebsd_committer freebsd_triage 2016-10-19 18:18:48 UTC
Torsten: do I need to do anything to get this ready for commit? Everything should be in order, since it's a maintainer update, right?
Comment 4 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-10-20 13:14:21 UTC
Hello Bradley,

yes everything is in order and correct. Thanks for your fine work! But since i'm new to the committer-team i need approval of my mentors for every commit. I already requested the approval and now i'm waiting for the "go" :)
Comment 5 Bradley T. Hughes freebsd_committer freebsd_triage 2016-10-20 13:15:46 UTC
Sounds good! :)
Comment 6 Bradley T. Hughes freebsd_committer freebsd_triage 2016-10-24 19:57:12 UTC
Created attachment 176114 [details]
0.1.9

Upstream has released 0.1.9, updated patch accordingly.
Comment 7 commit-hook freebsd_committer freebsd_triage 2016-10-25 08:53:47 UTC
A commit references this bug:

Author: tz
Date: Tue Oct 25 08:52:46 UTC 2016
New revision: 424607
URL: https://svnweb.freebsd.org/changeset/ports/424607

Log:
  net/py-s3transfer: upgrade 0.1.5 to 0.1.8

  Changelog:

  0.1.8
  - feature:download: Support downloading to FIFOs.

  0.1.7
  - bugfix:TransferManager: Fix memory leak when using same client to create multiple TransferManagers

  0.1.6
  - bugfix:download: Fix issue where S3 Object was not downloaded to disk when empty

  PR:           213353
  Submitted by: Bradley T. Hughes (maintainer)
  Approved by:  swills (mentor)

Changes:
  head/net/py-s3transfer/Makefile
  head/net/py-s3transfer/distinfo
Comment 8 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-10-25 08:55:56 UTC
(In reply to Bradley T. Hughes from comment #6)

> Upstream has released 0.1.9, updated patch accordingly.

Thank you for your fast and good work! I'm sorry it took so long - i have 3 mentors but all of them were busy until today. 

Since they reviewed the patch for 0.1.8 and the new patch would need a new review by them, i just committed the version 0.1.8. Hopefully this will help.

I would suggest to create a new PR for the new update. But if you want i can have a look at the new patch too ;)

Greetings,
Torsten
Comment 9 Bradley T. Hughes freebsd_committer freebsd_triage 2016-10-25 09:19:00 UTC
Can't we just leave this open with the updated patch? :)
Comment 10 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-10-25 11:34:33 UTC
(In reply to Bradley T. Hughes from comment #9)

> Can't we just leave this open with the updated patch? :)

We can. :)
Comment 11 Bradley T. Hughes freebsd_committer freebsd_triage 2016-11-03 09:38:00 UTC
Any chance we can get the 0.1.9 patch committed? It blocks updating devel/awscli to the latest upstream version. Thanks :)
Comment 12 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-11-03 16:22:35 UTC
I'm sorry. I was delayed because my wife got ill. I'm back today and starting to take care of all my PRs. This one shouldn't take long :) Tests are running :)
Comment 13 Bradley T. Hughes freebsd_committer freebsd_triage 2016-11-03 21:28:09 UTC
Hi Torsten! Sorry to hear about your wife, now I feel bad for nagging. Thanks for the update. :)

Let me know if you need me to test anything more than what I've already done (all supported FreeBSD RELEASEs on amd64 and i386).

Thanks again!
Comment 14 commit-hook freebsd_committer freebsd_triage 2016-11-04 11:34:21 UTC
A commit references this bug:

Author: tz
Date: Fri Nov  4 11:33:27 UTC 2016
New revision: 425314
URL: https://svnweb.freebsd.org/changeset/ports/425314

Log:
  net/py-s3transfer: Update from 0.1.8 to 0.1.9

  Changelog:
  - Add override flag to transfer coordinator
  - Don't override exceptions and only set them after done
  - Set exception on future

  PR:           213353
  Submitted by: Bradley T. Hughes (maintainer)
  Approved by:  pi (mentor)

Changes:
  head/net/py-s3transfer/Makefile
  head/net/py-s3transfer/distinfo
Comment 15 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-11-04 11:36:24 UTC
> Sorry to hear about your wife, now I feel bad for nagging.

No need for feeling sorry. You couldn't have know it - and this is how live is going.

AND finally: committed! Thanks for your patch and patience! :)
Comment 16 Bradley T. Hughes freebsd_committer freebsd_triage 2016-11-04 11:46:48 UTC
Thanks to you as well, Torsten. Have a nice day! :)