Your message dated Tue, 09 Sep 2014 10:03:38 +0200
with message-id <1410249818.579.12.camel@merkur>
and subject line Debian bug #544223
has caused the Debian Bug report #544223,
regarding argyll uses embedded yajl
to be marked as done.
This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.
(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)
--
544223: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=544223
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: argyll
Version: 1.0.4-1
Severity: normal
Tags: patch
Argyll embeds an old version of yajl (0.3.0) in its source code. Now
that yajl is in Debian, I'm attaching a patch that lets it compile
against the system yajl instead.
Note: Argyll modified yajl's API so argyll's jcnf can save comments.
But from what I can see, ucmm/ucmm.c is the only file that uses jcnf,
and it doesn't take advantage of that feature. So even though the patch
comments out a bit of argyll's code, it shouldn't affect its
functionality.
Regards,
John Stamp
--- a/jcnf/Makefile.am
+++ b/jcnf/Makefile.am
@@ -2,15 +2,13 @@
include $(top_srcdir)/Makefile.shared
-SUBDIRS = yajl
-
privatelib_LTLIBRARIES = libjcnf.la
privatelibdir = $(pkglibdir)
libjcnf_la_SOURCES = jcnf.h jcnf.c
-libjcnf_la_LIBADD = yajl/libyajl.la
+libjcnf_la_LIBADD = -lyajl
-LDADD = ./libjcnf.la yajl/libyajl.la
+LDADD = ./libjcnf.la -lyajl
check_PROGRAMS = test
--- a/jcnf/jcnf.c
+++ b/jcnf/jcnf.c
@@ -42,9 +42,9 @@
#include <string.h>
#include <math.h>
-#include "yajl/yajl_common.h"
-#include "yajl/yajl_gen.h"
-#include "yajl/yajl_parse.h"
+#include <yajl/yajl_common.h>
+#include <yajl/yajl_gen.h>
+#include <yajl/yajl_parse.h>
#include "jcnf.h"
@@ -955,9 +955,10 @@
}
}
+ /*
if (p->keys[i]->comment != NULL) {
yajl_gen_cpp_comment(g, p->keys[i]->comment, strlen(p->keys[i]->comment));
- }
+ }*/
#ifdef NEVER
yajl_gen_map_open(g);
--- a/spectro/Makefile.am
+++ b/spectro/Makefile.am
@@ -22,10 +22,10 @@
../numlib/libargyllnum.la ./libinst.la ../libargyll.la
LDADD = ./libinsttypes.la ./libdisp.la ./libinst.la ../ucmm/libucmm.la \
- ../jcnf/libjcnf.la ../jcnf/yajl/libyajl.la ../xicc/libxicc.la \
+ ../jcnf/libjcnf.la ../xicc/libxicc.la \
../xicc/libxcolorants.la $(ICC_LIBS) ../cgats/libcgats.la \
../rspl/librspl.la ../gamut/libgamut.la ../target/libtarget.la \
- ../plot/libplot.la ../numlib/libargyllnum.la $(X_LIBS) -lusb
+ ../plot/libplot.la ../numlib/libargyllnum.la $(X_LIBS) -lusb -lyajl
bin_PROGRAMS = synthcal dispwin dispread dispcal fakeread synthread \
chartread spotread spec2cie average spyd2en
--- End Message ---
--- Begin Message ---
Hello,
the upstream maintainer is working on some new functions at the included
yajl.
[quote]
Some of that functionality isn't critical (support for JSON comments),
while some new code I'm working on depends on an additional function
inside yajl_tree.c. Since the structures it uses are private to that
file, changes to yajl can't be avoided. yajl_tree is fairly useless
without this added function, since yajl_tree_get() doesn't cope with
JSON arrays.
[/quote]
I tag this bug to wontfix and close it.
Please feel free to reopen this bug.
Thanks for your work.
CU
Jörg
--
pgp Fingerprint: 7D13 3C60 0A10 DBE1 51F8 EBCB 422B 44B0 BE58 1B6E
pgp Key: BE581B6E
CAcert Key S/N: 0E:D4:56
Jörg Frings-Fürst
D-54526 Niederkail
Threema: SYR8SJXB
IRC: j_...@freenode.net
j_...@oftc.net
signature.asc
Description: This is a digitally signed message part
--- End Message ---