Re: Packaging guidelines update

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Le 23/06/2015 16:16, Michael Cronenworth a écrit :
> Hello,
> 
> During a package review[1] it has become apparent that the PHP packaging
> guidelines[2] need to be updated. Remi, it should not be my job to
> request or do the work that you and your SIG should be doing.
> 
> Please discuss the following changes and any others that have now become
> required for new PHP packages. If I must I will create a ticket with FPC.
> 
> 1. State that phpcompatinfo needs to be run.

I don't see the need for this. Nothing "mandatory".
This is only a packaging tips.

> 2. Clearly define that virtual provides, even of core PHP modules, must
> be Requires. The current text looks like only non-core modules are
> Requires.

Guidelines explain "all modules", and php-date is given as an example.

> 3. Add a section for tests. You state they are a "MUST" but the
> guidelines do not mention tests at all.

Test are a SHOULD. Standard Guidelines.

In some case this is just not possible (I see awfull example requiring
running "various" vagrant images, requiring root privileges, and not
available dependencies)

But yes, for me, this is a MUST (when possible)

> 4. Extend the composer section to require an autoloader and
> composer.json file.

Again, just a packaging tips. Nothing mandatory.
Just a helper.


Remi.

> 
> Thank you,
> Michael
> 
> 
> [1] https://bugzilla.redhat.com/show_bug.cgi?id=1231318
> [2] https://fedoraproject.org/wiki/Packaging:PHP
> 
> _______________________________________________
> php-devel mailing list
> php-devel@xxxxxxxxxxxxxxxxxxxxxxx
> https://admin.fedoraproject.org/mailman/listinfo/php-devel

_______________________________________________
php-devel mailing list
php-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/php-devel




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora KDE]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux