Created attachment 206830 [details]
I have just create OCI Python SDK port, you can find .shar file attached to this bug.
The Python SDK enables you to write code to manage Oracle Cloud Infrastructure resources.
Please let me know if there's anything else you need from me or if you have any comments.
Thanks for the new port submissions Allessandro.
A couple of review items:
- PORTREVISION shouldn't be set for a new port
- Port should be py-prefixed (directory name)
- Use USE_GITHUB/GH_* variables instead of direct to github MASTER_SITES 
Note: DISTNAME shouldn't be necessary if you use DISTVERSIONPREFIX. See Example 5.12 of the porters handbook section on USE_GITHUB below. WRKSRC and EXTRACT_SUFX shouldn't be necessary either
Please also, confirm this port passes QA (portlint and poudriere in particular), as it will pick up many other porting compliance issues and qualify the full packaging process. For more information and instructions see: https://www.freebsd.org/doc/en/books/porters-handbook/testing.html
Created attachment 206841 [details]
new shar file and logs from testing commands
Thanks for your prompt response Kubilay! I have made requested modifications to the port and attached new shar file and logs from testing section to this bug.
Please let me know if you have any further comments on it.
(In reply to Alessandro Sagratini from comment #3)
Thank you for following-up. If you could include the shar and logs as separate (and uncompressed) files, that would be great
Created attachment 206872 [details]
Created attachment 206873 [details]
portlint -A output
Created attachment 206874 [details]
Created attachment 206875 [details]
output from make describe
Created attachment 206876 [details]
output from port test
(In reply to Kubilay Kocak from comment #4)
Thanks for getting back to me Kibilay, this is done, please let me know if you need anything else from me.
Thanks Kubilay for the feedback on this port: is there anything else I should do or do we have to wait for further feedback in order to move forward?
(In reply to Alessandro Sagratini from comment #11)
Everything looks in order, just waiting on a developer to commit
(In reply to Alessandro Sagratini from comment #10)
thank you for your contribution. I've only two questions at the moment:
- Version 2.5 was released recently. If you like I can change the patch to that version in my local repository.
- The email address here at Bugzilla should match with the one in the MAINTAINER field. If you want to use the "@hotmail.com" address I can change it in my local repository. Otherwise if you want to use the "@gmail.com" address you should change the email address here at Bugzilla.
I've also opened an issue at the upstream repository because the testsuite in "tests/" fails if it run outside an virtualenv and the `tests/` aren't available in the sdist via PyPI.
But don't worry, the last paragraph isn't a showstopper. Once you can confirm me which email address should be used and if the newer version (= 2.5) of "devel/py-oci" may be used, we're all set to commit the patch.
Thanks for your prompt feedback on this Kai! Please find answers below:
1. Yes, version 2.5.0 was released yesterday, I agree it's good to have most recent version in the ports tree, these are sha256 and size for 2.5.0 version from make makesum
SHA256 (oracle-oci-python-sdk-v2.5.0_GH0.tar.gz) = 59a37653c36542f55de13828ae50ce287d2c5c1f3f6723116c49d4bf86545192
SIZE (oracle-oci-python-sdk-v2.5.0_GH0.tar.gz) = 3170558
2. It is ok for me to use @hotmail.com rather than @gmail.com in MAINTAINER field
Let me know if there's anything else you need from me.
A commit references this bug:
Date: Wed Sep 18 17:14:35 UTC 2019
New revision: 512279
[NEW PORT]: devel/py-oci
The Python SDK enables one to write code to manage Oracle Cloud Infrastructure
resources. Following services are supported:
* Announcements, Audit, Budgets
* Container Engine for Kubernetes
* Compute Autoscaling/Work Requests
* Content and Experience
* Core Services (Networking, Compute, Block Volume)
* Data Transfer, Database, DNS
* Email Delivery, Events
* File Storage, Functions
* Health Checks, IAM (= Identity and Access Management)
* Key Management
* Limits, Load Balancing
* Monitoring, Notifications
* Object Storage, Quotas
* Resource Manager, Search, Streaming
* Web Application Acceleration and Security
Submitted by: Alessando Sagratini <email@example.com>
Reviewed by: koobs
(In reply to Alessandro Sagratini from comment #14)
Committed, once again thank you for your contribution!
I did only some minor fixes:
- Synced the RUN_DEPENDS as given in setup.py
- Changed PORTNAME to "oci" to match the package name at PyPI
- Added UPL license information (the package is dual licensed)
- Improved the pkg-descr a bit to make it more meaningful for the users
- Changed MASTER_SITES to CHEESESHOP because it's the preferred way for Python packages (GitHub is used mostly as an alternative, e.g. for the case if testsuites are missing)
The latter would apply for devel/py-oci but the testsuite in the GitHub tarball isn't usable at the moment. Thus I added a remark for future updates to revise that point again.