Bug 33520 - Hardcoded memory_limit value in the wrapper script
Summary: Hardcoded memory_limit value in the wrapper script
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: composer (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Vitaly Lipatov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks: 35203
  Show dependency tree
 
Reported: 2017-06-01 13:03 MSK by Nikolay A. Fetisov
Modified: 2018-08-18 13:52 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nikolay A. Fetisov 2017-06-01 13:03:55 MSK
Для composer-1.4.1-alt1 в обёртке /usr/bin/composer жестко задано
значение memory_limit=256M. 
Как минимум для установки текущего Laravel этого не достаточно.

Наверное, имеет смысл убрать memory_limit из обёртки и использовать
значение по-умолчанию из системного php.ini - это очевидное место для 
правки при получении ошибок о нехватке памяти.
Comment 1 zidex@altlinux.org 2018-02-09 09:40:21 MSK
Да, 256 маловато. Хорошо бы увеличить или брать значение из php.ini
Comment 2 Nikolay A. Fetisov 2018-08-17 19:26:29 MSK
По итогам рассмотрения, просто брать значение memory_limit из php.ini нехорошо. 
По-умолчанию оно 128 Mb, а composer'у нужно минимум 256 Mb
(отсюда в 1.1.3-alt1 оно и было задано).

Соответственно, в ушедшем в Sisyphus 1.7.2-alt1:
- можно задать значение memory_limit при конкретном вызове composer как 
$ PHP_MEMORY_LIMIT=1024M composer install

- можно задать PHP_MEMORY_LIMIT в /etc/sysconfig/composer

- если PHP_MEMORY_LIMIT не задана, _и_ в php.ini memory_limit не менее 256 Mb - 
  используется значение из php.ini

- ну и если в php.ini memory_limit меньше 256 Mb - composer запускается с 
  ограничением в 256 Mb.


Т.е., для #35203 - можно использовать первый вариант, с заданием нужного
PHP_MEMORY_LIMIT при запуске composer.
Comment 3 Repository Robot 2018-08-17 19:39:07 MSK
composer-1.7.2-alt1 -> sisyphus:

Fri Aug 17 2018 Nikolay A. Fetisov <naf@altlinux> 1.7.2-alt1
- new version
- configurable memory_limit PHP setting (Closes: 33520)
- remove PHP7 requirements - composer can be used with PHP5 too