Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Multiple outdated Doxygens available on llvm.org #31759

Closed
llvmbot opened this issue Mar 24, 2017 · 6 comments
Closed

Multiple outdated Doxygens available on llvm.org #31759

llvmbot opened this issue Mar 24, 2017 · 6 comments
Assignees
@tlattner
Labels
bugzilla Issues migrated from bugzilla website

Comments

@llvmbot
Copy link
Collaborator

llvmbot commented Mar 24, 2017

Bugzilla Link 32412
Resolution FIXED
Resolved on Jun 30, 2017 23:27
Version unspecified
OS All
Reporter LLVM Bugzilla Contributor
CC @jyknight,@MatzeB,@TNorthover,@tlattner

Extended Description

There are at least two (very) outdated doxygen API docs available on llvm.org:

http://llvm.org/test-doxygen
http://llvm.org/docs/doxygen/html/

Especially the second one I find problematic, because searching API things on Google will end up on that page.

The correct Doxygen deployment is at http://llvm.org/doxygen/, which is a path that's Disallowed through robots.txt.

I think the outdated instances should be deleted (unless there is a reason to keep them around). And is there a reason to hide the real API Documentation from Google?
@llvmbot
Copy link
Collaborator Author

llvmbot commented Mar 24, 2017

assigned to @tlattner

@llvmbot
Copy link
Collaborator Author

llvmbot commented Apr 2, 2017

It appears that there are additional stray files to be found in the "real" documentation location, such as http://llvm.org/doxygen/Statistic_8h-source.html

@llvmbot
Copy link
Collaborator Author

llvmbot commented Jun 26, 2017

With the move of llvm.org, it appears the outdated doxygens have vanished. I guess by the letter of the word this bug could be closed.

There is however still the issue with googling for LLVM API. The new doxygen is hidden from the robots, and the old doxygens will fade from the index soon. I anticipate a wave of complaints :)

Is there still a reason to keep the new doxygen hidden?

@jyknight
Copy link
Member

A redirect should also be setup from the old URLs to the new URL.

@tlattner
Copy link
Contributor

tlattner commented Jul 1, 2017

Redirect added

@tlattner
Copy link
Contributor

tlattner commented Jul 1, 2017

Removed doxygen from disallow in robots.txt in r306949

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tlattner tlattner

Labels
bugzilla Issues migrated from bugzilla website
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jyknight @tlattner @llvmbot