Re: PhD using Composer - proof of concept and a few questions
| From: | Peter Cowburn | Date: | Mon, 12 Dec 2016 18:49:47 +0000 |
| Subject: | Re: PhD using Composer - proof of concept and a few questions | ||
| References: | 1 2 | Groups: | php.doc |
| Request: | Send a blank email to phpdoc+get-969386455@lists.php.net to get a copy of this message | ||
On 12 December 2016 at 08:15, Maciej Sobaczewski <sobak@php.net> wrote:
> Hi guys!
>
> Little status update from me:
> - I brought package-pman.php back
> - fixed phd --list
> - brought GeSHi support back to life (http://sobak.pl/img/screensho
> ts/Cp2aShsZ.png)
> - I tested everything on PHP 5.3.2 (as it seems to be the lowest version,
> we have it on euk2); Composer complains that this version is vunerable
> to CVE but it works and its able to install phd globally without any
> issues; phd also works and renders its own guide flawlessly
>
> I think it's ready to be merged. But before that, docs.php.net and
> edit.php.net need to be reconfigured to run "composer global require
> php/phd dev-master" and then "composer global update php/phd" instead
> of "git pull".
>
> How should I proceed from now? I can make pull request at any time but
> I'm not capable of changing the configuration. I'm not even entirerly
> sure that I'm right about the changes needed.
>
> To summup:
> 1) create Pull Request
> 2) merge the changes and submit to Packagist
3) change docs.php.net and edit.php.net to use composer instead of git,
> before they break due to the lack of an autoloader
> 4) continue development of dev version (as well as PhD's guide updates
> etc, I will handle it)
> 5) before the stable release, prepare rsync box to use composer instead
> of PEAR to install PhD
> 6) make a stable release
>
> Does it makes sense to you?
>
Just an update on the Packagist story. In short, step 2) above can be
marked as done.
Jordi Boggiano (from composer/packagist) has helped us out by talking to
the maintainers of the packages that were using the "php" vendor name with
a view to freeing up the name just for us. In fact, a bunch of the packages
have already been moved.
Also, we now have PhD on Packagist [1]. I added a composer.json to PhD so
that we could add the package to Packagist, ahead of merging Maciej's
branch when he thinks it is ready. Note, I'm currently the only maintainer
of the package right now so we should definitely add more people to the
list (for the "bus factor") and write down somewhere (on docs.php.net?) who
has access there.
[1] https://packagist.org/packages/php/phd
>
> Thanks,
> Maciej.
>