Skip to content

ci(deploy): fix inconsistent hyperlinks on landing page#277

Closed
ArchUsr64 wants to merge 1 commit intoTexasInstruments:masterfrom
ArchUsr64:fix_landing_page
Closed

ci(deploy): fix inconsistent hyperlinks on landing page#277
ArchUsr64 wants to merge 1 commit intoTexasInstruments:masterfrom
ArchUsr64:fix_landing_page

Conversation

@ArchUsr64
Copy link
Contributor

linux/Foundational_Components/Graphics/index.html is being selected by the existing find command instead of the desired index.html.

The added depth check ensures that the index.html files from the subdirectories inside build are not selected as hyperlinks on the landing page.

Steps to reproduce:

  1. Go to the currently deployed landing page.
  2. Compare the hyperlink of processor-sdk-linux-AM62X vs processor-sdk-linux-AM62AX
  3. The hyperlink for 62AX correctly points to the SDK's index.html but 62X links to linux/Foundational_Components/Graphics/index.html instead

Expected Behaviour:

The hyperlinks for all platforms should be set to the index.html in build output's root directory.

Changes to index.html for the landing page:

Before:

<!DOCTYPE html>
<html lang="en">
    <head>
        <meta charset="utf-8">
        <title>PSDK Documentation Landing Page</title>
    </head>
    <body>
        <main>
            <h1>PSDK Documentation Landing Page</h1>
            <ul>
                <li>
                    <a href="processor-sdk-android-AM62PX/esd/docs/10_01_01/index.html">processor-sdk-android-AM62PX</a>
                </li>
                <li>
                    <a href="processor-sdk-android-AM62X/esd/docs/10_01_01/index.html">processor-sdk-android-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-buildroot-AM62LX/esd/docs/11_00_05_02/index.html">processor-sdk-buildroot-AM62LX</a>
                </li>
                <li>
                    <a href="processor-sdk-buildroot-AM62X/esd/docs/11_00_09_04/index.html">processor-sdk-buildroot-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM62LX/esd/docs/11_00_05_02/index.html">processor-sdk-debian-AM62LX</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM62PX/esd/docs/11_00_09_04/index.html">processor-sdk-debian-AM62PX</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM62X/esd/docs/11_00_09_04/index.html">processor-sdk-debian-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM64X/esd/docs/11_00_09_04/index.html">processor-sdk-debian-AM64X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM335X/esd/docs/09_03_05_02/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM335X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM437X/esd/docs/09_03_05_02/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM437X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM57X/esd/docs/08_02_01_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM57X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62AX/esd/docs/11_00_09_04/index.html">processor-sdk-linux-AM62AX</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62LX/esd/docs/11_00_05_02/index.html">processor-sdk-linux-AM62LX</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62PX/esd/docs/11_00_09_04/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM62PX</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62X/esd/docs/11_00_09_04/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM64X/esd/docs/11_00_09_04/index.html">processor-sdk-linux-AM64X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM65X/esd/docs/09_03_05_02/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM65X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM67/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM67</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM68/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM68</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM69/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM69</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-CORESDK/esd/docs/11_00/linux/index.html">processor-sdk-linux-CORESDK</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-DRA821A/esd/docs/07_03_00/index.html">processor-sdk-linux-DRA821A</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J7200/esd/docs/11_00/index.html">processor-sdk-linux-J7200</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J721E/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J721E</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J721S2/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J721S2</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J722S/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J722S</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J742S2/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J742S2</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J784S4/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J784S4</a>
                </li>
            </ul>
        </main>
    </body>
</html>

After:

<!DOCTYPE html>
<html lang="en">
    <head>
        <meta charset="utf-8">
        <title>PSDK Documentation Landing Page</title>
    </head>
    <body>
        <main>
            <h1>PSDK Documentation Landing Page</h1>
            <ul>
                <li>
                    <a href="processor-sdk-android-AM62PX/esd/docs/10_01_01/index.html">processor-sdk-android-AM62PX</a>
                </li>
                <li>
                    <a href="processor-sdk-android-AM62X/esd/docs/10_01_01/index.html">processor-sdk-android-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-buildroot-AM62LX/esd/docs/11_00_05_02/index.html">processor-sdk-buildroot-AM62LX</a>
                </li>
                <li>
                    <a href="processor-sdk-buildroot-AM62X/esd/docs/11_00_09_04/index.html">processor-sdk-buildroot-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM62LX/esd/docs/11_00_05_02/index.html">processor-sdk-debian-AM62LX</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM62PX/esd/docs/11_00_09_04/index.html">processor-sdk-debian-AM62PX</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM62X/esd/docs/11_00_09_04/index.html">processor-sdk-debian-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-debian-AM64X/esd/docs/11_00_09_04/index.html">processor-sdk-debian-AM64X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM335X/esd/docs/09_03_05_02/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM335X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM437X/esd/docs/09_03_05_02/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM437X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM57X/esd/docs/08_02_01_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM57X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62AX/esd/docs/11_00_09_04/index.html">processor-sdk-linux-AM62AX</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62LX/esd/docs/11_00_05_02/index.html">processor-sdk-linux-AM62LX</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62PX/esd/docs/11_00_09_04/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM62PX</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM62X/esd/docs/11_00_09_04/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM62X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM64X/esd/docs/11_00_09_04/index.html">processor-sdk-linux-AM64X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM65X/esd/docs/09_03_05_02/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM65X</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM67/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM67</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM68/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM68</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-AM69/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-AM69</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-CORESDK/esd/docs/11_00/linux/index.html">processor-sdk-linux-CORESDK</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-DRA821A/esd/docs/07_03_00/index.html">processor-sdk-linux-DRA821A</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J7200/esd/docs/11_00/index.html">processor-sdk-linux-J7200</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J721E/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J721E</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J721S2/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J721S2</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J722S/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J722S</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J742S2/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J742S2</a>
                </li>
                <li>
                    <a href="processor-sdk-linux-J784S4/esd/docs/11_00/linux/Foundational_Components/Graphics/index.html">processor-sdk-linux-J784S4</a>
                </li>
            </ul>
        </main>
    </body>
</html>

linux/Foundational_Components/Graphics/index.html was being selected by
the existing find command instead of the desired index.html.

The added depth check ensures that the index.html files from the
subdirectories inside build are not selected as hyperlinks on the
landing page.

Signed-off-by: Anshul Dalal <anshuld@ti.com>
@cshilwant
Copy link
Member

Looks like #276 by @StaticRocket handled the same

@StaticRocket
Copy link
Member

Yeah, duplicate of #276 . Thanks for taking the time to look through this though.

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.

4 participants