[pkg-php-pear] Symfony^wSafety net (was: symfony copyright review)

Daniel Beyer dabe at deb.ymc.ch
Sun Sep 21 13:34:34 UTC 2014


Hi David,

On Sat, 2014-09-20 at 17:28 -0400, David Prévot wrote:
> Hi Daniel,
> 
> 
> > On Sat, 2014-09-20 at 11:43 -0400, David Prévot wrote:
> 
> >> I may update some php-symfony-* packages in the mean time in case we’re
> >> not able to upload this one in time to be processed by ftp-masters until
> >> the freeze
> 
> I’ve prepared php-symfony-classloader update on Git, and intend to
> prepare the six other php-symfony-* before uploading them all (letting
> some time to make changes or provide any other feedback in the mean
> time, or even better: upload the big symfony package and forget about
> these tiny packages for ever ;).
> 

I vote for uploading the big package :). But having those six other
packages prepared (or even uploaded) is a good idea.


> I took the time to look at the improved long description you provided,
> and added a few bits from the README [1], you may wish to update the
> php-symfony-class-loader description if you believe it’s an improvement.
> 

Thanks, I took over your updates. It had a double 'automatically' in it.
I updated php-symfonyclassloader.git to have it only once in the
description there, too - hope you don't mind.


> About the short description (synopsis), I went with the one provided in
> composer.json (Symfony ClassLoader Component), more in line with the
> current one (Symfony framework - Class Loader component) but maybe less
> interesting than yours (load PHP classes automatically). In case those
> descriptions actually are consistent among the various composer.json
> spread in the big symfony package, it may still be considered (I’m less
> sure about that, just sharing thoughts).
> 

I don't like the descriptions found in the composer.json files much.
They either add not much additional information over the package name or
provide too long text (like symfony/intl does). I would prefer to stay
with the existing hand-crafted ones currently found in d/control.


> I still find the current homepage [2] more interesting than the one you
> used [3], but understand it may be trickier to maintain for all components.
> 

I agree, but not all components provide such a page. Additionally I
worry about upstream changing or moving the [2]-style deep-links (they
already were broken in past). Anyhow I changed the links where possible,
but used the 2.3 version in the hope the 2.3 series documentation will
not be restructured by upstream and the links keep valid over the next
years.


> 1:
> http://anonscm.debian.org/cgit/pkg-php/php-symfony-classloader.git/tree/debian/control#n19
> 2: http://symfony.com/doc/current/components/class_loader/introduction.html
> 3: http://symfony.com/components/ClassLoader
> 
> Regards
> 
> David
> 
> P.-S. I’m slightly tempted to start fresh on a new thread when the
> subject changes, because it becomes more and more difficult to track
> this >50 message thread and its various sub-threads…

Having my own troubles tracking that thread, I really like this
approach.

Greetings
Daniel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.alioth.debian.org/pipermail/pkg-php-pear/attachments/20140921/4968749e/attachment.sig>


More information about the pkg-php-pear mailing list