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

Add support of php 7.2 #72

Closed
gfh31fgh2 opened this issue Jun 13, 2023 · 3 comments · Fixed by #76
Closed

Add support of php 7.2 #72

gfh31fgh2 opened this issue Jun 13, 2023 · 3 comments · Fixed by #76
Labels
bug Something isn't working

Comments

@gfh31fgh2
Copy link

gfh31fgh2 commented Jun 13, 2023

Bug Report

YDB PHP SDK version:
latest (1.5.5)

Environment
ubuntu18 and JWT

Current behavior:

В текущем виде ydb-php-sdk не работает с php7.2 (стоит ограничение php>7.3)
Уберите пожалуйста такие ограничения, последний PR вполне годится и для старых версий.
У многих еще в ходу версия 7.2. Нет пока острой необходимости так резко повышать версию php.
У меня например старая БД работает через библиотеки php7.2, когда выбиралась ydb-php-sdk учитывалось это, тк для миграции нужно чтобы параллельно работало одновременно две БД на одних и тех же версиях.

Steps to reproduce:
Запустить установку ydb-sdk из php 7.2

@gfh31fgh2 gfh31fgh2 added the bug Something isn't working label Jun 13, 2023
@rekby rekby changed the title 1.5.5 not working with php7.2 Add support of php 7.2 Jun 14, 2023
@rekby
Copy link
Member

rekby commented Jun 14, 2023

@gfh31fgh2, подскажите какой из вариантов правильный:

  1. сначала 12 часов скрипты на 7.2 работают, а потом - проявляется какая-то ошибка несовместимости с 7.2.
  2. composer отказывается ставить зависимость, т.к. у нас указано минимальное требование 7.2, а вы используете php 7.3?
  3. Другой вариант (опишите пожалуйста)

@gfh31fgh2
Copy link
Author

gfh31fgh2 commented Jun 14, 2023

2 вариант.
composer отказывается ставить зависимость, т.к. у вас указано минимальное требование 7.3, а я использую php 7.2

При этом я вручную накатил изменения 1.5.4 -> 1.5.5 - и все работает с версией 7.2
Но правильно, чтобы изменения прилетали в папку vendor, от composer.

This was referenced Jun 14, 2023
@gfh31fgh2
Copy link
Author

все работает, спасибо!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants