#36 $output not found in install command

Closed
opened 4 years ago by herrhase · 0 comments
herrhase commented 4 years ago
Owner
 app/Commands/NodejsInstallCommand.php:50
    46|         exec('curl -sL https://deb.nodesource.com/setup_'.$version.'.x | sudo -E bash -');
    47|         exec('apt-get install -y nodejs 2>&1');
    48| 
    49|         // @TODO apt add a Warning for no good, in a later version output will be scanned for helpfull infos
  > 50|         $this->line(implode("\n", Install::filterAptMessages($output)));
    51| 
    52|         $this->info('Nodejs add user...');
    53|         exec('useradd -d /home/nodejs -m nodejs', $output);
    54| 
``` app/Commands/NodejsInstallCommand.php:50 46| exec('curl -sL https://deb.nodesource.com/setup_'.$version.'.x | sudo -E bash -'); 47| exec('apt-get install -y nodejs 2>&1'); 48| 49| // @TODO apt add a Warning for no good, in a later version output will be scanned for helpfull infos > 50| $this->line(implode("\n", Install::filterAptMessages($output))); 51| 52| $this->info('Nodejs add user...'); 53| exec('useradd -d /home/nodejs -m nodejs', $output); 54| ```
herrhase added this to the 0.2 milestone 4 years ago
herrhase added the
bug
label 4 years ago
herrhase self-assigned this 4 years ago
herrhase added the
package / nodejs
label 4 years ago
herrhase referenced this issue from a commit 4 years ago
herrhase closed this issue 4 years ago
Sign in to join this conversation.
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.