8000 Doc: replace outdated references to trac.osgeo.org/gdal by current gdal.org ones by rouault · Pull Request #810 · postgis/postgis · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rouault
Copy link
Contributor
@rouault rouault commented May 31, 2025

No description provided.

Copy link
coderabbitai bot commented May 31, 2025
📝 Walkthrough

Walkthrough

Multiple 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

File(s) Change Summary
doc/po/da/postgis-manual.po
doc/po/de/postgis-manual.po
doc/po/es/postgis-manual.po
doc/po/fr/postgis-manual.po
doc/po/it_IT/postgis-manual.po
doc/po/ja/postgis-manual.po
doc/po/ka/postgis-manual.po
doc/po/ko_KR/postgis-manual.po
doc/po/pl/postgis-manual.po
doc/po/pt_BR/postgis-manual.po
doc/po/ro/postgis-manual.po
doc/po/ru/postgis-manual.po
doc/po/uk/postgis-manual.po
doc/po/zh_Hans/postgis-manual.po
Updated GDAL documentation URLs in translation files from trac.osgeo.org to gdal.org.
doc/po/templates/postgis-manual.pot Updated GDAL documentation URLs in the documentation template to use gdal.org.
doc/reference_guc.xml Updated GDAL configuration option URLs in GUC variable documentation to point to gdal.org.
doc/reference_raster.xml Updated example and environment variable documentation URLs to current links under gdal.org.

Poem

🐇
Old links have hopped away,
New URLs now lead the way.
From Trac to GDAL’s shiny site,
Documentation’s future is bright!
Across languages near and far,
Clear directions—no matter where you are.
Hop on, the docs are up to date!


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8c7b26d and 7a90edb.

📒 Files selected for processing (17)
  • doc/po/da/postgis-manual.po (5 hunks)
  • doc/po/de/postgis-manual.po (5 hunks)
  • doc/po/es/postgis-manual.po (5 hunks)
  • doc/po/fr/postgis-manual.po (5 hunks)
  • doc/po/it_IT/postgis-manual.po (5 hunks)
  • doc/po/ja/postgis-manual.po (5 hunks)
  • doc/po/ka/postgis-manual.po (5 hunks)
  • doc/po/ko_KR/postgis-manual.po (5 hunks)
  • doc/po/pl/postgis-manual.po (5 hunks)
  • doc/po/pt_BR/postgis-manual.po (5 hunks)
  • doc/po/ro/postgis-manual.po (5 hunks)
  • doc/po/ru/postgis-manual.po (5 hunks)
  • doc/po/templates/postgis-manual.pot (5 hunks)
  • doc/po/uk/postgis-manual.po (5 hunks)
  • doc/po/zh_Hans/postgis-manual.po (5 hunks)
  • doc/reference_guc.xml (3 hunks)
  • doc/reference_raster.xml (2 hunks)
✅ Files skipped from review due to trivial changes (3)
  • doc/po/ka/postgis-manual.po
  • doc/po/uk/postgis-manual.po
  • doc/reference_raster.xml
🚧 Files skipped from review as they are similar to previous changes (14)
  • doc/po/de/postgis-manual.po
  • doc/po/it_IT/postgis-manual.po
  • doc/po/ko_KR/postgis-manual.po
  • doc/po/templates/postgis-manual.pot
  • doc/po/fr/postgis-manual.po
  • doc/reference_guc.xml
  • doc/po/da/postgis-manual.po
  • doc/po/ru/postgis-manual.po
  • doc/po/pl/postgis-manual.po
  • doc/po/es/postgis-manual.po
  • doc/po/zh_Hans/postgis-manual.po
  • doc/po/ja/postgis-manual.po
  • doc/po/pt_BR/postgis-manual.po
  • doc/po/ro/postgis-manual.po
⏰ Context from checks skipped due to timeout of 90000ms (4)
  • GitHub Check: Analyze (c-cpp)
  • GitHub Check: macOS
  • GitHub Check: CI (pg13-geos38-gdal31-proj71, tests)
  • GitHub Check: CI (pg13-clang-geos39-gdal31-proj71, usan_clang)

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.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need 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)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
@coderabbitai coderabbitai bot left a 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 issue

Translation 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 issue

Missing 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 issue

Incorrect path in French translation link
The translation’s link href points to frmts_wtkraster.html instead of the intended drivers/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 issue

Incorrect URL path in translation link
The German translation’s href is split into https://gdal.org/user/frmts_wtkraster.html, which doesn’t match the intended https://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 issue

Missing update of translation link in msgstr for GDAL_DISABLE_READDIR_ON_OPEN
This translation still references the old trac.osgeo.org path; it should use https://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 issue

Missing update of translation link in msgstr for GDAL_DATA
The Chinese translation still references the old trac.osgeo.org URL. It should point to the updated https://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 issue

Missing 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 old trac.osgeo.org URL as display text, even though the href has been updated. Please revise the link text to https://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 old trac.osgeo.org URL. Please update it to match the new gdal.org URL in the msgid.

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 deprecated trac.osgeo.org link. It should match the new https://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 new gdal.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.

@rouault rouault changed the title Doc: replace outdated referances to trac.osgeo.org/gdal by current gdal.org ones Doc: replace outdated references to trac.osgeo.org/gdal by current gdal.org ones May 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0