Separate --enable for each draft RR type

--enable-rrtype-ninfo
--enable-rrtype-rkey
--enable-rrtype-cds
--enable-rrtype-uri
--enable-rrtype-ta
And removal of --enable-draft-rrtypes

diff --git a/Changelog b/Changelog
index 9ea163365476fb230fe44de600f5b3a2e1a6760d..dc635f2d5e922639eb7e294941c059c2d0aed39e 100644 (file)
--- a/Changelog
+++ b/Changelog
@@ -53,6 +53,9 @@
        * Optimize TSIG digest function name comparison (Thanks Marc Buijsman)
        * Compare names case insensitive with ldns_pkt_rr_list_by_name and
          ldns_pkt_rr_list_by_name_and_type (thanks Johannes Naab)
+       * Replace --enable-draft-rrtypes with a seperate --enable for each
+         draft RR type: --enable-rrtype-ninfo, --enable-rrtype-rkey,
+         --enable-rrtype-cds, --enable-rrtype-uri and --enable-rrtype-ta
 
 1.6.16 2012-11-13
        * Fix Makefile to build pyldns with BSD make

diff --git a/configure.ac b/configure.ac
index a157186f7d85a60dcc885b42ab40c8ef7d5a539c..74d12c953c37d7e777b2322d479c1266742d42d7 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -390,10 +390,42 @@ case "$enable_dane" in
       ;;
 esac
 
-AC_ARG_ENABLE(draft-rrtypes, AC_HELP_STRING([--enable-draft-rrtypes], [Enable draft RRtypes.]))
-case "$enable_draft_rrtypes" in
+AC_ARG_ENABLE(rrtype-ninfo, AC_HELP_STRING([--enable-rrtype-ninfo], [Enable draft RR type ninfo.]))
+case "$enable_rrtype_ninfo" in
        yes)
-               AC_DEFINE_UNQUOTED([DRAFT_RRTYPES], [], [Define this to enable draft RRtypes.])
+               AC_DEFINE_UNQUOTED([RRTYPE_NINFO], [], [Define this to enable RR type NINFO.])
+               ;;
+       no|*)
+               ;;
+esac
+AC_ARG_ENABLE(rrtype-rkey, AC_HELP_STRING([--enable-rrtype-rkey], [Enable draft RR type rkey.]))
+case "$enable_rrtype_rkey" in
+       yes)
+               AC_DEFINE_UNQUOTED([RRTYPE_RKEY], [], [Define this to enable RR type RKEY.])
+               ;;
+       no|*)
+               ;;
+esac
+AC_ARG_ENABLE(rrtype-cds, AC_HELP_STRING([--enable-rrtype-cds], [Enable draft RR type cds.]))
+case "$enable_rrtype_cds" in
+       yes)
+               AC_DEFINE_UNQUOTED([RRTYPE_CDS], [], [Define this to enable RR type CDS.])
+               ;;
+       no|*)
+               ;;
+esac
+AC_ARG_ENABLE(rrtype-uri, AC_HELP_STRING([--enable-rrtype-uri], [Enable draft RR type uri.]))
+case "$enable_rrtype_uri" in
+       yes)
+               AC_DEFINE_UNQUOTED([RRTYPE_URI], [], [Define this to enable RR type URI.])
+               ;;
+       no|*)
+               ;;
+esac
+AC_ARG_ENABLE(rrtype-ta, AC_HELP_STRING([--enable-rrtype-ta], [Enable draft RR type ta.]))
+case "$enable_rrtype_ta" in
+       yes)
+               AC_DEFINE_UNQUOTED([RRTYPE_TA], [], [Define this to enable RR type TA.])
                ;;
        no|*)
                ;;

diff --git a/ldns/rr.h b/ldns/rr.h
index 1485cadf1ee5e9fbc9829e610cf81bb97063440b..ff499395c00b11ca3422cf75152a32fbb08c8c97 100644 (file)
--- a/ldns/rr.h
+++ b/ldns/rr.h
@@ -204,9 +204,8 @@ enum ldns_enum_rr_type
        LDNS_RR_TYPE_L64 = 106, /* RFC 6742 */
        LDNS_RR_TYPE_LP = 107, /* RFC 6742 */
 
-       /** draft-jabley-dnsext-eui48-eui64-rrtypes */
-       LDNS_RR_TYPE_EUI48 = 108,
-       LDNS_RR_TYPE_EUI64 = 109,
+       LDNS_RR_TYPE_EUI48 = 108, /* RFC 7043 */
+       LDNS_RR_TYPE_EUI64 = 109, /* RFC 7043 */
 
        LDNS_RR_TYPE_TKEY = 249, /* RFC 2930 */
        LDNS_RR_TYPE_TSIG = 250,

diff --git a/rr.c b/rr.c
index 4b28f4de93f2bc96d57697b3f0e49a03489e9ab1..01aad321bb75b91939de3e87ff235d6f4a61d832 100644 (file)
--- a/rr.c
+++ b/rr.c
@@ -1991,13 +1991,13 @@ static const ldns_rdf_type type_lp_wireformat[] = {
        LDNS_RDF_TYPE_INT16,
        LDNS_RDF_TYPE_DNAME
 };
-#ifdef DRAFT_RRTYPES
 static const ldns_rdf_type type_eui48_wireformat[] = {
        LDNS_RDF_TYPE_EUI48
 };
 static const ldns_rdf_type type_eui64_wireformat[] = {
        LDNS_RDF_TYPE_EUI64
 };
+#ifdef RRTYPE_URI
 static const ldns_rdf_type type_uri_wireformat[] = {
        LDNS_RDF_TYPE_INT16,
        LDNS_RDF_TYPE_INT16,
@@ -2134,19 +2134,22 @@ static ldns_rr_descriptor rdata_field_descriptors[] = {
         */
        {LDNS_RR_TYPE_HIP, "HIP", 1, 1, type_hip_wireformat, LDNS_RDF_TYPE_DNAME, LDNS_RR_NO_COMPRESS, 0 },
 
-#ifdef DRAFT_RRTYPES
+#ifdef RRTYPE_NINFO
        /* 56 */
        {LDNS_RR_TYPE_NINFO, "NINFO", 1, 0, NULL, LDNS_RDF_TYPE_STR, LDNS_RR_NO_COMPRESS, 0 },
+#else
+{LDNS_RR_TYPE_NULL, "TYPE56", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
+#fi
+#ifdef RRTYPE_RKEY
        /* 57 */
        {LDNS_RR_TYPE_RKEY, "RKEY", 4, 4, type_key_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 #else
-{LDNS_RR_TYPE_NULL, "TYPE56", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 {LDNS_RR_TYPE_NULL, "TYPE57", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 #endif
        /* 58 */
        {LDNS_RR_TYPE_TALINK, "TALINK", 2, 2, type_talink_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 2 },
 
-#ifdef DRAFT_RRTYPES
+#ifdef RRTYPE_CDS
        /* 59 */
        {LDNS_RR_TYPE_CDS, "CDS", 4, 4, type_ds_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 #else
@@ -2213,16 +2216,10 @@ static ldns_rr_descriptor rdata_field_descriptors[] = {
        {LDNS_RR_TYPE_L64, "L64", 2, 2, type_l64_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
        /* 107 */
        {LDNS_RR_TYPE_LP, "LP", 2, 2, type_lp_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 1 },
-
-#ifdef DRAFT_RRTYPES
        /* 108 */
        {LDNS_RR_TYPE_EUI48, "EUI48", 1, 1, type_eui48_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
        /* 109 */
        {LDNS_RR_TYPE_EUI64, "EUI64", 1, 1, type_eui64_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
-#else
-{LDNS_RR_TYPE_NULL, "TYPE108", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
-{LDNS_RR_TYPE_NULL, "TYPE109", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
-#endif
 
 {LDNS_RR_TYPE_NULL, "TYPE110", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 {LDNS_RR_TYPE_NULL, "TYPE111", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
@@ -2386,7 +2383,7 @@ static ldns_rr_descriptor rdata_field_descriptors[] = {
        /* ANY: A request for all (available) records */
 {LDNS_RR_TYPE_NULL, "TYPE255", 1, 1, type_0_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 
-#ifdef DRAFT_RRTYPES
+#ifdef RRTYPE_URI
        /* 256 */
        {LDNS_RR_TYPE_URI, "URI", 3, 3, type_uri_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 #else
@@ -2397,7 +2394,7 @@ static ldns_rr_descriptor rdata_field_descriptors[] = {
 
 /* split in array, no longer contiguous */
 
-#ifdef DRAFT_RRTYPES
+#ifdef RRTYPE_TA
        /* 32768 */
        {LDNS_RR_TYPE_TA, "TA", 4, 4, type_ds_wireformat, LDNS_RDF_TYPE_NONE, LDNS_RR_NO_COMPRESS, 0 },
 #else