New packaging guidelines for PHP7 pacakges

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

New packaging guidelines for PHP7 pacakges

Johannes Weberhofer-2
Dear all!

With request https://build.opensuse.org/request/show/519119 I propose to enhance and simplify the PHP7 packaging process. I have written the following comment to the request which can be commented there. You can also respond to that E-Mail. Please let me know what you think about it.

As there are many PHP7 packages to be prepared and several requests are already waiting we should process quickly. I think we should not simply rename the old packages as most of them are completely outdated versions and packaged very complicated!

***
This request improves compatibility to Fedora and simplifies packaging a lot.

What is contained in the request? * php7-macros and php5-macros contains new packaging macros which makes packaging more simple, Have a look at the macros.php.

     I will check that those macros will be included in php5 and php7 packages in the future, but until that's done the macros can must be included in the spec files. Currently "BuildRequires: %{php_name}-macros" is required, but this can be moved to the "%if 0%{?suse_version} < 1320" as soon as PHP7 has been updated in factory

     Current openSUSE Versions require "BuildRequires: %{php_name}-pear-Archive_Tar" to be included. I have already submitted a fix to factory so new versions will no longer need that

     I highly vote for removing all those lower-case Provides/Obsoletes; let's just use the original package's name!

     For the future we should create virtual provides using the upstream names: Provides: php-pear(Auth_SASL) = %{version}

Related to this, the old php-macros project can be deleted. Therefor I have created https://build.opensuse.org/request/show/519123

I have attached an working example in: php7-pear-Auth_SASL

If you have no objections, I'd accept the changes to server:php:applications and prepare a request for future inclusion of the macros into factory and update the wiki pages.

***

--
Johannes Weberhofer
Weberhofer GmbH, Austria, Vienna
--
To unsubscribe, e-mail: [hidden email]
To contact the owner, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: New packaging guidelines for PHP7 pacakges

Bruno Friedmann-2
On mercredi, 30 août 2017 15.22:07 h CEST Johannes Weberhofer wrote:

> Dear all!
>
> With request https://build.opensuse.org/request/show/519119 I propose to
> enhance and simplify the PHP7 packaging process. I have written the
> following comment to the request which can be commented there. You can also
> respond to that E-Mail. Please let me know what you think about it.
>
> As there are many PHP7 packages to be prepared and several requests are
> already waiting we should process quickly. I think we should not simply
> rename the old packages as most of them are completely outdated versions
> and packaged very complicated!
>

This is really awesome, thanks for the work done.
Making packaging fun again is a nice way of life ;-)

--

Bruno Friedmann
 Ioda-Net Sàrl www.ioda-net.ch
 Bareos Partner, openSUSE Member, fsfe fellowship
 GPG KEY : D5C9B751C4653227
 irc: tigerfoot


--
To unsubscribe, e-mail: [hidden email]
To contact the owner, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: New packaging guidelines for PHP7 pacakges

Christian Boltz-5
In reply to this post by Johannes Weberhofer-2
Hello,

Am Mittwoch, 30. August 2017, 15:22:07 CEST schrieb Johannes Weberhofer:
>      I will check that those macros will be included in php5 and php7
> packages in the future, but until that's done the macros can must be
> included in the spec files. Currently "BuildRequires:
> %{php_name}-macros" is required, but this can be moved to the "%if
> 0%{?suse_version} < 1320" as soon as PHP7 has been updated in factory

This should probably be <= 1320 (unless you want to exclude 13.2, which
sounds unlikely)

Besides that, your proposal looks good :-)


Regards,

Christian Boltz
--
*hrhr* Ich hab die Tage mal statt nen Gruselfilm zu gucken das Win8
Betadings runtergeladen und in ner VM installiert ... *schauder* :D
[David Haller in opensuse-de]

--
To unsubscribe, e-mail: [hidden email]
To contact the owner, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: New packaging guidelines for PHP7 pacakges

Dominique Leuenberger / DimStar
In reply to this post by Johannes Weberhofer-2
On Wed, 2017-08-30 at 15:22 +0200, Johannes Weberhofer wrote:
> Dear all!
>
>      I will check that those macros will be included in php5 and php7
> packages in the future, but until that's done the macros can must be
> included in the spec files. Currently "BuildRequires: %{php_name}-
> macros" is required, but this can be moved to the "%if
> 0%{?suse_version} < 1320" as soon as PHP7 has been updated in factory

php5 is scheduled for removal from openSUSE:Factory - so adding new
macros there seems like a waste of your time. Focus on PHP7 'only'

Cheers,
Dominique

signature.asc (201 bytes) Download Attachment
Reply | Threaded
Open this post in threaded view
|

Re: New packaging guidelines for PHP7 pacakges

Johannes Weberhofer-2
Am 31.08.2017 um 13:15 schrieb Dominique Leuenberger / DimStar:
> php5 is scheduled for removal from openSUSE:Factory - so adding new
> macros there seems like a waste of your time. Focus on PHP7 'only'

I know, and it was only intended to be there as a proof of concept.

Maybe some maintainers in the devel project want to use it as long as PHP5 exists...

--
Johannes Weberhofer

--
To unsubscribe, e-mail: [hidden email]
To contact the owner, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: New packaging guidelines for PHP7 pacakges

Johannes Weberhofer-2
In reply to this post by Christian Boltz-5
Am 30.08.17 um 22:01 schrieb Christian Boltz:
> This should probably be <= 1320 (unless you want to exclude 13.2, which
> sounds unlikely)
>
> Besides that, your proposal looks good:-)

Thanks for you response!

Leap seems to be 1330 at the moment; I'll update this.

--
Johannes Weberhofer
Weberhofer GmbH, Austria, Vienna
--
To unsubscribe, e-mail: [hidden email]
To contact the owner, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: New packaging guidelines for PHP7 pacakges

Andrew Daugherity-3
> On Aug 31, 2017, at 2:07 PM, Johannes Weberhofer <[hidden email]> wrote:
>
> Am 30.08.17 um 22:01 schrieb Christian Boltz:
>> This should probably be <= 1320 (unless you want to exclude 13.2, which
>> sounds unlikely)
>> Besides that, your proposal looks good:-)
>
> Thanks for you response!
>
> Leap seems to be 1330 at the moment; I'll update this.

Not in 42.x.  Looks like Tumbleweed is currently 1330 (according to https://en.opensuse.org/openSUSE:Packaging_for_Leap#RPM_Distro_Version_Macros, but footnoted as "never rely on this version, it may change").


$ rpm --eval %suse_version
1315

$ grep '^%.*_version' /usr/lib/rpm/suse_macros
%suse_version 1315
%sles_version 0
%ul_version 0
%leap_version 420300
%sle_version 120300

$ cat /etc/os-release
NAME="openSUSE Leap"
VERSION="42.3"
ID=opensuse
ID_LIKE="suse"
VERSION_ID="42.3"
PRETTY_NAME="openSUSE Leap 42.3"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:opensuse:leap:42.3"
BUG_REPORT_URL="https://bugs.opensuse.org"
HOME_URL="https://www.opensuse.org/“


Thanks for your work; hope this leads to more packages making their way from server:php:extensions to server:php:extensions:php7 (assuming that’s what we’re supposed to use?).  BTW, that repo has 3 packages named php5-* — old packages which should be purged?  There are corresponding php7-* packages.


-AndrewN�����r��y隊Z)z{.��ZrF��x>�{.n�+������Ǩ��r��i�m��0��ޙ���������$j���0�����Ǩ�
Reply | Threaded
Open this post in threaded view
|

Re: New packaging guidelines for PHP7 pacakges

Johannes Weberhofer-2
In reply to this post by Johannes Weberhofer-2
Am 01.09.2017 um 01:21 schrieb Andrew Daugherity:
>> Leap seems to be 1330 at the moment; I'll update this.
>
> Not in 42.x.  Looks like Tumbleweed is currently 1330 (according to https://en.opensuse.org/openSUSE:Packaging_for_Leap#RPM_Distro_Version_Macros, but footnoted as "never rely on this version, it may change").
>

I meant Tumbleweed but wrote Leap...

--
Johannes Weberhofer
Weberhofer GmbH, Austria, Vienna
--
To unsubscribe, e-mail: [hidden email]
To contact the owner, e-mail: [hidden email]