numpy
diff --git a/‎numpy/_core/_add_newdocs.py
Copy file name to clipboardExpand all lines: numpy/_core/_add_newdocs.py
+2-2Lines changed: 2 additions & 2 deletions b/‎numpy/_core/_add_newdocs.py
Copy file name to clipboardExpand all lines: numpy/_core/_add_newdocs.py
+2-2Lines changed: 2 additions & 2 deletions
diff --git a/‎numpy/_core/src/multiarray/dlpack.c
Copy file name to clipboardExpand all lines: numpy/_core/src/multiarray/dlpack.c
+4-5Lines changed: 4 additions & 5 deletions b/‎numpy/_core/src/multiarray/dlpack.c
Copy file name to clipboardExpand all lines: numpy/_core/src/multiarray/dlpack.c
+4-5Lines changed: 4 additions & 5 deletions
diff --git a/‎numpy/_core/tests/test_dlpack.py
Copy file name to clipboardExpand all lines: numpy/_core/tests/test_dlpack.py
+11Lines changed: 11 additions & 0 deletions b/‎numpy/_core/tests/test_dlpack.py
Copy file name to clipboardExpand all lines: numpy/_core/tests/test_dlpack.py
+11Lines changed: 11 additions & 0 deletions
@@ -1663,8 +1663,8 @@
     from_dlpack(x, /, *, device=None, copy=None)
 
     Create a NumPy array from an object implementing the ``__dlpack__``
-    protocol. Generally, the returned NumPy array is a read-only view
-    of the input object. See [1]_ and [2]_ for more details.
+    protocol. Generally, the returned NumPy array is a view of the input
+    object. See [1]_ and [2]_ for more details.
 
     Parameters
     ----------
 
@@ -601,7 +601,7 @@ from_dlpack(PyObject *NPY_UNUSED(self),
             return NULL;
         }
         dl_tensor = managed->dl_tensor;
-        readonly = 0;
+        readonly = 1;
     }
 
     const int ndim = dl_tensor.ndim;
@@ -702,14 +702,13 @@ from_dlpack(PyObject *NPY_UNUSED(self),
     }
 
     PyObject *ret = PyArray_NewFromDescr(&PyArray_Type, descr, ndim, shape,
-            dl_tensor.strides != NULL ? strides : NULL, data, 0, NULL);
+            dl_tensor.strides != NULL ? strides : NULL, data, readonly ? 0 :
+            NPY_ARRAY_WRITEABLE, NULL);
+
     if (ret == NULL) {
         Py_DECREF(capsule);
         return NULL;
     }
-    if (readonly) {
-        PyArray_CLEARFLAGS((PyArrayObject *)ret, NPY_ARRAY_WRITEABLE);
-    }
 
     PyObject *new_capsule;
     if (versioned) {
 
@@ -144,6 +144,17 @@ def test_readonly(self):
         y = np.from_dlpack(x)
         assert not y.flags.writeable
 
+    def test_writeable(self):
+        x_new, x_old = new_and_old_dlpack()
+
+        # new dlpacks respect writeability
+        y = np.from_dlpack(x_new)
+        assert y.flags.writeable
+
+        # old dlpacks are not writeable for backwards compatibility
+        y = np.from_dlpack(x_old)
+        assert not y.flags.writeable
+
     def test_ndim0(self):
         x = np.array(1.0)
         y = np.from_dlpack(x)