Bug 33520

Summary: Hardcoded memory_limit value in the wrapper script
Product: Sisyphus Reporter: Nikolay A. Fetisov <naf>
Component: composerAssignee: Vitaly Lipatov <lav>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: danil, lav, zidex
Version: unstable   
Hardware: all   
OS: Linux   
Bug Depends on:    
Bug Blocks: 35203    

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