Skip Menu |

This queue is for tickets about the YAML-Syck CPAN distribution.

Report information
The Basics
Id: 78636
Status: resolved
Priority: 0/
Queue: YAML-Syck
People
Owner: Nobody in particular
Requestors: leosusanto [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 1.20
Fixed in: 1.30

History Show all quoted text
  Sun Jul 29 05:12:19 2012 leosusanto [...] gmail.com - Ticket created
Subject: Error while testing on MinGW64
Fixed after editing syck.h: typedef long st_data_t; -> typedef long long st_data_t; #define SYMID unsigned long -> #define SYMID unsigned long long
  Mon Jul 30 15:34:33 2012 leosusanto [...] gmail.com - Correspondence added
From: leosusanto [...] gmail.com
I created a patch based on the latest pull from github
Subject: 0001-Patched-for-MinGW64.patch
From 2e0e21499bba85b17dc89bd749465f44d8d11a74 Mon Sep 17 00:00:00 2001 From: Leo Susanto <leosusanto@gmail.com> Date: Mon, 30 Jul 2012 12:30:11 -0700 Subject: [PATCH] Patched for MinGW64 --- syck.h | 8 ++++++++ 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/syck.h b/syck.h index 90527cb..c1a265c 100644 --- a/syck.h +++ b/syck.h @@ -97,10 +97,18 @@ extern "C" { * Node definitions */ #ifndef ST_DATA_T_DEFINED +#ifndef MINGW64 +typedef long long st_data_t; +#else typedef long st_data_t; #endif +#endif +#ifndef MINGW64 +#define SYMID unsigned long long +#else #define SYMID unsigned long +#endif typedef struct _syck_node SyckNode; -- 1.7.7.msysgit.1
  Sat Mar 02 03:25:12 2013 TODDR [...] cpan.org - Correspondence added
I just saw this ticket. It looks good. It's now staged for the next release. Todd
  Sat Mar 02 03:25:14 2013 The RT System itself - Status changed from 'new' to 'open'
  Sat Mar 02 03:25:31 2013 TODDR [...] cpan.org - Status changed from 'open' to 'resolved'
  Sat Mar 02 03:57:54 2013 TODDR [...] cpan.org - Correspondence added
Actually I did have to apply 1 fix. You had the ifndef logic backward from what I could tell. It's been changed to this. Can you confirm this works for you? https://github.com/toddr/YAML-Syck/commit/7a64f93e22ffb32b001b5c90ee402d96e5bff420
  Sat Mar 02 03:57:55 2013 The RT System itself - Status changed from 'resolved' to 'open'
  Sat Mar 02 03:57:55 2013 TODDR [...] cpan.org - Status changed from 'open' to 'patched'
  Thu May 11 05:00:00 2017 TODDR [...] cpan.org - Status changed from 'patched' to 'resolved'
  Thu May 11 05:00:01 2017 TODDR [...] cpan.org - Fixed in 1.30 added