Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-882

Update site to adhere to a `baseurl` property

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.5.0
    • Component/s: site
    • Labels:
      None

      Description

      Noticed while working CALCITE-866, the trick to deploy a Jekyll site to a subdirectory (instead of the root of some vhost) doesn't work because Calcite has hard-coded anchors at "/".

      We can work around this by using

      {{site.baseurl}}
      

      There's a description on how to do this at https://jekyllrb.com/docs/github-pages/#project-page-url-structure. Would be convenient to be able to deploy at any location, not just the root of some vhost.

        Issue Links

          Activity

          Hide
          julianhyde Julian Hyde added a comment -

          Sounds like a great idea. Note we already have sourceRoot and apiRoot, although they are not managed very systematically.

          Show
          julianhyde Julian Hyde added a comment - Sounds like a great idea. Note we already have sourceRoot and apiRoot, although they are not managed very systematically.
          Hide
          elserj Josh Elser added a comment -

          Pull request that appears to do the trick for me. Successfully deployed the site to my people.a.o space. Lifted the apidocs and sourceroot variables to _config.yml too to control URLs in one place instead of scattered throughout the templates.

          Show
          elserj Josh Elser added a comment - Pull request that appears to do the trick for me. Successfully deployed the site to my people.a.o space. Lifted the apidocs and sourceroot variables to _config.yml too to control URLs in one place instead of scattered throughout the templates.
          Show
          julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/ad4bdca3 and deployed to http://calcite.incubator.apache.org/ . Thanks for the patch, Josh Elser !
          Hide
          elserj Josh Elser added a comment -

          Great! Thanks for the commit!

          Show
          elserj Josh Elser added a comment - Great! Thanks for the commit!
          Hide
          jcamachorodriguez Jesus Camacho Rodriguez added a comment -

          Resolved in release 1.5.0 (2015-11-10)

          Show
          jcamachorodriguez Jesus Camacho Rodriguez added a comment - Resolved in release 1.5.0 (2015-11-10)

            People

            • Assignee:
              elserj Josh Elser
              Reporter:
              elserj Josh Elser
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development