Bug 530824

Summary: php_stream_filter_api.h:65: error: comma at end of enumerator list
Product: Red Hat Enterprise Linux 5 Reporter: Robert Scheck <redhat-bugzilla>
Component: phpAssignee: Joe Orton <jorton>
Status: CLOSED ERRATA QA Contact: BaseOS QE - Apps <qe-baseos-apps>
Severity: high Docs Contact:
Priority: low    
Version: 5.4CC: kai2+redhat, rbgarga, tao, vanmeeuwen+fedora
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
URL: http://forums.zarafa.com/viewtopic.php?f=9&t=1002&start=0#p13748
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-03-30 08:24:54 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 498194    

Description Robert Scheck 2009-10-25 13:39:50 UTC
Description of problem:
/usr/include/php/main/streams/php_stream_filter_api.h:65: error: comma at end 
of enumerator list

--- snipp ---
typedef enum {
  PSFS_ERR_FATAL, /* error in data stream */
  PSFS_FEED_ME,   /* filter needs more data; stop processing chain until more is available */
  PSFS_PASS_ON,   /* filter generated output buckets; pass them on to next in chain */
} php_stream_filter_status_t;
--- snapp ---

That tiny mistake seems to avoid that we can ship Zarafa in EPEL 5. Remove the
last comma, http://forums.zarafa.com/viewtopic.php?f=9&t=1002&start=0#p13748 -
and building works.

Version-Release number of selected component (if applicable):
php-devel-5.1.6-23.2.el5_3

How reproducible:
Everytime if you try to build Zarafa on Red Hat Enterprise Linux 5.

Actual results:
/usr/include/php/main/streams/php_stream_filter_api.h:65: error: comma at end 
of enumerator list

Expected results:
No error and working stuff...

Comment 1 Joe Orton 2009-10-25 14:30:16 UTC
Thanks for the report.

Comment 2 Robert Scheck 2009-10-25 14:42:10 UTC
Joe, do we have a serious chance to get a fix simply pushed out with the
next PHP update for RHEL 5 (e.g. with the next PHP security and/or bugfix
update)? As it's really a trivial fix, we maybe don't need to escalate
that the big way via Red Hat Release Engineering etc.?

Comment 3 Joe Orton 2009-10-25 15:00:30 UTC
I've nominated the bug for inclusion in the next bug fix update - I can't say whether that'll be RHEL 5.5 or later.  It might be possible to include the fix in a security update in the mean time, I will bear this in mind; no guarantees though.

Comment 4 Jeroen van Meeuwen 2009-10-25 18:48:49 UTC
As a customer, I would appreciate a little more certainty before this is going to take a year to get fixed (e.g., by the time 5.6 hits).

Comment 7 Joe Orton 2009-12-16 12:42:19 UTC
I've made test packages available which should fix this issue.  These
packages are unsupported, have not been through the standard Red Hat
QA process, and are not recommended for use on production systems.

   http://people.redhat.com/~jorton/Tikanga-php/

Use of these packages may prevent you from (automatically) upgrading
to any asynchronous security errata which are issued before the
release of RHEL 5.5 due to version mismatches.

Please record any feedback on use of these test packages (positive or
negative!) on this bug report.

Comment 10 Joe Orton 2010-02-12 11:24:18 UTC
*** Bug 564307 has been marked as a duplicate of this bug. ***

Comment 11 Renato Botelho 2010-02-12 11:38:14 UTC
Is this the way redhat.com take care of its packages? A known and simple-to-fix bug, and the package just stay *BROKEN* until next release? This is my first contact with RedHat and it's so disappointing...

Almost 4 months since bug was reported and reporter got a "I will bear this in mind; no guarantees though.", what's the concept of "stability" for you guys?

Comment 12 Kai Schaetzl 2010-02-13 14:31:55 UTC
Come on:
- this effects only very few users
- it is *easily* worked around
- it does not affect operation of php *at all*
This has nothing to do with stability or security, no need to push it in an errata update, really.

Comment 13 Joe Orton 2010-02-13 16:08:59 UTC
Renato - I'm sorry you are disappointed.   We generally do not push bug fixes out in security updates unless they are of high priority, so I still cannot make any promises that will happen.  

If this bug is particularly severe for you (or anybody else commenting here), I strongly recommend that you contact Red Hat Support in the first instance.  Prioritisation is in the main based on customer feedback via Support.

Otherwise; see comment 7 for details of the test packages which are available.

Comment 14 Renato Botelho 2010-02-15 12:46:51 UTC
(In reply to comment #13)
> Renato - I'm sorry you are disappointed.   We generally do not push bug fixes
> out in security updates unless they are of high priority, so I still cannot
> make any promises that will happen.  
> 
> If this bug is particularly severe for you (or anybody else commenting here), I
> strongly recommend that you contact Red Hat Support in the first instance. 
> Prioritisation is in the main based on customer feedback via Support.
> 
> Otherwise; see comment 7 for details of the test packages which are available.    

I'm sorry to make noise, it's just my concept is different than yours. After 10 years involved with FreeBSD project, i'm just familiar with its concepts. There when we have a problem, it doesn't matter if it affects 1 ore a million of users, what matter is "it affects the operating system", and we don't want an operating system with problems (small or big ones).

Since things here follow a more Microsoft-like way, you'll release the fix when you want to do. I just needed to show my POV.

But based on this, i'll start to use CentOS instead of paying to use RHEL, CentOS community seem to be fast responsive in this cases and i don't need to pay ;)

About the package, i won't use a non-official rpm, but thanks anyway.

Comment 15 Renato Botelho 2010-02-15 12:47:43 UTC
(In reply to comment #12)
> Come on:
> - this effects only very few users
> - it is *easily* worked around
> - it does not affect operation of php *at all*
> This has nothing to do with stability or security, no need to push it in an
> errata update, really.    

Sorry about the noise, i'll back to my cave. :)

Comment 17 errata-xmlrpc 2010-03-30 08:24:54 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHBA-2010-0241.html