Bug 216579 - [patch] devel/py-boto: Enable concurrent installs
Summary: [patch] devel/py-boto: Enable concurrent installs
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Ruslan Makhmatkhanov
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-01-29 19:19 UTC by John W. O'Brien
Modified: 2017-02-05 16:15 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (rm)

Attachments
devel/py-boto: Enable concurrent installs (533 bytes, patch)
2017-01-29 19:19 UTC, John W. O'Brien 		john: maintainer-approval?
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description John W. O'Brien 2017-01-29 19:19:00 UTC 
Created attachment 179404 [details]
devel/py-boto: Enable concurrent installs

devel/py-boto installs twenty one command line utilities to ${PREFIX}/bin.

The attached patch enables "concurrent" and bumps PORTREVISION due to the new, version-qualified script installed under the default build.

QA
==

portlint: OK
poudriere: OK -- tested on 10.3R amd64 with py27 (default), py34, py35, and py36
Comment 1 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2017-01-29 19:38:57 UTC 
(In reply to John W. O'Brien from comment #0)
Hi John,

what particular script do you mean? "new, version-qualified script installed under the default build"
Comment 2 John W. O'Brien 2017-01-29 20:43:37 UTC 
(In reply to Ruslan Makhmatkhanov from comment #1)

Should have said "scripts" because there are lots.

Without concurrent the plist for py27-boto (the default) includes, e.g.,

    /usr/local/bin/asadmin
    /usr/local/bin/bundle_image
    /usr/local/bin/cfadmin
    ...

With concurrent it changes to

    /usr/local/bin/asadmin
    /usr/local/bin/asadmin-2.7
    /usr/local/bin/bundle_image
    /usr/local/bin/bundle_image-2.7
    /usr/local/bin/cfadmin
    /usr/local/bin/cfadmin-2.7
    ...
Comment 3 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2017-01-29 20:51:43 UTC 
(In reply to John W. O'Brien from comment #2)
Ah, so you just need them to contain python version in scripts name?
What the point of that? Do you need to install py36-boto and py27-boto in parallel?
Comment 4 John W. O'Brien 2017-01-29 20:52:47 UTC 
(In reply to Ruslan Makhmatkhanov from comment #3)

Yes, that's the point of 'concurrent'.
Comment 5 commit-hook freebsd_committer freebsd_triage 2017-02-05 16:13:38 UTC 
A commit references this bug:

Author: rm
Date: Sun Feb  5 16:13:31 UTC 2017
New revision: 433425
URL: https://svnweb.freebsd.org/changeset/ports/433425

Log:
  devel/py-boto: enable concurrent install

  Enable concurrent feature to let boto be installed simultaneously both
  with python2 and python3.

  PR:		216579
  Submitted by:	John W. O'Brien <john@saltant.com>

Changes:
  head/devel/py-boto/Makefile
Comment 6 Ruslan Makhmatkhanov freebsd_committer freebsd_triage 2017-02-05 16:15:25 UTC 
Committed, thank you!