Integrate php lint syntax checks in your build process

Tuesday, March 18. 2008

Integrate php lint syntax checks in your build process

Here is a small code snippet that can be used to integrate php's internal lint syntax check into your build process. The ant task apply behaves like the exec task, but it accepts a nested fileset element. This allows to process a set of input files thru a single command.

<?xml version="1.0" encoding="UTF-8"?>
<project name="phpUnderControl" basedir="." default="build">
...
<target name="checkphp">
<apply executable="php" failonerror="true">
<arg value="-l" />
<fileset dir="source/src">
<include name="**/*.php" />
</fileset>
</apply>
</target>

<target name="build" depends="checkphp,php-codesniffer,phpdoc,phpunit" />
...
</project>

This can also be used as an addition in your phpunit target or any other command which executes code during the build-process, when the command would quit due to a syntax error.

Posted by Manuel Pichler in php, phpugdo, phpundercontrol, projects at 21:05 | Comments (4) | Trackbacks (0)

Defined tags for this entry: ant, continuous integration, cruisecontrol, lint, php, phpundercontrol

Related entries by tags:

Trackbacks

Trackback specific URI for this entry

No Trackbacks

Comments

Display comments as (Linear | Threaded)

You could also use the available Phing task called PhpLintTask to validate the syntax in a single file or fileset.

#1 Raphael Stolt (Homepage) on 2008-03-19 08:16 (Reply)

The problem with the phing task is, that it just throws a warning and not erorrs out. More important however, linting a huge project is not really fast. One option to make it faster would mean patching the cli SAPI to allow more than one file passed to -l.

#1.1 Lars Strojny (Homepage) on 2008-03-19 12:21 (Reply)

While this is obviously many months after post, I did want to clear up this misinformation. The Phing phplint task does have a haltonfailure option which would cause the build process to error out. (Perhaps this changed since the comment, I'm not sure.)

#1.1.1 Hans on 2008-11-07 22:44 (Reply)

Hi Raphael,

thanks for that hint, but I use ant as build tool and not phing

But for the phpUnderControl users who use phing as build tool, it is good to know that such a task already exists.

#2 Manuel Pichler (Homepage) on 2008-03-19 10:11 (Reply)

Add Comment

Name
Email
Homepage
In reply to
Comment	HTML-Tags will be converted to Entities. Enclosing asterisks marks text as bold (word), underscore are made via _word_. Standard emoticons like :-) and ;-) are converted to images. E-Mail addresses will not be displayed and will only be used for E-Mail notifications. To prevent automated Bots from commentspamming, please enter the string you see in the image below in the appropriate input box. Your comment will only be submitted if the strings match. Please ensure that your browser supports and accepts cookies, or your comment cannot be verified correctly. Enter the string from the spam-prevention image above:
	Remember Information? Subscribe to this entry
Submitted comments will be subject to moderation before being displayed.

Integrate php lint syntax checks in your build process

Manuel Pichler

Tuesday, March 18. 2008

Integrate php lint syntax checks in your build process

Projects

Further stuff

See also

Categories