Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

php-mysql-xdevapi: Adding new Pecl extension MySQL X DevAPI for PHP #3887

Merged
merged 1 commit into from Apr 30, 2019

Conversation

kirmorozov
Copy link
Contributor

@kirmorozov kirmorozov commented Mar 19, 2019

Description

MySQL X DevAPI for PHP
Here is how test it

php -r '$session = \mysql_xdevapi\getSession("mysqlx://root:123123qa@127.0.0.1"); print_r($session->getSchemas());'
Type(s)
  • bugfix
  • enhancement
  • security fix
Tested on

macOS 10.14.3 18D109
Xcode 10.1 10B61

Verification

Have you

  • checked your Portfile with port lint?
  • tried existing tests with sudo port test?
  • tried a full install with sudo port -vst install?
  • tested basic functionality of all binary files?

@macportsbot
Copy link

Travis Build #5671 Failed.

Lint results
--->  Verifying Portfile for php-mysql-xdevapi
--->  0 errors and 0 warnings found.

Port php73-mysql-xdevapi fail on xcode10.2. Log
Port php-mysql-xdevapi's dependencies fail on xcode10.2. Log
Port php71-mysql-xdevapi fail on xcode10.2. Log
Port php72-mysql-xdevapi fail on xcode10.2. Log
Port php73-mysql-xdevapi fail on xcode9.4. Log
Port php-mysql-xdevapi's dependencies fail on xcode9.4. Log
Port php71-mysql-xdevapi fail on xcode9.4. Log
Port php72-mysql-xdevapi fail on xcode9.4. Log
Port php73-mysql-xdevapi fail on xcode8.3. Log
Port php-mysql-xdevapi's dependencies fail on xcode8.3. Log
Port php71-mysql-xdevapi fail on xcode8.3. Log
Port php72-mysql-xdevapi fail on xcode8.3. Log
Port php73-mysql-xdevapi fail on xcode7.3. Log
Port php-mysql-xdevapi's dependencies fail on xcode7.3. Log
Port php71-mysql-xdevapi fail on xcode7.3. Log

long_description The X DevAPI is the new common API for MySQL Connectors built on the X Protocol introduced in MySQL 5.7.12.

depends_build-append port:boost
depends_lib-append port:boost
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The depends_build-append line can be removed because it is included in depends_lib-append.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was not sure which once to include and how, I manages to build it from sources manually other day.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed Protobuf, building now.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mf2k Apparently. php extension keep dynamic link to protobuf3.
Can you give a hint how to build it static, and remove dependencies if binary is installed?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not understanding what you are asking. We don't normally like static builds. We also don't generally work with non Macports dependencies. So adding a dependency on protobuf3-cpp is correct.

set cap_name mysql_xdevapi
categories-append net devel
platforms darwin freebsd openbsd
maintainers {ryandesign @ryandesign} openmaintainer
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since Ryan is being added as maintainer, I want him to approve this.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ryandesign Keeps eye on php, so put him here also.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed to nomaintainer

@macportsbot
Copy link

Travis Build #5673 Failed.

Lint results
--->  Verifying Portfile for php-mysql-xdevapi
--->  0 errors and 0 warnings found.

Port php73-mysql-xdevapi fail on xcode10.2. Log
Port php-mysql-xdevapi's dependencies fail on xcode10.2. Log
Port php71-mysql-xdevapi fail on xcode10.2. Log
Port php72-mysql-xdevapi fail on xcode10.2. Log
Port php73-mysql-xdevapi fail on xcode9.4. Log
Port php-mysql-xdevapi's dependencies fail on xcode9.4. Log
Port php71-mysql-xdevapi fail on xcode9.4. Log
Port php72-mysql-xdevapi fail on xcode9.4. Log
Port php73-mysql-xdevapi success on xcode8.3. Log
Port php-mysql-xdevapi success on xcode8.3. Log
Port php71-mysql-xdevapi success on xcode8.3. Log
Port php73-mysql-xdevapi success on xcode7.3. Log
Port php-mysql-xdevapi success on xcode7.3. Log
Port php71-mysql-xdevapi success on xcode7.3. Log
Port php72-mysql-xdevapi success on xcode7.3. Log

@macportsbot
Copy link

Travis Build #5674 Passed.

Lint results
--->  Verifying Portfile for php-mysql-xdevapi
--->  0 errors and 0 warnings found.

Port php73-mysql-xdevapi success on xcode10.2. Log
Port php-mysql-xdevapi success on xcode10.2. Log
Port php71-mysql-xdevapi success on xcode10.2. Log
Port php72-mysql-xdevapi success on xcode10.2. Log
Port php73-mysql-xdevapi success on xcode9.4. Log
Port php-mysql-xdevapi success on xcode9.4. Log
Port php71-mysql-xdevapi success on xcode9.4. Log
Port php72-mysql-xdevapi success on xcode9.4. Log
Port php73-mysql-xdevapi success on xcode8.3. Log
Port php-mysql-xdevapi success on xcode8.3. Log
Port php71-mysql-xdevapi success on xcode8.3. Log
Port php73-mysql-xdevapi success on xcode7.3. Log
Port php-mysql-xdevapi success on xcode7.3. Log
Port php71-mysql-xdevapi success on xcode7.3. Log
Port php72-mysql-xdevapi success on xcode7.3. Log

@macportsbot
Copy link

Travis Build #5675 Passed.

Lint results
--->  Verifying Portfile for php-mysql-xdevapi
--->  0 errors and 0 warnings found.

Port php73-mysql-xdevapi success on xcode10.2. Log
Port php-mysql-xdevapi success on xcode10.2. Log
Port php71-mysql-xdevapi success on xcode10.2. Log
Port php72-mysql-xdevapi success on xcode10.2. Log
Port php73-mysql-xdevapi success on xcode9.4. Log
Port php-mysql-xdevapi success on xcode9.4. Log
Port php71-mysql-xdevapi success on xcode9.4. Log
Port php72-mysql-xdevapi success on xcode9.4. Log
Port php73-mysql-xdevapi success on xcode8.3. Log
Port php-mysql-xdevapi success on xcode8.3. Log
Port php71-mysql-xdevapi success on xcode8.3. Log
Port php72-mysql-xdevapi success on xcode8.3. Log
Port php73-mysql-xdevapi success on xcode7.3. Log
Port php-mysql-xdevapi success on xcode7.3. Log
Port php71-mysql-xdevapi success on xcode7.3. Log
Port php72-mysql-xdevapi success on xcode7.3. Log

@mf2k
Copy link
Contributor

mf2k commented Mar 19, 2019

This looks OK but I'm not committing until Ryan approves being maintainer. I'm OK with removing him as maintainer and committing however.

@pmetzger
Copy link
Member

@kirmorozov ?

@mf2k
Copy link
Contributor

mf2k commented Mar 29, 2019

It is @ryandesign that I am waiting for to approve/commit this.

@kirmorozov
Copy link
Contributor Author

Hi, I set nomaintainer as maintainer.

@macportsbot
Copy link

Travis Build #5949 Passed.

Lint results
--->  Verifying Portfile for php-mysql-xdevapi
--->  0 errors and 0 warnings found.

Port php73-mysql-xdevapi success on xcode10.2. Log
Port php-mysql-xdevapi success on xcode10.2. Log
Port php71-mysql-xdevapi success on xcode10.2. Log
Port php72-mysql-xdevapi success on xcode10.2. Log
Port php73-mysql-xdevapi success on xcode9.4. Log
Port php-mysql-xdevapi success on xcode9.4. Log
Port php71-mysql-xdevapi success on xcode9.4. Log
Port php72-mysql-xdevapi success on xcode9.4. Log
Port php73-mysql-xdevapi success on xcode8.3. Log
Port php-mysql-xdevapi success on xcode8.3. Log
Port php71-mysql-xdevapi success on xcode8.3. Log
Port php72-mysql-xdevapi success on xcode8.3. Log
Port php73-mysql-xdevapi success on xcode7.3. Log
Port php-mysql-xdevapi success on xcode7.3. Log
Port php71-mysql-xdevapi success on xcode7.3. Log
Port php72-mysql-xdevapi success on xcode7.3. Log

@mf2k
Copy link
Contributor

mf2k commented Apr 11, 2019

It looks like there are conflicts that need to be resolved now.

@macportsbot
Copy link

Travis Build #5964 Passed.

Lint results
--->  Verifying Portfile for php-mysql-xdevapi
--->  0 errors and 0 warnings found.

Port php73-mysql-xdevapi success on xcode10.2. Log
Port php-mysql-xdevapi success on xcode10.2. Log
Port php71-mysql-xdevapi success on xcode10.2. Log
Port php72-mysql-xdevapi success on xcode10.2. Log
Port php73-mysql-xdevapi success on xcode9.4. Log
Port php-mysql-xdevapi success on xcode9.4. Log
Port php71-mysql-xdevapi success on xcode9.4. Log
Port php72-mysql-xdevapi success on xcode9.4. Log
Port php73-mysql-xdevapi success on xcode8.3. Log
Port php-mysql-xdevapi success on xcode8.3. Log
Port php71-mysql-xdevapi success on xcode8.3. Log
Port php72-mysql-xdevapi success on xcode8.3. Log
Port php73-mysql-xdevapi success on xcode7.3. Log
Port php-mysql-xdevapi success on xcode7.3. Log
Port php71-mysql-xdevapi success on xcode7.3. Log
Port php72-mysql-xdevapi success on xcode7.3. Log

@pmetzger
Copy link
Member

@mf2k No more conflicts. Should we merge?

@kirmorozov
Copy link
Contributor Author

I'm using port locally anyway this one comes handy if someone wants to use Mysql as object storage.

@pmetzger
Copy link
Member

@mf2k ?

@mf2k mf2k merged commit 809abed into macports:master Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
5 participants