[PEAR-BUG] Bug #20246 [Fbk->Csd]: Broken with php 5.5.9

[email protected]
Newsgroups php.pear.bugs
Message-ID <[email protected]>
Edit report at https://pear.php.net/bugs/bug.php?id=20246&edit=1

 ID:               20246
 Updated by:       [email protected]
 Reported By:      contact at greyboxconcepts dot com dot au
 Summary:          Broken with php 5.5.9
-Status:           Feedback
+Status:           Closed
 Type:             Bug
 Package:          Archive_Tar
 Operating System: Ubuntu 14.04
 Package Version:  1.3.11
 PHP Version:      5.5.9
-Assigned To:      
+Assigned To:      mrook
 Roadmap Versions: 
 New Comment:

-Status:      Feedback
+Status:      Closed
-Assigned To:
+Assigned To: mrook
This bug has been fixed in SVN.

If this was a documentation problem, the fix will appear on pear.php.net
by the end of next Sunday (CET).

If this was a problem with the pear.php.net website, the change should
be live shortly.

Otherwise, the fix will appear in the package's next release.

Thank you for the report and for helping us make PEAR better.




Previous Comments:
------------------------------------------------------------------------

[2014-08-01 13:23:08] mrook

I've changed the implementation to 
https://github.com/pear/Archive_Tar/commit/06e7b69bbe670bb3695fdde75e7937
d1e5295de2, please review.

------------------------------------------------------------------------

[2014-07-30 04:49:06] dmausner

CW, lines 814 & 888 are incorrect.  The previous code used
function_exists() and so does the new 
wrapper. Therefore, if gzopen64 exists, gzopen does not exist, and your
new wrapper will not get 
called.  Then the code will soft-abort.

Suggestion: remove the original function_exists() calls.

------------------------------------------------------------------------

[2014-07-29 14:02:14] mrook

I've pushed 
https://github.com/pear/Archive_Tar/commit/22431e47455c0a703c6145a38553d7
202a9cde10, can someone review that commit?

Thanks.

------------------------------------------------------------------------

[2014-07-23 07:51:52] mrook

-Roadmap Versions:
+Roadmap Versions: 1.3.12
I'm putting this on a roadmap. While I don't like wrapping those calls,
I don't think we 
have any alternative with those function signatures.

------------------------------------------------------------------------

[2014-07-23 02:45:29] dmausner

Agreed.  Someone with write privs on the code, please update the package
to either 
wrap the functions or check at each call.

The point is, the solution must work in either a 32 or 64 build.

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://pear.php.net/bugs/bug.php?id=20246

-- 
Edit this bug report at https://pear.php.net/bugs/bug.php?id=20246&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.