Skip to content

feat: add pre-install script execution #19

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 7, 2021

Conversation

boesing
Copy link
Member

@boesing boesing commented Apr 7, 2021

Q A
New Feature yes

Description

In laminas-cache, we have use-cases where we either want to manually compile PHP extensions with pecl to suit composer requirements or extending the ENV for composer to avoid circular dependency issues.

The current pre-run.sh is being executed after composer install is being executed and thus, it will fail before we reach the script.
In order to solve that issue, I'd like to introduce a pre-install.sh script, which is being executed in an earlier stage of the process.

@boesing boesing added this to the 1.5.0 milestone Apr 7, 2021
@boesing boesing requested a review from weierophinney April 7, 2021 13:59
@boesing boesing force-pushed the feature/pre-install-script branch 2 times, most recently from adbd233 to de75b9b Compare April 7, 2021 14:47
@boesing boesing force-pushed the feature/pre-install-script branch from de75b9b to 40f2f2d Compare April 7, 2021 15:14
@boesing boesing requested a review from weierophinney April 7, 2021 15:14
Signed-off-by: Maximilian Bösing <[email protected]>
@boesing boesing force-pushed the feature/pre-install-script branch from 40f2f2d to d57c5cc Compare April 7, 2021 15:27
@boesing boesing requested a review from weierophinney April 7, 2021 15:27
@weierophinney weierophinney merged commit 034a005 into laminas:1.5.x Apr 7, 2021
@boesing boesing deleted the feature/pre-install-script branch April 7, 2021 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants