pantheon-systems / terminus

The Pantheon CLI — a standalone utility for performing operations on the Pantheon Platform
https://pantheon.io
Other
316 stars 194 forks source link

Cannot upgrade using command line instructions #1752

Open ghost opened 7 years ago

ghost commented 7 years ago

Expected behavior

I expected to run the command provided in the terminal to upgrade.

Actual behavior

It doesn't work. I receive the following message and no upgrade occurs:

Updating Terminus...
<warning>You are running composer with xdebug enabled. This has a major impact on runtime performance. See https://getcomposer.org/xdebug</warning>
Composer could not find a composer.json file
To initialize a project, please create a composer.json file as described in the https://getcomposer.org/ "Getting Started" section

Steps to reproduce the behavior

Run the upgrade command provided by the CLI:

curl -O https://raw.githubusercontent.com/pantheon-systems/terminus-installer/master/builds/installer.phar && php installer.phar update
Environment info
 ------------------------- ---------------------------------------------------------- 
  PHP binary                /usr/local/Cellar/php70/7.0.20_12/bin/php                 
  PHP version               7.0.20                                                    
  php.ini used              /usr/local/etc/php/7.0/php.ini                            
  Terminus project config                                                             
  Terminus root dir         /Users/jhabdas/terminus/vendor/pantheon-systems/terminus  
  Terminus version          1.4.0                                                     
  Operating system          Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT     
                            2017; root:xnu-3789.70.16~2/RELEASE_X86_64                
 ------------------------- ---------------------------------------------------------- 
ghost commented 7 years ago

The ideal experience here is something like WP-CLI where you issue wp cli update and it Just Works™

greg-1-anderson commented 7 years ago

Composer changed behavior recently. The composer require command in the installer must now be changed to composer require -n to maintain the historic behavior.