Php-imap: запрос re: поддержка EOL-версий PHP

@barbushin изначально хотел сохранить поддержку этих версий PHP.

На мой взгляд, мы можем удалить поддержку всех неподдерживаемых версий PHP: https://www.php.net/supported-versions.php

Поэтому я бы поддерживал только активные версии PHP, такие как PHP 7.2 и новее / выше.

Нам также нужно каким-то образом заставить пользователей обновлять свои старые вещи. Когда они всегда видят, что программное обеспечение все еще поддерживается более старой версией, они никогда не будут обновлять свою систему, поскольку она работает.

Некоторые улучшения кодирования этой библиотеки возможны только тогда, когда мы удаляем поддержку более старых версий, поэтому нам следует подумать об удалении поддержки версий EoL PHP.

Любые отзывы / предложения? @ bapcltd-marv @barbushin

@ Sebi94nbg re: улучшения кода, см .:

• падение 5.6 EOL
• падение 7,0 после сброса 5,6 EOL
• падение 7.1 после падения <= 7.0 идет EOL в ноябре
• падение 7,2 после падения <= 7,1 переходит в EOL в следующем году
• снижение 7,3 после падения <= 7,2 без выхода на EOL еще 2 с небольшим года

Если вы хотите сразу перейти к php:^7.2 , вы получите строго типизированные параметры, строго типизированные возвращаемые значения, подсказки типа объекта и возвращаемые значения void.

Наша нисходящая ветвь переходит прямо к php:^7.4 исключительно из-за того, что нет необходимости поддерживать более старые версии PHP.

Из зависимых пакетов, перечисленных в packagist,

• 19 не имеют перечисленных ограничений php
• 5 требуется php:>=5.3 , php:>=5.3.2 или php:>=5.3.3
• 3 требуется php:>=5.4.0 или php:>=5.4.5
• 1 требует php:~5.4|~7.0
• 1 требует php:>=5.5.0
• 1 требует php:^5.5|~7.0
• 2 требуется php:>=5.6 или php:>=5.6.0
• 1 требует php:^7 или php:>=7.0
• 1 требует php:^7.1
• 2 требуется php:^7.2 или php:>=7.2
• 1 требует php:^7.3
• 1 требует php:^7.4

Вы можете рассмотреть возможность поддержки пары веток ^5.6|<7.2 и ^7.2 , но это создаст дополнительные накладные расходы, которые могут не стоить дополнительных усилий?

ps преимущества перехода на 7.4 - это типизированные свойства и возможность разумного включения файла composer.lock в репо.

Хорошо, давайте удалим больше не поддерживаемые версии PHP.

Было бы здорово сразу перейти к коду, поддерживаемому PHP 7.4, но я хочу сохранить поддержку PHP 7.2 и 7.3.

Я предполагаю, что вы захотите выпустить новую версию 3.x, прежде чем я буду предлагать соответствующую ветку для версии 4.x?

Судя по звукам, вам понадобится ветка "drop / php-7.1" (еще не PRing из-за повторяющейся привычки перебазировать ее / настроить набор патчей).

re: "принуждение" пользователей к обновлению, я не уверен, что у меня есть какой-либо соответствующий опыт, чтобы надлежащим образом комментировать, помимо политик поддержки на 3.x / 4.x.

Обновление ps по 19 частным ограничениям php:

• 13 требуется ~ 2.0, или 2.0.9, или ^ 2.0, ( php:>=5.3 )
• 
  
  • 6 из них были для более старой ветки разработки / выпуска (похоже, это ветки одной и той же системы)
  
  
• 
  
  • 1 из которых имеет зависимость php:>=5.5.9
  
  
• 
  
  • 1 из которых был для более старой ветки разработчика с ограничением php:>=5.3.3,<7.0
  
  
• 5 требуется ^ 3.0, ( php:>=5.6 )
• 
  
  • 1 из которых имеет зависимость php:>=5.6
  
  
• 
  
  • 1 из которых имеет зависимость php:^5.6.4 || ^7.0
  
  
• 
  
  • 1 из которых имеет зависимость php:>=7.0.0
  
  
• 
  
  • 1 из которых имеет зависимость php:>=7.2
  
  
• 
  
  • 1 из которых использует его как зависимость разработчика.
  
  
• 1 требует @stable
• 1 требуется dev-master

(возможно, неправильно подсчитал одно из значений 2.x / 3.x)

re: "принуждение" пользователей к обновлению, я не уверен, что у меня есть какой-либо соответствующий опыт, чтобы надлежащим образом комментировать, помимо политик поддержки на 3.x / 4.x.

дальнейшие мысли:

• [] php:^5.6 (полностью исключает PHP 7)
• [x] php:^5.6|^7 (кроме PHP 8)
• [] php:>=5.6,<7.2 (кроме PHP 7.2+)

Ммм, я бы сейчас исключил PHP 8 в этом случае.

Исключение PHP 7.2+ - плохая идея, как и все версии PHP 7.

Исключение PHP 7.2+ - плохая идея, как и все версии PHP 7.

идея, лежащая в основе php:>=5.6,<7.2 для 3.x и php:^7.2 для 4.x, будет заставлять людей обновляться - отрицательное влияние было бы, если бы 3.x был зависимостью одной зависимости, а 4.x был зависимость от другой зависимости, вы столкнетесь с проблемой.

вы столкнетесь с проблемой.

практический пример: symfony/console - обычно инструментальные средства разработчика используют symfony/console для обработки задач CLI, но некоторые новые выпуски инструментов также не используют более новые версии пакета консоли, например:
1) последние помеченные выпуски maglnet/composer-require-checker & vimeo/psalm оба поддерживают symfony/console 5.x
2) последний выпуск povils/phpmnd тегами поддерживает не более symfony/console 4.x

что приводит к более низкому ограничению, применяемому к maglnet/composer-require-checker и иногда к необходимости вручную ограничивать symfony/console (требуя, а затем удаляя его, если оно не используется напрямую).

tl: dr; понижение верхней границы php, поддерживаемого для 3.x, может вынудить пользователей к обновлению и может привести к разочарованию пользователя.

Всегда сложно не расстраивать пользователей. :(

@ bapcltd-marv каковы теперь планы? Когда мы сможем опубликовать все ваши улучшения в какой версии?

текущий план:
1) * [x] ждем новый (минорный) релиз barbushin / php-imap
2) * [x] [oauth удаляется из ветки разработки] (https://github.com/barbushin/php-imap/issues/486#issuecomment-595153139)
3) * [x] Я перебазирую # 475 и выношу его из черновика
4) * [x] # 475 объединяется с barbushin / php-imap и новым (основным) выпуском, в котором отсутствует поддержка php 5.6, 7.0 и 7.1.
5) * [] в какой-то момент в будущем подтолкните меня к PR в наборе изменений для отказа от поддержки php 7.2 и 7.3
6) * [] в какой-то момент в далеком будущем, подтолкните меня на случай, если я полностью отказался от php 7.

ps см. также необязательное руководство по добавлению некоторых частных тестов в репо в той или иной форме.

@ bapcltd-marv все готово в ветке develop ? Если да, я бы сейчас создал второстепенный выпуск 3.1.0 , чтобы мы могли продолжить ваши другие упомянутые моменты. :)

@ Sebi94nbg , мы что-то делаем для интеграции ваших частных тестов в репо?

Большинство этих тестов уже в репо. Я просто перепроверил их раньше все время, так как никогда не был уверен, что тесты репо работают должным образом.

Итак, я бы теперь слил develop с master и выпустил бы 3.1.0 тогда, когда вы согласитесь.

@ bapcltd-marv пропустил ли я что-нибудь из того, что изменилось? # 491

выглядит неплохо :)

Выпущена 3.1.0 ! :)

Если вы можете выполнить вилку и удалить oauth из development , я могу перейти к перебазированию # 475, как только закончу поливать офисные растения.

Конечно, я сделаю это примерно через час. Я сначала принесу мороженое. :)

К сожалению, у меня возникли проблемы с извлечением / отправкой из / в Git из моей локальной системы. Мне нужно исследовать эту проблему дальше. : '(

какие сообщения вы получаете?

Извините за задержку, я все еще переезжаю в новую квартиру, которая продлится до конца мая из-за COVID-19.

Теперь я мог частично исправить проблему с помощью Git / VSCode.

Он всегда сообщал, что ключ SSH не может быть загружен или использован для аутентификации или что ключ хоста не может быть проверен. На данный момент я могу запустить git pull и git push из интерфейса командной строки VSCode, но функция пользовательского интерфейса для синхронизации всего одним щелчком мыши по-прежнему не работает - она ​​сообщает об ошибке Git: Host key verification failed. .

Я удалил код OAuth из ветки разработки ( commit f85a74 ) и создал новую ветку на основе ветки разработки, которая включает код OAuth: https://github.com/barbushin/php-imap/compare/develop ... 411-Поддержка OAuth

~ в настоящее время ожидает, что Трэвис завершит перебазирование. ~ совершил ошибку при запуске команд обновления в окне php-7.4 ¬_¬

Хорошо, я объединил ваш PR, отказавшись от старой поддержки PHP. А также реализовал изменение кода из другого PR в отношении этого декодирования Base 64.

Готовы ли мы выпустить 3.2.0 ? Все обновлено? README, composer.json, ...?

Было бы хорошо, если бы у нас была удачная сборка: https://travis-ci.org/github/barbushin/php-imap/jobs/676976779 : D

удаление версий php должно быть основным выпуском, а не второстепенным выпуском, например 4.0.0

Да, ты прав. Должен быть основным выпуском, поскольку он отказывается от старой поддержки PHP.

Есть идеи, как мы можем исправить эту проблему с Travis CI? Есть ли что-нибудь еще, кроме исправления для Travis CI?

Трэвис был закрыт на техническое обслуживание ранее, не мог видеть ошибку; ~ Теперь взглянем. ~ Ах да, просто введите тип как в других случаях.

Я добавил это приведение, спасибо.

Я добавил в README таблицу поддерживаемых версий PHP: https://github.com/barbushin/php-imap/blob/develop/README.md
Вы бы изменили это или все в порядке?

develop build теперь в порядке: https://travis-ci.org/github/barbushin/php-imap/builds/677385039 :)

Если ничего не пропустил, опубликую этот PR https://github.com/barbushin/php-imap/pull/493 как 4.0.0 .

ридми не совсем точен re: ext-iconv; и он, и ext-mbstring в настоящее время требуются в composer.json - без них установка не удастся.

Обновлено! Что-нибудь еще, что я пропустил?

• «FR # 1:» не кажется правильным?
• phpcpd и phpmnd не упоминаются
• В настоящее время требуется ext-fileinfo, но он предназначен специально для кода, связанного с MIME, поэтому его можно реорганизовать и выделить в отдельный проект 🤷‍♂️
• все тесты можно запустить, запустив composer run tests , а не просто тесты phpunit
• 
  
  • может потребоваться изменить composer.json, чтобы включить флаг testdox, упомянутый в файле readme?
  
  
• Были ли упомянуты тесты живого почтового ящика Travis в версии 3.x?

Интересно. Понятия не имею, откуда у меня эта FR1-штука. Больше не могу найти. : o

Были ли упомянуты тесты живого почтового ящика Travis в версии 3.x?

Нет, не упоминалось. См. Https://github.com/barbushin/php-imap/releases.
Я добавил это как информацию в журнал изменений.

Я обновил README:

• Отсортировал необходимые расширения и убедился, что все необходимые указаны
• Обновлен раздел «Выполнить тесты».

Когда сборка не завершится ошибкой, я выпущу новую версию 4.0.0 . :)

Выпущена v4.0.0 . :)