rocketeer icon indicating copy to clipboard operation
rocketeer copied to clipboard

Is Composer required?

Open JanisE opened this issue 8 years ago • 6 comments

Is composer required for Rocketeer to operate? I don't use Composer in the particular project, still Rocketeer complains on both "check" and "deploy" commands that "The Composer package manager could not be found".

JanisE avatar Oct 19 '15 11:10 JanisE

Well, I guess it's not really required, but it is indeed part of the checks Rocketeer does for PHP applications as most apps nowadays use Composer.

Anahkiasen avatar Oct 19 '15 12:10 Anahkiasen

You should be able to disable the check by changing the Check strategy to null in strategies.php

Anahkiasen avatar Oct 19 '15 12:10 Anahkiasen

Which checks would it cancel? I would rather only Composer checks get disabled, not PHP as such (or what it involves). I've commented out the default Composer-related lines in "strategies.php" (install and update hooks), but Rocketeer still checks for Composer.

JanisE avatar Oct 19 '15 12:10 JanisE

There is a similar case with mcrypt: "The following extensions could not be found: mcrypt." As it is a warning and the deployment seems to happen all right, I suppose, it's not critical. Still, every time I deploy, the red error message gives an impression that something is not working correctly. I could get used to it and don't worry, but then it would lose its meaning.

P.S. Oh, and if deploying a stage for the first time, the errors appear even if the "Check" strategy is commented out.

JanisE avatar Oct 19 '15 15:10 JanisE

@JanisE I don't understand why mcrypt is required by default either.

Required extensions can be defined in composer.json.

IMHO the PHP check should look for ext- requirements in composer.json (if available), and check whether they are installed, rather than just mcrypt.

P.S. @Anahkiasen are you still maintaining Rocketeer? (I completely understand if you've got "open-source fatigue".)

gwagroves avatar Apr 23 '16 05:04 gwagroves

Pull request with the required changes: #662

Drop-in replacement that can be used with the current release.

gwagroves avatar Apr 23 '16 06:04 gwagroves