Bug 194479

When I updated bug #194470 to announce the fixed php-magickwand package, I also 
updated php-idn to match with all common things - unfortunately this was during 
the unrecoverable time of Bugzilla...

On my current testing system I get this:

[hans@localhost ~]$ rpmbuild -ba /usr/src/redhat/SPECS/php-idn.spec 
sh: php-config: command not found
awk: cmd. line:1: fatal: cannot open file `/main/php.h' for reading (No such
file or directory)
error: line 12: Version required: Requires:     php-api =

This is because php-devel isn't installed yet, its BuildRequired, but rpmbuild
first fully parses the spec before checking BR, so this fails. Some discussion
on the mailinglist about a similar problem with python on FC-4 has lead me to
believe that the same will happen on the buildsys when building under mock.

Ok,

I've found the following "hack" for this, add at the top of your specfile:
# Useful defaults when building in chroots on systems where PHP is unavailable
# during the get BR step of the build
%define default_apiver  20041225
%define php_apiver %((echo %{default_apiver}; php -i 2>/dev/null | sed -n 's/^P

And then change the Requires line to:
Requires:       php-api = %{php_apiver}

The same should be done for php-magickwand btw.

MUST:
=====
* rpmlint output is clean
* Package and spec file named appropriately
* Packaged according to packaging guidelines
* License (GPL) ok, license file included
* spec file is legible and in Am. English.
* Source matches upstream
* Compiles and builds on FC5-i386
* BR: ok
* No locales
* No shared libraries (its a plugin)
* Not relocatable
* Package owns / or requires all dirs
* No duplicate files & Permissions ok
* %clean & macro usage OK
* Contains code only
* %doc does not affect runtime, and isn't large enough to warrent a sub package
* no -devel package needed, no libs / .la files.
* no gui -> no .desktop file required


MUST fix:
=========
The use of php-config before the php-devel BR is resolved, see above.

One line from your comment was trunicated, but I found the complete hack in php-
eaccelerator: %((echo %{default_apiver}; php -i 2>/dev/null | sed -n 's/^PHP API 
=> //p') | tail -1) - it is applied very similar as suggested.

(In reply to comment #5)
> One line from your comment was trunicated, but I found the complete hack in php-
> eaccelerator: %((echo %{default_apiver}; php -i 2>/dev/null | sed -n 's/^PHP API 
> => //p') | tail -1) - it is applied very similar as suggested.

Aai, sorry about that cut and paste error. Glad you found the full hack yourself.
That fixes the only MUST fix item -> Approved.

And I must say I'm happy with the shown packageing skills sofat and thus I am
willing to sponsor you, go create an account in the account system as described
here:
http://fedoraproject.org/wiki/Extras/Contributors

And then I'll sponsor you after which you can continue with step described at
the above URL. You need to find another reviewer for you other 2 packages
though, I'm no good in perl and ircbots aren't my thing either :)

11171 (php-idn): Build on target fedora-5-extras succeeded.
11172 (php-idn): Build on target fedora-development-extras succeeded.

as per http://fedoraproject.org/wiki/Extras/Contributors I'll close this
bug report with NEXTRELEASE now. If I did something wrong or when I missed 
something, just tell me.

Are we allowed to approve php packages now?  I thought we were still waiting for
finished guidelines and an update of the core PHP package.

Jason, didn't I get something or am I lacking any information? I applied last 
must fix before comment #5, FE-ACCEPT was added with comment #6. After getting 
sponsored, I just followed the rest of http://fedoraproject.org/wiki/Packaging/
ReviewGuidelines#head-f3a5010530b260ae2b3d8c835cc6d25e9ac91ceb 

Are there any blockers, show stoppers or further guidelines I'm not aware of? 
And as I'm new to that kind of the FE stuff, I'm very confused now...

(In reply to comment #8)
> Are we allowed to approve php packages now?  I thought we were still waiting for
> finished guidelines and an update of the core PHP package.

Not that I know of, besides there are alreayd plenty of php packages in extras.
I've read nothing of what you're suggesting. Do you have some pointers for this?

Package Change Request
======================
Package Name: php-idn
New Branches: epel7
Owners: robert

Git done (by process-git-requests).