Differential D768

Install phan and php-ast
ClosedPublic
Actions

Authored by dereckson on Dec 26 2016, 02:41.

Details

Reviewers

dereckson

Maniphest Tasks

T1063: Implement phan analysis support in CI
T1075: Install php-ast (and phan?) on CI PHP image

Commits

rDJENKINSAGENTS503a2cb8a7b9: Install phan and php-ast

Summary

As rNOTIF wants to use phan, we need to install the php-ast extension.

The goal is to allow static analyze tools to access the PHP 7 parser.

Fixes T1075.

Test Plan

Build, run phan

Commands also tested on silly_bardeen, a container on our CI to run
such tests for rNOTIF

Diff Detail

Repository

rDJENKINSAGENTS Docker images for Jenkins agents

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fix typo

Fix syntax error

dereckson accepted this revision.Dec 29 2016, 20:46

dereckson edited edge metadata.

This revision is now accepted and ready to land.Dec 29 2016, 20:46

dereckson closed this revision.Dec 29 2016, 20:48

Closed by commit rDJENKINSAGENTS503a2cb8a7b9: Install phan and php-ast (authored by dereckson, committed by dereckson). · Explain WhyDec 29 2016, 20:48

This revision was automatically updated to reflect the committed changes.

dereckson added a task: T1063: Implement phan analysis support in CI.Dec 30 2016, 03:11

Revision Contents
Changeset List

Path

Size

jenkins-slave-php/

Dockerfile

6 lines

files/

usr/

local/

etc/

php/

conf.d/

ast.ini

1 line

Diff 1967

View Options

jenkins-slave-php/Dockerfile