-
-
Notifications
You must be signed in to change notification settings - Fork 394
Doc: replace outdated references to trac.osgeo.org/gdal by current gdal.org ones #810
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughMultiple documentation and translation files were updated to replace outdated GDAL documentation URLs (formerly pointing to trac.osgeo.org) with the current official links under gdal.org. These changes affect references in both source and translated documentation, ensuring all hyperlinks now direct to the maintained GDAL documentation. No code or logic was altered. Changes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (17)
✅ Files skipped from review due to trivial changes (3)
🚧 Files skipped from review as they are similar to previous changes (14)
⏰ Context from checks skipped due to timeout of 90000ms (4)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Caution
Inline review comments failed to post. This is likely due to GitHub's limits when posting large numbers of comments. If you are seeing this consistently it is likely a permissions issue. Please check "Moderation" -> "Code review limits" under your organization settings.
Actionable comments posted: 11
🧹 Nitpick comments (1)
doc/po/de/postgis-manual.po (1)
19198-19203
: Translate link text for localization consistency
In this hunk the link text remains in English (Configuration Options
) while other nearby entries translate it to German (Konfigurationsoptionen
). For consistency and better UX in the German manual, please localize the link text here as well.
🛑 Comments failed to post (11)
doc/po/fr/postgis-manual.po (3)
19128-19128:
⚠️ Potential issueTranslation inconsistency: English link text in French msgstr
The French translation still uses the English phrase “Configuration Options.” It should be localized to “Options de configuration.”Proposed fix:
- "GDAL <link xlink:href=\"https://gdal.org/user/configoptions.html\">Configuration Options</link>." + "GDAL <link xlink:href=\"https://gdal.org/user/configoptions.html\">Options de configuration</link>."Also applies to: 19132-19133
🤖 Prompt for AI Agents
In doc/po/fr/postgis-manual.po at lines 19128 and 19132-19133, the French translation uses the English phrase "Configuration Options" in the msgstr fields. Replace these English phrases with the correct French translation "Options de configuration" to ensure consistency and proper localization.
19264-19265:
⚠️ Potential issueMissing update in French translation: deprecated trac.osgeo.org link
The msgstr still references the old Trac URL. It must be updated to the new gdal.org path and match the link text.Proposed fix:
- "l'utilisation d'un raster out-db. <link xlink:href=\"http://trac.osgeo.org/gdal/wiki/configoptions.html\">Les options de configuration</link> contrôlent des " + "l'utilisation d'un raster out-db. <link xlink:href=\"https://gdal.org/user/configoptions.html\">Options de configuration</link> contrôlent des "Also applies to: 19271-19271
🤖 Prompt for AI Agents
In doc/po/fr/postgis-manual.po around lines 19264 to 19265 and also line 19271, update the French translation msgstr entries to replace the deprecated trac.osgeo.org URLs with the new gdal.org URLs. Ensure the link text in the translation matches the updated URL to maintain consistency and accuracy.
37346-37347:
⚠️ Potential issueIncorrect path in French translation link
The translation’s link href points tofrmts_wtkraster.html
instead of the intendeddrivers/raster/postgisraster.html
.Proposed fix:
- <link xlink:href="https://gdal.org/user/frmts_wtkraster.html">https://gdal.org/user/drivers/raster/postgisraster.html</link> + <link xlink:href="https://gdal.org/user/drivers/raster/postgisraster.html">https://gdal.org/user/drivers/raster/postgisraster.html</link>Also applies to: 37351-37352
🤖 Prompt for AI Agents
In doc/po/fr/postgis-manual.po around lines 37346 to 37347 and also lines 37351 to 37352, the href attribute in the French translation links incorrectly points to frmts_wtkraster.html. Update these href values to the correct path drivers/raster/postgisraster.html to fix the broken or incorrect links in the translation.
doc/po/de/postgis-manual.po (1)
37553-37554:
⚠️ Potential issueIncorrect URL path in translation link
The German translation’shref
is split intohttps://gdal.org/user/frmts_wtkraster.html
, which doesn’t match the intendedhttps://gdal.org/user/drivers/raster/postgisraster.html
. This will break the hyperlink.Apply something like:
- <link xlink:href="https://gdal.org/user/frmts_wtkraster.html"> + <link xlink:href="https://gdal.org/user/drivers/raster/postgisraster.html">Also applies to: 37558-37559
🤖 Prompt for AI Agents
In doc/po/de/postgis-manual.po at lines 37553-37554 and 37558-37559, the href URL in the German translation is incorrect and split, causing broken links. Update the href attribute to the correct full URL "https://gdal.org/user/drivers/raster/postgisraster.html" in both places to fix the hyperlink.
doc/po/zh_Hans/postgis-manual.po (3)
35330-35334:
⚠️ Potential issueMissing update of translation link in msgstr for GDAL_DISABLE_READDIR_ON_OPEN
This translation still references the oldtrac.osgeo.org
path; it should usehttps://gdal.org/user/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN
.Apply this diff:
- "为了控制此行为,GDAL 提供了以下环境变量: <link xlink:href=\"https://trac.osgeo.org/gdal/wiki/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN\">GDAL_DISABLE_READDIR_ON_OPEN</link>。 将 <varname>GDAL_DISABLE_READDIR_ON_OPEN</varname> 设置为<varname> TRUE </varname>以禁用目录扫描。" + "为了控制此行为,GDAL 提供了以下环境变量: <link xlink:href=\"https://gdal.org/user/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN\">GDAL_DISABLE_READDIR_ON_OPEN</link>。 将 <varname>GDAL_DISABLE_READDIR_ON_OPEN</varname> 设置为<varname> TRUE </varname>以禁用目录扫描。"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.- "为了控制此行为,GDAL 提供了以下环境变量: <link xlink:href=\"https://trac.osgeo.org/gdal/wiki/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN\">GDAL_DISABLE_READDIR_ON_OPEN</link>。 将 <varname>GDAL_DISABLE_READDIR_ON_OPEN</varname> 设置为<varname> TRUE </varname>以禁用目录扫描。" + "为了控制此行为,GDAL 提供了以下环境变量: <link xlink:href=\"https://gdal.org/user/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN\">GDAL_DISABLE_READDIR_ON_OPEN</link>。 将 <varname>GDAL_DISABLE_READDIR_ON_OPEN</varname> 设置为<varname> TRUE </varname>以禁用目录扫描。"
🤖 Prompt for AI Agents
In doc/po/zh_Hans/postgis-manual.po around lines 35330 to 35334, the translation for GDAL_DISABLE_READDIR_ON_OPEN still uses the outdated trac.osgeo.org URL. Update the msgstr to replace the old URL with the new one: https://gdal.org/user/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN to ensure the translation link is current and correct.
17015-17016:
⚠️ Potential issueMissing update of translation link in msgstr for GDAL_DATA
The Chinese translation still references the oldtrac.osgeo.org
URL. It should point to the updatedhttps://gdal.org/user/configoptions.html
.Apply this diff:
- "有关 GDAL_DATA 的其他信息可在 GDAL 的<link xlink:href=\"http://trac.osgeo.org/gdal/wiki/configoptions.html\">配置选项</link>中找到。" + "有关 GDAL_DATA 的其他信息可在 GDAL 的<link xlink:href=\"https://gdal.org/user/configoptions.html\">配置选项</link>中找到。"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements."有关 GDAL_DATA 的其他信息可在 GDAL 的<link xlink:href=\"https://gdal.org/user/configoptions.html\">配置选项</link>中找到。"
🤖 Prompt for AI Agents
In doc/po/zh_Hans/postgis-manual.po around lines 17015 to 17016, the translation string for GDAL_DATA still contains the outdated trac.osgeo.org URL. Update the msgstr to replace the old URL with the new one: https://gdal.org/user/configoptions.html to ensure the translation points to the current resource.
17139-17140:
⚠️ Potential issueMissing update of translation link in msgstr for GDAL_SKIP
The translated string still uses the outdated Trac URL. Update it to the new GDAL documentation URL.Apply this diff:
- "有关 GDAL_SKIP 的其他信息可在 GDAL 的<link xlink:href=\"http://trac.osgeo.org/gdal/wiki/configoptions.html\">配置选项</link>中找到。" + "有关 GDAL_SKIP 的其他信息可在 GDAL 的<link xlink:href=\"https://gdal.org/user/configoptions.html\">配置选项</link>中找到。"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements."有关 GDAL_SKIP 的其他信息可在 GDAL 的<link xlink:href=\"https://gdal.org/user/configoptions.html\">配置选项</link>中找到。"
🤖 Prompt for AI Agents
In doc/po/zh_Hans/postgis-manual.po around lines 17139 to 17140, the translation string for GDAL_SKIP contains an outdated Trac URL. Update the msgstr to replace the old Trac URL with the new GDAL documentation URL to ensure the translation link is current and accurate.
doc/po/ja/postgis-manual.po (4)
35497-35505: 🛠️ Refactor suggestion
Fix translation hyperlink text for raster driver example.
The
msgstr
uses the oldtrac.osgeo.org
URL as display text, even though thehref
has been updated. Please revise the link text tohttps://gdal.org/user/drivers/raster/postgisraster.html
to match the new location.Proposed diff:
--- a/doc/po/ja/postgis-manual.po +++ b/doc/po/ja/postgis-manual.po @@ -35502,3 +35502,3 @@ msgstr "" - "<link xlink:href=\"https://gdal.org/user/drivers/raster/postgisraster.html\">http://trac.osgeo.org/gdal/wiki/frmts_wtkraster.html</link>" + "<link xlink:href=\"https://gdal.org/user/drivers/raster/postgisraster.html\">https://gdal.org/user/drivers/raster/postgisraster.html</link>"Committable suggestion skipped: line range outside the PR's diff.
🤖 Prompt for AI Agents
In doc/po/ja/postgis-manual.po around lines 35497 to 35505, the translation msgstr contains a hyperlink with outdated display text using the old trac.osgeo.org URL, while the href points to the updated gdal.org URL. Update the hyperlink display text in msgstr to exactly match the new URL https://gdal.org/user/drivers/raster/postgisraster.html so that the link text and href are consistent.
18126-18131: 🛠️ Refactor suggestion
Update translated link for GDAL_DATA section.
The
msgstr
still references the oldtrac.osgeo.org
URL. Please update it to match the newgdal.org
URL in themsgid
.Proposed diff:
--- a/doc/po/ja/postgis-manual.po +++ b/doc/po/ja/postgis-manual.po @@ -18130,2 +18130,2 @@ msgstr "" - "GDAL_DATAに関する追加情報報は、GDALの<link xlink:href=\"http://trac.osgeo.org/gdal/wiki/configoptions.html\">Configuration Options</link>にあります。" + "GDAL_DATAに関する追加情報報は、GDALの<link xlink:href=\"https://gdal.org/user/configoptions.html\">Configuration Options</link>にあります。"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.msgstr "" "GDAL_DATAに関する追加情報報は、GDALの<link xlink:href=\"https://gdal.org/user/configoptions.html\">Configuration Options</link>にあります。"
🤖 Prompt for AI Agents
In doc/po/ja/postgis-manual.po around lines 18126 to 18131, the translated string (msgstr) still contains the old trac.osgeo.org URL. Update the msgstr to replace the old URL with the new gdal.org URL so it matches the msgid's updated link.
18253-18258: 🛠️ Refactor suggestion
Update translated link for GDAL_SKIP section.
The
msgstr
still points to the deprecatedtrac.osgeo.org
link. It should match the newhttps://gdal.org/user/configoptions.html
URL.Proposed diff:
--- a/doc/po/ja/postgis-manual.po +++ b/doc/po/ja/postgis-manual.po @@ -18257,2 +18257,2 @@ msgstr "" - "GDAL_SKIPに関する追加情報は、GDALの<link xlink:href=\"http://trac.osgeo.org/gdal/wiki/configoptions.html\">Configuration Options</link>にあります。" + "GDAL_SKIPに関する追加情報は、GDALの<link xlink:href=\"https://gdal.org/user/configoptions.html\">Configuration Options</link>にあります。"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.msgstr "" "GDAL_SKIPに関する追加情報は、GDALの<link xlink:href=\"https://gdal.org/user/configoptions.html\">Configuration Options</link>にあります。"
🤖 Prompt for AI Agents
In doc/po/ja/postgis-manual.po around lines 18253 to 18258, the translated message string (msgstr) contains an outdated URL pointing to trac.osgeo.org for the GDAL_SKIP section. Update the msgstr to replace the deprecated link with the current URL https://gdal.org/user/configoptions.html to ensure the translation references the correct and active documentation.
37408-37416: 🛠️ Refactor suggestion
Update translated link for GDAL_DISABLE_READDIR_ON_OPEN.
The current
msgstr
still references the old GDAL Trac URL. Please update it to the newgdal.org
location.Proposed diff:
--- a/doc/po/ja/postgis-manual.po +++ b/doc/po/ja/postgis-manual.po @@ -37413,4 +37413,4 @@ msgstr "" - "<link xlink:href=\"https://trac.osgeo.org/gdal/wiki/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN\">GDAL_DISABLE_READDIR_ON_OPEN</link>" + "<link xlink:href=\"https://gdal.org/user/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN\">GDAL_DISABLE_READDIR_ON_OPEN</link>"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.msgstr "" "<link xlink:href=\"https://gdal.org/user/configoptions.html#GDAL_DISABLE_READDIR_ON_OPEN\">GDAL_DISABLE_READDIR_ON_OPEN</link>"
🤖 Prompt for AI Agents
In doc/po/ja/postgis-manual.po around lines 37408 to 37416, the translated string for GDAL_DISABLE_READDIR_ON_OPEN contains an outdated GDAL Trac URL. Update the msgstr to replace the old URL with the new gdal.org URL to ensure the link points to the current and correct location.
No description provided.