Merge pull request #13415 from rouault/fix_13400

gdal raster convert: avoid error message when outputing to /vsistdout/

Author: rouault

apps/gdal_translate_lib.cpp

@@ -1120,7 +1120,8 @@ GDALDatasetH GDALTranslate(const char *pszDest, GDALDatasetH hSrcDataset,
                 "@QUIET_DELETE_ON_CREATE_COPY", "NO");
         }
 
-        if (psOptions->bNoOverwrite && !EQUAL(pszDest, ""))
+        if (psOptions->bNoOverwrite && !EQUAL(pszDest, "") &&
+            !EQUAL(pszDest, "/vsistdout/"))
         {
             VSIStatBufL sStat;
             if (VSIStatL(pszDest, &sStat) == 0)
@@ -1131,13 +1132,23 @@ GDALDatasetH GDALTranslate(const char *pszDest, GDALDatasetH hSrcDataset,
                          pszDest);
                 return nullptr;
             }
-            else if (std::unique_ptr<GDALDataset>(GDALDataset::Open(pszDest)))
+            else
             {
-                CPLError(CE_Failure, CPLE_AppDefined,
-                         "Dataset '%s' already exists. Specify the --overwrite "
-                         "option to overwrite it.",
-                         pszDest);
-                return nullptr;
+                bool bExists;
+                {
+                    CPLErrorStateBackuper oBackuper(CPLQuietErrorHandler);
+                    bExists = std::unique_ptr<GDALDataset>(
+                                  GDALDataset::Open(pszDest)) != nullptr;
+                }
+                if (bExists)
+                {
+                    CPLError(
+                        CE_Failure, CPLE_AppDefined,
+                        "Dataset '%s' already exists. Specify the --overwrite "
+                        "option to overwrite it.",
+                        pszDest);
+                    return nullptr;
+                }
             }
         }
 

autotest/utilities/test_gdalalg_raster_convert.py

@@ -102,3 +102,19 @@ def test_gdalalg_raster_convert_comma_in_filename(tmp_vsimem):
 
     convert = get_convert_alg()
     assert convert.ParseRunAndFinalize([tmp_vsimem / "a,b.tif", out_filename])
+
+
+@pytest.mark.require_driver("XYZ")
+def test_gdalalg_raster_convert_to_vsistdout(tmp_vsimem):
+    import gdaltest
+    import test_cli_utilities
+
+    gdal_path = test_cli_utilities.get_gdal_path()
+    if gdal_path is None:
+        pytest.skip("gdal binary missing")
+    out, err = gdaltest.runexternal_out_and_err(
+        f"{gdal_path} raster convert --of XYZ ../gcore/data/byte.tif /vsistdout/"
+    )
+    gdal.FileFromMemBuffer(tmp_vsimem / "test.xyz", out)
+    assert gdal.Open(tmp_vsimem / "test.xyz")
+    assert err == ""