[DTrace-devel] [PATCH 3/4] Sync up the version numbers
eugene.loh at oracle.com
eugene.loh at oracle.com
Sat Feb 8 19:06:21 UTC 2025
From: Eugene Loh <eugene.loh at oracle.com>
DTrace has many version numbers -- e.g., for the release, packaging,
and API.
In reality, the variations in numbering have become nearly meaningless:
- Packaging numbers -- like the Version in the dtrace*spec RPM spec
file and the VERSION in GNUmakefile -- have basically been tracking
the DTrace release since 2.0 anyway.
- Stability attributes for idents are haphazard. Generally, they
are 1.0 (or sometimes other 1.x), and all the BPFs are 2.0. While
this is generally accurate, it is not exactly robust, and idents
are sometimes introduced or modified without careful regard to
the version number. Further, the stability of user D scripts is
likely to depend more on kernel variations -- e.g., the contents of
available_filter_functions -- than on D changes.
- Version number updates are susceptible to mistakes, and so there
have been version mismatches.
Bring the version numbers into sync for 2.0.2. Clean up the descriptions
in dt_version.h.
Signed-off-by: Eugene Loh <eugene.loh at oracle.com>
---
GNUmakefile | 5 +-
dtrace.spec | 1 +
libdtrace/dt_open.c | 3 ++
libdtrace/dt_version.h | 53 ++++++++++++--------
test/unittest/dtrace-util/tst.APIVersion.r | 2 +-
test/unittest/dtrace-util/tst.APIVersion.r.p | 6 +++
test/unittest/options/tst.version.r | 2 +-
test/unittest/options/tst.version.sh | 4 +-
8 files changed, 50 insertions(+), 26 deletions(-)
create mode 100755 test/unittest/dtrace-util/tst.APIVersion.r.p
diff --git a/GNUmakefile b/GNUmakefile
index d1e18bb1b..ddf997878 100644
--- a/GNUmakefile
+++ b/GNUmakefile
@@ -3,7 +3,7 @@
# Build files in subdirectories are included by this file.
#
# Oracle Linux DTrace.
-# Copyright (c) 2011, 2024, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2011, 2025, Oracle and/or its affiliates. All rights reserved.
# Licensed under the Universal Permissive License v 1.0 as shown at
# http://oss.oracle.com/licenses/upl.
@@ -14,7 +14,8 @@
SHELL = /bin/bash
PROJECT := dtrace
-VERSION := 2.0.1
+# When updating version, see comments in dt_version.h.
+VERSION := 2.0.2
# Verify supported hardware.
diff --git a/dtrace.spec b/dtrace.spec
index 902ad7d8b..46950bf5a 100644
--- a/dtrace.spec
+++ b/dtrace.spec
@@ -87,6 +87,7 @@ Requires: libdtrace-ctf >= 1.1.0
BuildRequires: libdtrace-ctf-devel >= 1.1.0
%endif
Summary: DTrace user interface.
+# When updating version, see comments in dt_version.h.
Version: 2.0.2
Release: 1%{?dist}
Source: dtrace-%{version}.tar.bz2
diff --git a/libdtrace/dt_open.c b/libdtrace/dt_open.c
index b4d160359..b843228b3 100644
--- a/libdtrace/dt_open.c
+++ b/libdtrace/dt_open.c
@@ -58,6 +58,9 @@ const dt_version_t _dtrace_versions[] = {
DT_VERS_1_6_3, /* D API 1.6.3 */
DT_VERS_1_6_4, /* D API 1.6.4 */
DT_VERS_2_0, /* D API 2.0 */
+ DT_VERS_2_0_1, /* D API 2.0.1 */
+ DT_VERS_2_0_2, /* D API 2.0.2 */
+ /* When updating version, see comments in dt_version.h. */
0
};
diff --git a/libdtrace/dt_version.h b/libdtrace/dt_version.h
index bef3243e9..4e230ceae 100644
--- a/libdtrace/dt_version.h
+++ b/libdtrace/dt_version.h
@@ -38,32 +38,28 @@ extern "C" {
*
* These #defines are used in identifier tables to fill in the version fields
* associated with each identifier. The DT_VERS_* macros declare the encoded
- * integer values of all versions used so far. DT_VERS_STRING must be an ASCII
- * string that contains the latest version within it along with any suffixes
- * (e.g. Beta). You must update DT_VERS_STRING when adding a new version,
- * and then add the new version to the _dtrace_versions[] array declared in
- * dt_open.c.
+ * integer values of all versions used so far.
*
- * Refer to the Solaris Dynamic Tracing Guide Versioning chapter for an
- * explanation of these DTrace features and their values.
+ * The major number should be incremented when a fundamental change has been
+ * made that would affect all consumers, and would reflect sweeping changes
+ * to DTrace or the D language.
+ *
+ * The minor number should be incremented when a change is introduced that
+ * could break scripts that had previously worked; for example, adding a
+ * new built-in variable could break a script which was already using that
+ * identifier.
+ *
+ * The micro number should be changed when introducing functionality changes
+ * or major bug fixes that do not affect backward compatibility -- this is
+ * merely to make capabilities easily determined from the version number.
+ *
+ * Minor bugs do not require any modification to the version number.
*
* NOTE: Although the DTrace versioning scheme supports the labeling and
* introduction of incompatible changes (e.g. dropping an interface in a
* major release), the libdtrace code does not currently support this.
* All versions are assumed to strictly inherit from one another. If
* we ever need to provide divergent interfaces, this will need work.
- *
- * The version number should be increased for every customer visible release
- * of Solaris. The major number should be incremented when a fundamental
- * change has been made that would affect all consumers, and would reflect
- * sweeping changes to DTrace or the D language. The minor number should be
- * incremented when a change is introduced that could break scripts that had
- * previously worked; for example, adding a new built-in variable could break
- * a script which was already using that identifier. The micro number should
- * be changed when introducing functionality changes or major bug fixes that
- * do not affect backward compatibility -- this is merely to make capabilities
- * easily determined from the version number. Minor bugs do not require any
- * modification to the version number.
*/
#define DT_VERS_1_0 DT_VERSION_NUMBER(1, 0, 0)
#define DT_VERS_1_1 DT_VERSION_NUMBER(1, 1, 0)
@@ -81,8 +77,25 @@ extern "C" {
#define DT_VERS_1_6_4 DT_VERSION_NUMBER(1, 6, 4)
#define DT_VERS_2_0 DT_VERSION_NUMBER(2, 0, 0)
#define DT_VERS_2_0_1 DT_VERSION_NUMBER(2, 0, 1)
+#define DT_VERS_2_0_2 DT_VERSION_NUMBER(2, 0, 2)
+
+/*
+ * When the version number is updated, the following must be kept in sync:
+ *
+ * libdtrace/dt_version.h DT_VERS_STRING, an ASCII string that contains
+ * the latest version within it along with any
+ * suffixes (e.g. Beta)
+ *
+ * libdtrace/dt_open.c _dtrace_versions[]
+ *
+ * dtrace.spec Version
+ *
+ * libdtrace/Build libdtrace_VERSION
+ *
+ * GNUmakefile VERSION
+ */
-#define DT_VERS_STRING "Oracle D 2.0"
+#define DT_VERS_STRING "Oracle D 2.0.2"
#ifdef __cplusplus
}
diff --git a/test/unittest/dtrace-util/tst.APIVersion.r b/test/unittest/dtrace-util/tst.APIVersion.r
index 6bc7b9d72..02bf03150 100644
--- a/test/unittest/dtrace-util/tst.APIVersion.r
+++ b/test/unittest/dtrace-util/tst.APIVersion.r
@@ -1 +1 @@
-dtrace: Oracle D 2.0
+dtrace: Oracle D 2.0.x
diff --git a/test/unittest/dtrace-util/tst.APIVersion.r.p b/test/unittest/dtrace-util/tst.APIVersion.r.p
new file mode 100755
index 000000000..32ec94df4
--- /dev/null
+++ b/test/unittest/dtrace-util/tst.APIVersion.r.p
@@ -0,0 +1,6 @@
+#!/usr/bin/gawk -f
+
+# The test allows the version string to vary in micro number as well as
+# other suffixes (like "Beta"). The .r.p and .r files still need to be
+# updated for each minor number change.
+{ sub("^dtrace: Oracle D 2\\.0\\..*$", "dtrace: Oracle D 2.0.x"); print }
diff --git a/test/unittest/options/tst.version.r b/test/unittest/options/tst.version.r
index 15010b3db..882e208ed 100644
--- a/test/unittest/options/tst.version.r
+++ b/test/unittest/options/tst.version.r
@@ -1,2 +1,2 @@
-version is 2.0
+version is 2.0.x
diff --git a/test/unittest/options/tst.version.sh b/test/unittest/options/tst.version.sh
index ffffcdd8b..684120af8 100755
--- a/test/unittest/options/tst.version.sh
+++ b/test/unittest/options/tst.version.sh
@@ -1,7 +1,7 @@
#!/bin/bash
#
# Oracle Linux DTrace.
-# Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2023, 2025, Oracle and/or its affiliates. All rights reserved.
# Licensed under the Universal Permissive License v 1.0 as shown at
# http://oss.oracle.com/licenses/upl.
#
@@ -9,7 +9,7 @@
dtrace=$1
myversion=`$dtrace $dt_flags -V | gawk '{ print $NF }'`
-echo version is $myversion
+echo version is $myversion | sed 's:2.0.[0-9]:2.0.x:'
$dtrace $dt_flags -xversion=$myversion -qn 'BEGIN { exit(0) }'
exit $?
--
2.43.5
More information about the DTrace-devel
mailing list