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

No Lint Coverage

Unit

No Test Coverage

Branch

ast-and-phan (branched from master)

Build Status

Buildable 1174
Build 1397: arc lint + arc unit

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 1941

View Options

jenkins-slave-php/Dockerfile