Updated all links to bugzilla#2522
Conversation
* ACE/ChangeLogs/ChangeLog-2002a:
* ACE/ChangeLogs/ChangeLog-2002b:
* ACE/ChangeLogs/ChangeLog-2003c:
* ACE/ChangeLogs/ChangeLog-2004a:
* ACE/ChangeLogs/ChangeLog-2004b:
* ACE/ChangeLogs/ChangeLog-2007b:
* ACE/ace/Abstract_Timer_Queue.h:
* ACE/docs/ACE-guidelines.html:
* ACE/tests/Based_Pointer_Test.cpp:
* ACE/tests/Bug_1576_Regression_Test.cpp:
* ACE/tests/Bug_1890_Regression_Test.cpp:
* ACE/tests/Bug_2368_Regression_Test.cpp:
* ACE/tests/Bug_2497_Regression_Test.cpp:
* ACE/tests/Bug_2540_Regression_Test.cpp:
* ACE/tests/Bug_2609_Regression_Test.cpp:
* ACE/tests/Bug_2610_Regression_Test.cpp:
* ACE/tests/Bug_2659_Regression_Test.cpp:
* ACE/tests/Bug_2740_Regression_Test.cpp:
* ACE/tests/Bug_2975_Regression_Test.cpp:
* ACE/tests/Bug_3102_Regression_Test.cpp:
* ACE/tests/Bug_3319_Regression_Test.cpp:
* ACE/tests/Bug_3500_Regression_Test.cpp:
* ACE/tests/Bug_3532_Regression_Test.cpp:
* ACE/tests/Bug_3541_Regression_Test.cpp:
* ACE/tests/Bug_3673_Regression_Test.cpp:
* ACE/tests/Bug_3709_Regression_Test.cpp:
* ACE/tests/Bug_3710_Regression_Test.cpp:
* ACE/tests/Bug_3911_Regression_Test.cpp:
* ACE/tests/Bug_3974_Regression_Test.cpp:
* ACE/tests/Compiler_Features_01_Test.cpp:
* ACE/tests/Compiler_Features_02_Test.cpp:
* ACE/tests/Compiler_Features_03_Test.cpp:
* ACE/tests/Compiler_Features_04_Test.cpp:
* ACE/tests/Compiler_Features_05_Test.cpp:
* ACE/tests/Compiler_Features_06_Test.cpp:
* ACE/tests/Compiler_Features_07_Test.cpp:
* ACE/tests/Compiler_Features_09_Test.cpp:
* ACE/tests/Compiler_Features_10_Test.cpp:
* ACE/tests/Compiler_Features_11_Test.cpp:
* ACE/tests/Compiler_Features_12_Test.cpp:
* ACE/tests/Compiler_Features_13_Test.cpp:
* ACE/tests/Compiler_Features_14_Test.cpp:
* ACE/tests/SSL/Bug_2912_Regression_Test.cpp:
* TAO/ChangeLogs/ChangeLog-2000b:
* TAO/ChangeLogs/ChangeLog-2001b:
* TAO/ChangeLogs/ChangeLog-2001c:
* TAO/ChangeLogs/ChangeLog-2002a:
* TAO/ChangeLogs/ChangeLog-2002b:
* TAO/ChangeLogs/ChangeLog-2002c:
* TAO/ChangeLogs/ChangeLog-2003b:
* TAO/ChangeLogs/ChangeLog-2003c:
* TAO/ChangeLogs/ChangeLog-2003e:
* TAO/ChangeLogs/ChangeLog-2004a:
* TAO/ChangeLogs/ChangeLog-2004b:
* TAO/ChangeLogs/ChangeLog-2005a:
* TAO/ChangeLogs/ChangeLog-2005b:
* TAO/ChangeLogs/ChangeLog-2009a:
* TAO/docs/Options.html:
* TAO/docs/Security/SSLIOP-USAGE.html:
* TAO/docs/releasenotes/TODO.html:
* TAO/docs/releasenotes/ec.html:
* TAO/docs/releasenotes/index.html:
* TAO/docs/releasenotes/orbcore.html:
* TAO/docs/rtcorba/issues.html:
* TAO/orbsvcs/orbsvcs/Event/EC_Dispatching_Task.inl:
* TAO/orbsvcs/orbsvcs/HTIOP/HTIOP_Acceptor.cpp:
* TAO/orbsvcs/tests/Bug_2248_Regression/README:
* TAO/orbsvcs/tests/Bug_2285_Regression/README:
* TAO/orbsvcs/tests/Bug_2287_Regression/README:
* TAO/orbsvcs/tests/Bug_2925_Regression/README:
* TAO/orbsvcs/tests/Bug_3215_Regression/README:
* TAO/orbsvcs/tests/Security/MT_IIOP_SSL/README:
* TAO/tao/IIOP_Acceptor.cpp:
* TAO/tao/Leader_Follower.h:
* TAO/tao/Strategies/DIOP_Acceptor.cpp:
* TAO/tao/Strategies/SCIOP_Acceptor.cpp:
* TAO/tests/Big_Oneways/README:
* TAO/tests/Big_Request_Muxing/README:
* TAO/tests/Big_Twoways/README:
* TAO/tests/Blocking_Sync_None/README:
* TAO/tests/Bug_3384_Regression/README.txt:
* TAO/tests/Crash_On_Write/README:
* TAO/tests/DynAny_Test/test_dynsequence.cpp:
* TAO/tests/LongUpcalls/README:
* TAO/tests/ORB_shutdown/README:
* TAO/tests/Stack_Recursion/README:
* ACE/docs/usage-bugzilla.html:
Deleted.
* ACE/docs/index.html:
* ACE/NEWS:
* TAO/NEWS:
WalkthroughThis PR systematically migrates bug references throughout the ACE/TAO codebase from legacy Vanderbilt Bugzilla URLs to new GitHub-hosted issue tracker URLs. The updates span changelog entries, test documentation, HTML documentation files, source code comments, and README files, with no functional code changes. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~18 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 10
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@ACE/tests/Bug_2975_Regression_Test.cpp`:
- Line 5: Replace or remove the broken GitHub URL in the file's header comment
(the line containing "https://github.com/DOCGroup/bugzilla/issues/2975");
investigate the correct location for the referenced issue (either the original
Bugzilla tracker or the current GitHub issues page for the DOCGroup repository),
then update the comment to a valid URL or replace it with a stable reference
(e.g., original Bugzilla issue URL or the correct GitHub issue number) so the
test header points to an accessible issue for Bug_2975_Regression_Test.cpp.
In `@ACE/tests/Bug_3319_Regression_Test.cpp`:
- Around line 4-5: Update the inconsistent bug ID in the header comment so both
references match the test filename and linked issue: change the text "Reproduces
the problems reported in bug 2975:" to use bug 3319 (so it reads "Reproduces the
problems reported in bug 3319:") to align with the URL
"https://github.com/DOCGroup/bugzilla/issues/3319".
In `@ACE/tests/Compiler_Features_01_Test.cpp`:
- Line 9: The comment contains a broken GitHub issue URL (issue `#3715`)
referenced in the file header; confirm the correct issue numbers for the
migrated references (3715, 3500, 2497) and update the comment text to use valid
GitHub issue URLs or remove the dead links—locate the header comment in
ACE/tests/Compiler_Features_01_Test.cpp (the comment line mentioning
"https://github.com/DOCGroup/bugzilla/issues/3715") and replace it with the
corrected URL(s) or a brief note (e.g., "original issue not available") so the
comment no longer points to a 404.
In `@ACE/tests/Compiler_Features_02_Test.cpp`:
- Line 9: The GitHub URL literal
"https://github.com/DOCGroup/bugzilla/issues/3715" is invalid; locate
occurrences of this URL in the test files (e.g., in
ACE/tests/Compiler_Features_02_Test.cpp and the other 12 similar test files) and
either replace it with the correct migrated URL or remove/update the reference
to point to the original Bugzilla issue (e.g., the Bugzilla bug URL or a valid
issue number), and ensure the ChangeLog phrase referencing "Bugzilla `#3715`" is
updated consistently to the corrected link or removed if the issue was not
migrated.
In `@TAO/ChangeLogs/ChangeLog-2000b`:
- Line 17187: Replace the corrupted contributor name "Pe�ataro" with the correct
Unicode spelling "Peñataro" in the ChangeLog entry; locate the line containing
the string "Pe�ataro" and update it to "Peñataro" (preserving the surrounding
punctuation and formatting) so the contributor is correctly attributed.
- Line 4515: Replace the corrupted character in the contributor name in the
string "to Roland R�denauer <roland.ruedenauer@freenet.de> for reporting" by
restoring the original spelling to "to Roland Rüdenauer
<roland.ruedenauer@freenet.de> for reporting", preserving the surrounding
punctuation and spacing; save the file with UTF-8 encoding to prevent
reintroducing the encoding artifact.
In `@TAO/ChangeLogs/ChangeLog-2005b`:
- Line 12079: Replace the broken GitHub URL
"https://github.com/DOCGroup/bugzilla/issues/2224" in the changelog entry with a
valid target: either restore the original Bugzilla link (e.g.,
"bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=2224") or substitute the correct
current issue URL if it exists elsewhere; update the ChangeLog-2005b entry so
the link points to an accessible issue location and verify the URL resolves.
In `@TAO/docs/releasenotes/index.html`:
- Line 2404: The two hardcoded links to DOCGroup/bugzilla issues (references
"3087" and "3327") in the release notes are broken; update the sentence that
mentions these issues to either point to the canonical tracker (replace both
bugzilla URLs with https://github.com/DOCGroup/ACE_TAO/issues) or explicitly
mark them as historical Bugzilla references not migrated to GitHub (e.g.,
“historical Bugzilla issues 3087 and 3327, not available online”); modify the
HTML fragment containing the sentence about server side 'Current'
implementations accordingly so the links are fixed or clarified.
In `@TAO/docs/releasenotes/orbcore.html`:
- Around line 57-58: Fix the malformed HTML in the issue-link list item: in the
fragment containing "1238</A>and<A
HREF=\"https://github.com/DOCGroup/bugzilla/issues/1239\">1239</A> <p></li>",
add spaces around the word "and" (i.e., "1238</A> and <A...") and remove the
erroneous opening paragraph tag "<p>" so the item properly closes with "</li>"
only; update the line with the corrected spacing and closing tag in the same
snippet.
In `@TAO/NEWS`:
- Line 4: Restore the original historical release note in TAO/NEWS (revert the
change that replaced the past release section value "None" with the repository
migration sentence) so the release semantics remain intact, and instead add the
migration information as a separate repo-maintenance or changelog entry (e.g.,
create or append to a MAINTENANCE or CHANGELOG file) describing the Bugzilla ->
GitHub migration and linking to https://github.com/DOCGroup/bugzilla; reference
the TAO/NEWS section you modified and the new MAINTENANCE/CHANGELOG file when
making these edits.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: 56354dc3-37c4-48d4-b238-6228ccddd728
📒 Files selected for processing (90)
ACE/ChangeLogs/ChangeLog-2002aACE/ChangeLogs/ChangeLog-2002bACE/ChangeLogs/ChangeLog-2003cACE/ChangeLogs/ChangeLog-2004aACE/ChangeLogs/ChangeLog-2004bACE/ChangeLogs/ChangeLog-2007bACE/NEWSACE/ace/Abstract_Timer_Queue.hACE/docs/ACE-guidelines.htmlACE/docs/index.htmlACE/docs/usage-bugzilla.htmlACE/tests/Based_Pointer_Test.cppACE/tests/Bug_1576_Regression_Test.cppACE/tests/Bug_1890_Regression_Test.cppACE/tests/Bug_2368_Regression_Test.cppACE/tests/Bug_2497_Regression_Test.cppACE/tests/Bug_2540_Regression_Test.cppACE/tests/Bug_2609_Regression_Test.cppACE/tests/Bug_2610_Regression_Test.cppACE/tests/Bug_2659_Regression_Test.cppACE/tests/Bug_2740_Regression_Test.cppACE/tests/Bug_2975_Regression_Test.cppACE/tests/Bug_3102_Regression_Test.cppACE/tests/Bug_3319_Regression_Test.cppACE/tests/Bug_3500_Regression_Test.cppACE/tests/Bug_3532_Regression_Test.cppACE/tests/Bug_3541_Regression_Test.cppACE/tests/Bug_3673_Regression_Test.cppACE/tests/Bug_3709_Regression_Test.cppACE/tests/Bug_3710_Regression_Test.cppACE/tests/Bug_3911_Regression_Test.cppACE/tests/Bug_3974_Regression_Test.cppACE/tests/Compiler_Features_01_Test.cppACE/tests/Compiler_Features_02_Test.cppACE/tests/Compiler_Features_03_Test.cppACE/tests/Compiler_Features_04_Test.cppACE/tests/Compiler_Features_05_Test.cppACE/tests/Compiler_Features_06_Test.cppACE/tests/Compiler_Features_07_Test.cppACE/tests/Compiler_Features_09_Test.cppACE/tests/Compiler_Features_10_Test.cppACE/tests/Compiler_Features_11_Test.cppACE/tests/Compiler_Features_12_Test.cppACE/tests/Compiler_Features_13_Test.cppACE/tests/Compiler_Features_14_Test.cppACE/tests/SSL/Bug_2912_Regression_Test.cppTAO/ChangeLogs/ChangeLog-2000bTAO/ChangeLogs/ChangeLog-2001bTAO/ChangeLogs/ChangeLog-2001cTAO/ChangeLogs/ChangeLog-2002aTAO/ChangeLogs/ChangeLog-2002bTAO/ChangeLogs/ChangeLog-2002cTAO/ChangeLogs/ChangeLog-2003bTAO/ChangeLogs/ChangeLog-2003cTAO/ChangeLogs/ChangeLog-2003eTAO/ChangeLogs/ChangeLog-2004aTAO/ChangeLogs/ChangeLog-2004bTAO/ChangeLogs/ChangeLog-2005aTAO/ChangeLogs/ChangeLog-2005bTAO/ChangeLogs/ChangeLog-2009aTAO/NEWSTAO/docs/Options.htmlTAO/docs/Security/SSLIOP-USAGE.htmlTAO/docs/releasenotes/TODO.htmlTAO/docs/releasenotes/ec.htmlTAO/docs/releasenotes/index.htmlTAO/docs/releasenotes/orbcore.htmlTAO/docs/rtcorba/issues.htmlTAO/orbsvcs/orbsvcs/Event/EC_Dispatching_Task.inlTAO/orbsvcs/orbsvcs/HTIOP/HTIOP_Acceptor.cppTAO/orbsvcs/tests/Bug_2248_Regression/READMETAO/orbsvcs/tests/Bug_2285_Regression/READMETAO/orbsvcs/tests/Bug_2287_Regression/READMETAO/orbsvcs/tests/Bug_2925_Regression/READMETAO/orbsvcs/tests/Bug_3215_Regression/READMETAO/orbsvcs/tests/Security/MT_IIOP_SSL/READMETAO/tao/IIOP_Acceptor.cppTAO/tao/Leader_Follower.hTAO/tao/Strategies/DIOP_Acceptor.cppTAO/tao/Strategies/SCIOP_Acceptor.cppTAO/tests/Big_Oneways/READMETAO/tests/Big_Request_Muxing/READMETAO/tests/Big_Twoways/READMETAO/tests/Blocking_Sync_None/READMETAO/tests/Bug_3384_Regression/README.txtTAO/tests/Crash_On_Write/READMETAO/tests/DynAny_Test/test_dynsequence.cppTAO/tests/LongUpcalls/READMETAO/tests/ORB_shutdown/READMETAO/tests/Stack_Recursion/README
💤 Files with no reviewable changes (2)
- ACE/docs/index.html
- ACE/docs/usage-bugzilla.html
| * | ||
| * Reproduces the problems reported in bug 2975: | ||
| * http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=2975 | ||
| * https://github.com/DOCGroup/bugzilla/issues/2975 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify the new GitHub bugzilla URL is accessible
# Test: Check if the new URL returns a successful HTTP response
curl -s -o /dev/null -w "%{http_code}" "https://github.com/DOCGroup/bugzilla/issues/2975"Repository: DOCGroup/ACE_TAO
Length of output: 63
🏁 Script executed:
# Search for the original Bugzilla URL or related issue references
rg "2975|bugzilla" --type cpp --type txt -A 2 -B 2Repository: DOCGroup/ACE_TAO
Length of output: 28238
🏁 Script executed:
# Check if there's any documentation about the URL migration
fd -e md -e txt | xargs grep -l "2975\|bugzilla\|GitHub issue" 2>/dev/null | head -20Repository: DOCGroup/ACE_TAO
Length of output: 141
🏁 Script executed:
# Look at the file context to understand what this regression test is about
head -20 ACE/tests/Bug_2975_Regression_Test.cppRepository: DOCGroup/ACE_TAO
Length of output: 475
The new GitHub bugzilla URL is inaccessible (returns HTTP 404).
The URL https://github.com/DOCGroup/bugzilla/issues/2975 returns a 404 error and cannot be verified. This requires investigation: either the URL format is incorrect, or the GitHub bugzilla repository/issue does not exist at that location. Confirm that the correct GitHub issues tracker URL has been used and that issue 2975 is accessible there.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@ACE/tests/Bug_2975_Regression_Test.cpp` at line 5, Replace or remove the
broken GitHub URL in the file's header comment (the line containing
"https://github.com/DOCGroup/bugzilla/issues/2975"); investigate the correct
location for the referenced issue (either the original Bugzilla tracker or the
current GitHub issues page for the DOCGroup repository), then update the comment
to a valid URL or replace it with a stable reference (e.g., original Bugzilla
issue URL or the correct GitHub issue number) so the test header points to an
accessible issue for Bug_2975_Regression_Test.cpp.
| * Reproduces the problems reported in bug 2975: | ||
| * http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=3319 | ||
| * https://github.com/DOCGroup/bugzilla/issues/3319 |
There was a problem hiding this comment.
Bug ID text and link are inconsistent.
Line 4 says bug 2975, while Line 5 links to issue 3319. Please align both to the same bug ID to avoid confusion.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@ACE/tests/Bug_3319_Regression_Test.cpp` around lines 4 - 5, Update the
inconsistent bug ID in the header comment so both references match the test
filename and linked issue: change the text "Reproduces the problems reported in
bug 2975:" to use bug 3319 (so it reads "Reproduces the problems reported in bug
3319:") to align with the URL
"https://github.com/DOCGroup/bugzilla/issues/3319".
| * in: | ||
| * | ||
| * http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=3715 | ||
| * https://github.com/DOCGroup/bugzilla/issues/3715 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify that a sample of the migrated GitHub issue URLs are accessible
echo "Checking GitHub issue URLs..."
for issue_id in 3715 1265 858 3500 1125 2497; do
url="https://github.com/DOCGroup/bugzilla/issues/${issue_id}"
echo "Checking: ${url}"
status_code=$(curl -s -o /dev/null -w "%{http_code}" -L "${url}")
if [ "$status_code" -eq 200 ]; then
echo " ✓ Issue ${issue_id} is accessible (HTTP ${status_code})"
else
echo " ✗ Issue ${issue_id} returned HTTP ${status_code}"
fi
doneRepository: DOCGroup/ACE_TAO
Length of output: 660
Fix broken GitHub issue URLs in comments.
Several of the migrated GitHub issue URLs are inaccessible. Verification shows issue IDs 3715, 3500, and 2497 return HTTP 404, including the URL referenced in this file. Verify that the issue numbers are correct and the URLs point to valid GitHub issues.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@ACE/tests/Compiler_Features_01_Test.cpp` at line 9, The comment contains a
broken GitHub issue URL (issue `#3715`) referenced in the file header; confirm the
correct issue numbers for the migrated references (3715, 3500, 2497) and update
the comment text to use valid GitHub issue URLs or remove the dead links—locate
the header comment in ACE/tests/Compiler_Features_01_Test.cpp (the comment line
mentioning "https://github.com/DOCGroup/bugzilla/issues/3715") and replace it
with the corrected URL(s) or a brief note (e.g., "original issue not available")
so the comment no longer points to a 404.
| * in: | ||
| * | ||
| * http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=3715 | ||
| * https://github.com/DOCGroup/bugzilla/issues/3715 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify the migrated bugzilla URL is accessible
# Check if the new GitHub issue URL returns a successful response
curl -s -o /dev/null -w "%{http_code}" -L "https://github.com/DOCGroup/bugzilla/issues/3715"Repository: DOCGroup/ACE_TAO
Length of output: 63
🏁 Script executed:
# Check if the DOCGroup/bugzilla repository exists and if issue 3715 is there
curl -s -w "\nStatus: %{http_code}\n" -H "Accept: application/vnd.github.v3+json" \
"https://api.github.com/repos/DOCGroup/bugzilla/issues/3715"Repository: DOCGroup/ACE_TAO
Length of output: 352
🏁 Script executed:
# Search for any documentation or migration mapping in the repository
rg "bugzilla.*3715|3715.*bugzilla" -iRepository: DOCGroup/ACE_TAO
Length of output: 1281
🏁 Script executed:
# Check the actual file to see what URL was changed
cat ACE/tests/Compiler_Features_02_Test.cpp | head -20Repository: DOCGroup/ACE_TAO
Length of output: 550
🏁 Script executed:
# Check if DOCGroup/bugzilla repository exists
curl -s -o /dev/null -w "%{http_code}" "https://github.com/DOCGroup/bugzilla"Repository: DOCGroup/ACE_TAO
Length of output: 63
🏁 Script executed:
# Search for issue 3715 in the ACE_TAO repository directly on GitHub
curl -s "https://github.com/DOCGroup/ACE_TAO/issues/3715" | head -50Repository: DOCGroup/ACE_TAO
Length of output: 69
🏁 Script executed:
# Look for any documentation about the bugzilla migration or issue 3715
rg "3715|bugzilla|migration" -i | grep -i "readme\|doc\|issue" | head -20Repository: DOCGroup/ACE_TAO
Length of output: 2026
🏁 Script executed:
# Check git log to see if there are any messages about this URL migration
git log --oneline --all | grep -i "bugzilla\|url\|migration\|3715" | head -10Repository: DOCGroup/ACE_TAO
Length of output: 633
🏁 Script executed:
# Find the bugzilla migration commit details
git show f719d1a90 --stat | head -20Repository: DOCGroup/ACE_TAO
Length of output: 951
🏁 Script executed:
# Look for any migration mapping file
find . -name "*migration*" -o -name "*bugzilla*" -type f | grep -v ".git"Repository: DOCGroup/ACE_TAO
Length of output: 89
🏁 Script executed:
# Get the git diff for this specific file to see the old URL
git log -p --all -- ACE/tests/Compiler_Features_02_Test.cpp | grep -A5 -B5 "bugzilla\|github" | head -40Repository: DOCGroup/ACE_TAO
Length of output: 42
🏁 Script executed:
# Check if the issue exists by looking for any references to it
rg "DOCGroup.*bugzilla.*3715|bugzilla.*3715.*github" --type-listRepository: DOCGroup/ACE_TAO
Length of output: 449
🏁 Script executed:
# Get the full diff of the migration commit
git show f719d1a90 | head -100Repository: DOCGroup/ACE_TAO
Length of output: 3097
🏁 Script executed:
# Search for when this file was created/modified
git log --all --follow -- ACE/tests/Compiler_Features_02_Test.cpp | head -20Repository: DOCGroup/ACE_TAO
Length of output: 619
🏁 Script executed:
# Check if issue 3715 might exist but under a different format or repo
curl -s "https://api.github.com/repos/DOCGroup/bugzilla" 2>&1 | head -20Repository: DOCGroup/ACE_TAO
Length of output: 339
🏁 Script executed:
# Look for any reference to what issue 3715 maps to in the original Bugzilla
rg "3715" --type-all | head -30Repository: DOCGroup/ACE_TAO
Length of output: 498
🏁 Script executed:
# Search for any migration mapping or bugzilla-related documentation
find . -type f \( -name "*.txt" -o -name "*.md" -o -name "*.html" \) | xargs grep -l "bugzilla\|migration" 2>/dev/null | head -10Repository: DOCGroup/ACE_TAO
Length of output: 379
🏁 Script executed:
# Look for commits that changed from old bugzilla URLs to new ones
git log --all --oneline --grep="bugzilla" | head -20Repository: DOCGroup/ACE_TAO
Length of output: 1275
🏁 Script executed:
# Search for issue 3715 references in any form
rg "3715" | head -20Repository: DOCGroup/ACE_TAO
Length of output: 1943
🏁 Script executed:
# Check if there's a NEWS file mentioning bugzilla migration
find . -name "NEWS" -o -name "CHANGES" | xargs cat 2>/dev/null | grep -A10 -B10 "bugzilla" | head -50Repository: DOCGroup/ACE_TAO
Length of output: 1756
The migrated GitHub URL is broken and returns a 404 error.
The URL https://github.com/DOCGroup/bugzilla/issues/3715 is inaccessible. While the original Bugzilla issue #3715 is referenced in the ChangeLog ("This completes the fixes for Buzilla #3715"), it was not successfully migrated to the GitHub repository. This same broken URL is repeated across 13 test files. Either the issue number changed during migration and the mapping needs to be corrected, or the issue was excluded from the migration and the reference should be removed or updated to point to the correct location.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@ACE/tests/Compiler_Features_02_Test.cpp` at line 9, The GitHub URL literal
"https://github.com/DOCGroup/bugzilla/issues/3715" is invalid; locate
occurrences of this URL in the test files (e.g., in
ACE/tests/Compiler_Features_02_Test.cpp and the other 12 similar test files) and
either replace it with the correct migrated URL or remove/update the reference
to point to the original Bugzilla issue (e.g., the Bugzilla bug URL or a valid
issue number), and ensure the ChangeLog phrase referencing "Bugzilla `#3715`" is
updated consistently to the corrected link or removed if the issue was not
migrated.
| If the -d (AST dump) option was used with the IDl compiler, | ||
| a null ostream was being passed to the dump function. Thanks | ||
| to Roland R�denauer <roland.ruedenauer@freenet.de> for reporting | ||
| to Roland R�denauer <roland.ruedenauer@freenet.de> for reporting |
There was a problem hiding this comment.
Fix corrupted character in contributor name
Line 4515 appears to have an encoding artifact (R�denauer). Please restore the original spelling (Rüdenauer) to avoid data loss in historical attribution text.
Proposed fix
- to Roland R�denauer <roland.ruedenauer@freenet.de> for reporting
+ to Roland Rüdenauer <roland.ruedenauer@freenet.de> for reporting📝 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.
| to Roland R�denauer <roland.ruedenauer@freenet.de> for reporting | |
| to Roland Rüdenauer <roland.ruedenauer@freenet.de> for reporting |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@TAO/ChangeLogs/ChangeLog-2000b` at line 4515, Replace the corrupted character
in the contributor name in the string "to Roland R�denauer
<roland.ruedenauer@freenet.de> for reporting" by restoring the original spelling
to "to Roland Rüdenauer <roland.ruedenauer@freenet.de> for reporting",
preserving the surrounding punctuation and spacing; save the file with UTF-8
encoding to prevent reintroducing the encoding artifact.
| files. This fixes a compile error with SunCC 4.2 | ||
| when using exceptions=1 and interface_repo=1. | ||
| Thanks to Ramiro Pe�ataro <penataro@ll.iac.es> | ||
| Thanks to Ramiro Pe�ataro <penataro@ll.iac.es> |
There was a problem hiding this comment.
Fix corrupted character in contributor name
Line 17187 appears to have an encoding artifact (Pe�ataro). Please restore the intended spelling (Peñataro) to preserve correct attribution.
Proposed fix
- Thanks to Ramiro Pe�ataro <penataro@ll.iac.es>
+ Thanks to Ramiro Peñataro <penataro@ll.iac.es>📝 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.
| Thanks to Ramiro Pe�ataro <penataro@ll.iac.es> | |
| Thanks to Ramiro Peñataro <penataro@ll.iac.es> |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@TAO/ChangeLogs/ChangeLog-2000b` at line 17187, Replace the corrupted
contributor name "Pe�ataro" with the correct Unicode spelling "Peñataro" in the
ChangeLog entry; locate the line containing the string "Pe�ataro" and update it
to "Peñataro" (preserving the surrounding punctuation and formatting) so the
contributor is correctly attributed.
| Fix for bug | ||
|
|
||
| http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=2224 | ||
| https://github.com/DOCGroup/bugzilla/issues/2224 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Verify that the new GitHub issue URL is accessible
# Test: Check if the URL returns a successful HTTP response
curl -s -o /dev/null -w "%{http_code}" "https://github.com/DOCGroup/bugzilla/issues/2224"Repository: DOCGroup/ACE_TAO
Length of output: 63
🏁 Script executed:
# Read the ChangeLog file around line 12079 to see context
sed -n '12070,12090p' TAO/ChangeLogs/ChangeLog-2005bRepository: DOCGroup/ACE_TAO
Length of output: 658
🏁 Script executed:
# Search for other references to bug 2224 in the codebase
rg "2224" --type-list | head -20Repository: DOCGroup/ACE_TAO
Length of output: 449
🏁 Script executed:
# Search for the old Bugzilla URL pattern in the ChangeLog file
rg "bugzilla.*2224" TAO/ChangeLogs/ChangeLog-2005bRepository: DOCGroup/ACE_TAO
Length of output: 116
🏁 Script executed:
# Check if there's a DOCGroup/bugzilla repository or if the issue exists
# Try alternative GitHub API calls to verify
curl -s "https://api.github.com/repos/DOCGroup/bugzilla" | head -20Repository: DOCGroup/ACE_TAO
Length of output: 339
🏁 Script executed:
# Search for any GitHub issue references in ChangeLog files to understand the migration pattern
rg "github.com/DOCGroup" TAO/ChangeLogs/ -m 5Repository: DOCGroup/ACE_TAO
Length of output: 7807
🌐 Web query:
DOCGroup ACE TAO Bugzilla migration GitHub issues
💡 Result:
DOCGroup's ACE and TAO projects, previously tracked bugs using Bugzilla at bugzilla.dre.vanderbilt.edu. The projects migrated to GitHub (https://github.com/DOCGroup/ACE_TAO), where issues are now managed via GitHub Issues. No evidence of a bulk migration of historical Bugzilla issues to GitHub; old Bugzilla references appear in NEWS files for fixes (e.g., Bugzilla #2368 in ACE/NEWS), but current bug reporting uses GitHub Issues, as seen in examples like issue #1060. The Bugzilla site is no longer active or referenced in recent official docs (post-2014). GitHub repo is active with ongoing issues, commits, and releases up to 2026.
Citations:
- 1: https://github.com/DOCGroup/ACE_TAO
- 2: https://github.com/DOCGroup
- 3: https://github.com/DOCGroup/ACE_TAO/blob/master/TAO/NEWS
- 4: https://github.com/DOCGroup/ACE_TAO/blob/master/ACE/NEWS
- 5: AIX 6.1 ACE building process not working #1060
- 6: https://sources.debian.org/src/ace/6.0.3%2Bdfsg-0.1/docs/ACE-bug-process.html
- 7: https://docs.huihoo.com/ace_tao/ACE-5.7+TAO-1.7+CIAO-0.7/ACE/usage-bugzilla.html
- 8: https://download.dre.vanderbilt.edu/
- 9: https://list.isis.vanderbilt.edu/pipermail/ace-announce/2024-May/000057.html
- 10: https://list.isis.vanderbilt.edu/pipermail/tao-users/2021-January/000244.html
Fix broken GitHub issue URL in changelog.
The URL https://github.com/DOCGroup/bugzilla/issues/2224 returns a 404 error and is inaccessible. The DOCGroup/bugzilla repository does not exist on GitHub. While the original Bugzilla migration converted legacy issue references to this format systematically, the target repository was never created. Either revert to the original Bugzilla URL (e.g., bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=2224), or update to point to the actual issue location if it exists elsewhere.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@TAO/ChangeLogs/ChangeLog-2005b` at line 12079, Replace the broken GitHub URL
"https://github.com/DOCGroup/bugzilla/issues/2224" in the changelog entry with a
valid target: either restore the original Bugzilla link (e.g.,
"bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=2224") or substitute the correct
current issue URL if it exists elsewhere; update the ChangeLog-2005b entry so
the link points to an accessible issue location and verify the URL resolves.
| </li> | ||
| <li> | ||
| This feature does not support the operation of the various server side 'Current' implementations (e.g. PICurrent, POACurrent, etc.). It should not be used by applications that require these to function correctly. See issues <a href="http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=3087">3087</a> and <a href="http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=3327">3327</a>. | ||
| This feature does not support the operation of the various server side 'Current' implementations (e.g. PICurrent, POACurrent, etc.). It should not be used by applications that require these to function correctly. See issues <a href="https://github.com/DOCGroup/bugzilla/issues/3087">3087</a> and <a href="https://github.com/DOCGroup/bugzilla/issues/3327">3327</a>. |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify that the GitHub repository and issues exist
echo "Checking if DOCGroup/bugzilla repository exists..."
gh repo view DOCGroup/bugzilla --json name,url
echo -e "\nChecking if issue 3087 exists..."
gh issue view 3087 --repo DOCGroup/bugzilla --json number,title,url
echo -e "\nChecking if issue 3327 exists..."
gh issue view 3327 --repo DOCGroup/bugzilla --json number,title,url
echo -e "\nVerifying HTTP accessibility of the URLs..."
curl -sI "https://github.com/DOCGroup/bugzilla/issues/3087" | head -n 1
curl -sI "https://github.com/DOCGroup/bugzilla/issues/3327" | head -n 1Repository: DOCGroup/ACE_TAO
Length of output: 562
🌐 Web query:
DOCGroup/bugzilla GitHub repository issues migrated
💡 Result:
No evidence found of issues from a DOCGroup/bugzilla GitHub repository being migrated. The DOCGroup GitHub organization (https://github.com/DOCGroup) exists and hosts a repository named "bugzilla" (last updated Mar 23, 2026), but it has 0 stars and no mention of issue migration in available sources. General Bugzilla-to-GitHub migrations exist for other projects (e.g., D programming language, LLVM, Coq, Doxygen), but none reference DOCGroup specifically. The bugzilla repo appears to be the Bugzilla source code mirror or fork, with no documented issue migration activity.
Citations:
- 1: https://github.com/DOCGroup
- 2: https://togithub.com/DOCGroup
- 3: https://github.com/bugzilla/bugzilla
- 4: https://github.com/bugzilla
- 5: https://github.com/orgs/bugzilla/repositories
- 6: https://www.theozimmermann.net/2017/10/bugzilla-to-github/
- 7: https://lists.llvm.org/pipermail/llvm-dev/2020-April/141290.html
- 8: GNOME Bugzilla migration to GitLab doxygen/doxygen#734
- 9: Bugzilla to GitHub Summary tianocore/edk2#6544
🌐 Web query:
ACE TAO Bugzilla GitHub migration issues 3087 3327
💡 Result:
ACE TAO used Bugzilla at bugzilla.dre.vanderbilt.edu for issue tracking before migrating to GitHub at DOCGroup/ACE_TAO. No evidence of a full issues migration like some projects (e.g., Tianocore), so old Bugzilla issues like 3087 and 3327 are not directly available on GitHub. These specific bugs are referenced as known issues in TAO's Custom Servant Dispatching (CSD) Thread Pool Strategy documentation, particularly noting that CSD TP does not support server-side 'Current' implementations (e.g., PICurrent, POACurrent). See Micro Focus OpenFusion TAO release notes for details. Bugzilla is now offline; current issues are at https://github.com/DOCGroup/ACE_TAO/issues. No specific details on bugs 3087/3327 contents found in available sources.
Citations:
- 1: https://www.microfocus.com/documentation/openfusion/tao_2110/211/docs/release/TAO/releasenotes/index.html
- 2: https://www.microfocus.com/documentation/openfusion/tao_2310/docs/HTML/docs/release/TAO/releasenotes/index.html
- 3: https://github.com/DOCGroup/ACE_TAO/issues
- 4: https://docs.huihoo.com/ace_tao/ACE-5.7+TAO-1.7+CIAO-0.7/ACE/usage-bugzilla.html
- 5: http://www.dre.vanderbilt.edu/~schmidt/ACE-future.html
- 6: https://www.dre.vanderbilt.edu/~schmidt/TAO-overview.html
- 7: https://download.dre.vanderbilt.edu/ACE+TAO-distribution/
🏁 Script executed:
curl -sI "https://github.com/DOCGroup/ACE_TAO/issues" | head -n 1Repository: DOCGroup/ACE_TAO
Length of output: 71
Fix broken issue links in documentation.
The migrated issue URLs return HTTP 404 errors and are inaccessible. The DOCGroup/bugzilla repository is a Bugzilla source code mirror, not an issue tracker, and the old Bugzilla issues (3087 and 3327) were not migrated to GitHub. Either:
- Point to https://github.com/DOCGroup/ACE_TAO/issues (main issue tracker), or
- Remove/clarify that these are historical Bugzilla references no longer accessible online.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@TAO/docs/releasenotes/index.html` at line 2404, The two hardcoded links to
DOCGroup/bugzilla issues (references "3087" and "3327") in the release notes are
broken; update the sentence that mentions these issues to either point to the
canonical tracker (replace both bugzilla URLs with
https://github.com/DOCGroup/ACE_TAO/issues) or explicitly mark them as
historical Bugzilla references not migrated to GitHub (e.g., “historical
Bugzilla issues 3087 and 3327, not available online”); modify the HTML fragment
containing the sentence about server side 'Current' implementations accordingly
so the links are fixed or clarified.
| 1238</A>and<A | ||
| HREF="http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=1239">1239</A> <p></li> | ||
| HREF="https://github.com/DOCGroup/bugzilla/issues/1239">1239</A> <p></li> |
There was a problem hiding this comment.
Fix invalid paragraph closure in the updated issue-link line.
On Line 58, the list item ends with <p></li>, which opens a new paragraph instead of closing the existing one, causing invalid HTML. Also, Line 57 is missing spaces around “and”.
Suggested patch
- 1238</A>and<A
- HREF="https://github.com/DOCGroup/bugzilla/issues/1239">1239</A> <p></li>
+ 1238</A> and <A
+ HREF="https://github.com/DOCGroup/bugzilla/issues/1239">1239</A></p></li>📝 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.
| 1238</A>and<A | |
| HREF="http://bugzilla.dre.vanderbilt.edu/show_bug.cgi?id=1239">1239</A> <p></li> | |
| HREF="https://github.com/DOCGroup/bugzilla/issues/1239">1239</A> <p></li> | |
| 1238</A> and <A | |
| HREF="https://github.com/DOCGroup/bugzilla/issues/1239">1239</A></p></li> |
🧰 Tools
🪛 HTMLHint (1.9.2)
[error] 58-58: Tag must be paired, missing: [
], start tag match failed [] on line 58.
(tag-pair)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@TAO/docs/releasenotes/orbcore.html` around lines 57 - 58, Fix the malformed
HTML in the issue-link list item: in the fragment containing "1238</A>and<A
HREF=\"https://github.com/DOCGroup/bugzilla/issues/1239\">1239</A> <p></li>",
add spaces around the word "and" (i.e., "1238</A> and <A...") and remove the
erroneous opening paragraph tag "<p>" so the item properly closes with "</li>"
only; update the line with the corrected spacing and closing tag in the same
snippet.
| ==================================================== | ||
|
|
||
| . None | ||
| . All issues from bugzilla.dre.vanderbilt.edu migrated to https://github.com/DOCGroup/bugzilla |
There was a problem hiding this comment.
Avoid rewriting historical release semantics in TAO/NEWS.
Line 4 changes a past release section from “None” to a repository-maintenance statement. Consider keeping the historical release note intact and moving this migration note to a dedicated repo-maintenance/changelog entry instead.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@TAO/NEWS` at line 4, Restore the original historical release note in TAO/NEWS
(revert the change that replaced the past release section value "None" with the
repository migration sentence) so the release semantics remain intact, and
instead add the migration information as a separate repo-maintenance or
changelog entry (e.g., create or append to a MAINTENANCE or CHANGELOG file)
describing the Bugzilla -> GitHub migration and linking to
https://github.com/DOCGroup/bugzilla; reference the TAO/NEWS section you
modified and the new MAINTENANCE/CHANGELOG file when making these edits.
1 participant
Summary by CodeRabbit