Page MenuHomePhabricator

Move assert-phpflavor macro into a bash script in integration/jenkins
Closed, DeclinedPublic

Description

The assert-phpflavor macro should really be a bash script...shouldn't need to refresh a few hundred jobs everytime it changes :/

Event Timeline

Legoktm raised the priority of this task from to Needs Triage.
Legoktm updated the task description. (Show Details)
Legoktm subscribed.

Change 296060 had a related patch set uploaded (by Paladox):
Add assert-phpfalvour.sh shell script

https://gerrit.wikimedia.org/r/296060

Change 296061 had a related patch set uploaded (by Paladox):
Move script out of assert-phpflavor macro

https://gerrit.wikimedia.org/r/296061

The change above has been uploaded over a year ago and there have been 0 comments on it. bump!

Change 296060 abandoned by Dzahn:
Add assert-phpflavor.sh shell script

https://gerrit.wikimedia.org/r/296060

Change 296061 abandoned by Paladox:
Move script out of assert-phpflavor macro

https://gerrit.wikimedia.org/r/296061

hashar subscribed.

With Docker container the logic is no more needed. The containers come with a single PHP version.

The last jobs relying on the macro are going to be phased out eventually: