RE: [PEAR-DEV] PHPUnit News
| From: | Lukas Smith | Date: | Sat, 28 Sep 2002 09:44:42 +0000 |
| Subject: | RE: [PEAR-DEV] PHPUnit News | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-9631@lists.php.net to get a copy of this message | ||
> From: Stig S. Bakken [mailto:ssb@fast.no]
> Sent: Saturday, September 28, 2002 11:42 AM
> > Sebastian Bergmann wrote:
> >
> > > Hi.
> > >
> > > I created a branch "ZEND_ENGINE_2" in the PHPUnit/ directory.
> > >
> > > This branch requires, as you already guessed from its name, the
Zend
> > > Engine 2. It will eventually become PHPUnit 1.0, once PHP 5.0.0
is
> out
> > > of the door.
> > >
> > > Current changes include the use of namespaces, proper pass-by-
> reference
> > > and the use of private members.
> > >
> > > Next thing on my TODO is proper exception handling for the
> > > PHPUnit::TestFailure class.
> > >
> > >
> > >
> > Very cool,
> > What about making a ZEND_ENGINE_2 branch for the whole pear cvs
module?
> > (and then each maintainer will convert his/her package whenever
he/she
> > likes)
>
> In general, I think it is better to try making the code ZE1/2
> compatible, and in the cases where this won't work, put just the
> ZE2-specific code in separate files, and avoid using branches. Having
> "permanent branches" such as this becomes a really big maintenance
> headache after some time.
>
also it makes things more confusing from the user perspective.
Allthough for the long term clean seperation it might be better to go
with branches I think it poses to many disadvantages for the
short-mid-term.
Not using branches will ensure that more work is done to make code ZE1
and ZE2 compliant. Will make it easier for CVS novices.
BTW: Stig added (will add?) the possibility to specify the required ZE
in the package.xml
Regards,
Lukas