php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #78353 php-yaml extension not compiled
Submitted: 2019-07-30 06:17 UTC Modified: 2019-08-06 10:18 UTC
Votes:1
Avg. Score:3.0 ± 0.0
Reproduced:0 of 1 (0.0%)
From: a dot dankovtsev at mail dot ru Assigned: bd808 (profile)
Status: Assigned Package: yaml (PECL)
PHP Version: master-Git-2019-07-30 (Git) OS: Mac OS
Private report: No CVE-ID: None
Have you experienced this issue?
Rate the importance of this bug to you:

 [2019-07-30 06:17 UTC] a dot dankovtsev at mail dot ru
Description:
------------
In php 8 breaking changes with removing TSRMLS_CC, TSRMLS_DC
I try fix it by removing this macros from this extension, but i don'k know true fix it.
Or being more right solve.


Patches

0001-remove-deleted-macros.patch (last revision 2019-08-05 05:35 UTC by a dot dankovtsev at mail dot ru)
0001-remove-deleted-macros (last revision 2019-07-30 06:18 UTC by a dot dankovtsev at mail dot ru)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2019-07-30 06:18 UTC] a dot dankovtsev at mail dot ru
The following patch has been added/updated:

Patch Name: 0001-remove-deleted-macros
Revision:   1564467488
URL:        https://bugs.php.net/patch-display.php?bug=78353&patch=0001-remove-deleted-macros&revision=1564467488
 [2019-07-30 06:55 UTC] cmb@php.net
-Package: *Extensibility Functions +Package: yaml
 [2019-07-31 14:27 UTC] a dot dankovtsev at mail dot ru
In additional previous patch, fix with segmentation fault on fresh master

patch:
```
From beea93b37f952da48e610984de71450666606ddc Mon Sep 17 00:00:00 2001
From: Aleksandr Dankovtsev <adankovtsev@avito.ru>
Date: Wed, 31 Jul 2019 17:24:08 +0300
Subject: [PATCH] convert to zend_object

---
 emit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/emit.c b/emit.c
index c931b8d..a6a7bbd 100644
--- a/emit.c
+++ b/emit.c
@@ -640,7 +640,7 @@ static int y_write_timestamp(
 	ZVAL_STRING(&dtfmt, "Y-m-d\\TH:i:s.uP");
 
 	/* format date as iso-8601 string */
-	zend_call_method_with_1_params(data, clazz, NULL, "format", &timestamp, &dtfmt);
+	zend_call_method_with_1_params(Z_OBJ_P(data), clazz, NULL, "format", &timestamp, &dtfmt);
 
 	/* emit formatted date */
 	status = yaml_scalar_event_initialize(&event, NULL, tag,
-- 
2.17.2 (Apple Git-113)

```
 [2019-08-05 05:35 UTC] a dot dankovtsev at mail dot ru
The following patch has been added/updated:

Patch Name: 0001-remove-deleted-macros.patch
Revision:   1564983307
URL:        https://bugs.php.net/patch-display.php?bug=78353&patch=0001-remove-deleted-macros.patch&revision=1564983307
 [2019-08-05 05:35 UTC] a dot dankovtsev at mail dot ru
Refresh patch with test fixes
 [2019-08-06 10:18 UTC] cmb@php.net
-Assigned To: +Assigned To: bd808
 [2019-08-06 10:18 UTC] cmb@php.net
While this doesn't look urgent, could you please have a look at the patch, Bryan?
 [2019-10-27 10:41 UTC] a dot dankovtsev at mail dot ru
Can you validate changes?
 
PHP Copyright © 2001-2019 The PHP Group
All rights reserved.
Last updated: Tue Nov 19 02:01:34 2019 UTC