Uploaded image for project: 'Forrest (Retired)'
  1. Forrest (Retired)
  2. FOR-509

Deprecated skin designated to replace non-existant skin

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Later
    • 0.6
    • 0.7
    • Skins (general issues)
    • None

    Description

      In forrest.build.xml, a skin-aliasing occurs to replace skins which no longer exist by an alternative, and a warning is issued. Similarly, a warning is issued when a deprecated skin is chosen. It can happen that a deprecated skin is designated to replace a no-longer-existing skin: this should be avoided to prevent confusion. Maintenance should ensure that only non-deprecated (i.e. supported) skins are designated replacements each time the build file is updated or skins are deprecated.

      Attachments

        Activity

          rgardler Ross Gardler added a comment -
          Scheduled for 0.7 release
          rgardler Ross Gardler added a comment - Scheduled for 0.7 release
          williamstw Tim Williams added a comment -
          Unless I've oversimplified this, here's the change for this. "crust"
          now goes to "pelt".

             Skin aliasing for backwards compatability
                 0.5 => 0.6
                 krysalis-site => ... warn about future removal
                 forrest-site => ... warn about future removal
                 crust => pelt
                 forrest-css => pelt
                 avalon-tigris => tigris
                 tigris-style => tigris

          I wasn't sure whether that version info should change or not?

          --tim
          williamstw Tim Williams added a comment - Unless I've oversimplified this, here's the change for this. "crust" now goes to "pelt".    Skin aliasing for backwards compatability        0.5 => 0.6        krysalis-site => ... warn about future removal        forrest-site => ... warn about future removal        crust => pelt        forrest-css => pelt        avalon-tigris => tigris        tigris-style => tigris I wasn't sure whether that version info should change or not? --tim
          crossley David Crossley added a comment -
          This is an over-simplification. The "crust" skin was never released. The closest skin to it is "krysalis-site" which is deprecated. So the current messages are correct. We cannot force people to change their skin.
          crossley David Crossley added a comment - This is an over-simplification. The "crust" skin was never released. The closest skin to it is "krysalis-site" which is deprecated. So the current messages are correct. We cannot force people to change their skin.

          People

            Unassigned Unassigned
            suitable Maurice Lanselle
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: