theseer / Autoload

A lightweight php namespace aware autoload generator and phar archive builder
Other
388 stars 47 forks source link

Missing dependency on PHP/Timer #33

Closed vaceletm closed 12 years ago

vaceletm commented 12 years ago

On fresh install, the default instructions set in README doesn't leave a working phpab.

Autoload phpab depends on PHP/Timer but doesn't install it with pear.

theseer commented 12 years ago

Are you sure this is the correct project? The Autoload builder does not make any use of PHP_Timer

vaceletm commented 12 years ago

I'm pretty sure yes. If you look at phpab launcher at L49 there is a require 'PHP/Timer.php'; https://github.com/theseer/Autoload/blob/f39fa5b1e0c14e6f4fc573d2a4359413645c16ee/phpab.php

theseer commented 12 years ago

That was probably the most useless require line ever. Nice catch. Fixed with commit 38c092a and is, among new trusting default mode, part of the Autoload 1.9.1 release.

vaceletm commented 12 years ago

You're welcome, happy to help. You should have used something to generate your autoload automatically :D