Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-1824

Discontinue use of the META-INF directory for configuration files

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: Not version specific
    • Component/s: Development
    • Labels:
      None
    • Similar issues:
      KULRICE-6496Remove css files from kns directory that we are not using
      KULRICE-3959database-impex build directories
      KULRICE-3588Improve rice attachment directory structure
      KULRICE-9320Determine out-of-the-box configuration for EclipseLink dynamic weaving support for lazy loading
      KULRICE-11752Add more ingest resources files to include previous and split scripts
      KULRICE-230Investigate Commons Configuration
      KULRICE-10200Allow full min file path to be specified when using manual theme configuration
      KULRICE-8533Generate minified files in a min directory instead of css
      KULRICE-10140ViewTheme has inconsistent use of trailing file separator from getThemeDirectory
      KULRICE-9739Improve ease of configuration of JPA for a module that is using KRAD

      Description

      I think I started this convention with the original plugin layout using META-INF/workflow.xml (regretfully so). That directory has special meaning within Java and so I think we should stop using it for configuration files and go back and fix the places where we do use it.

      We could allow it to be used still for plugins moving forward to support backward compatibility if desired but I think it should at the very least be deprecated there.

        Activity

        Hide
        Nate Johnson (Inactive) added a comment -

        Are you just saying for the workflow.xml file? I think this is the default for all non-java resources according to maven.

        Show
        Nate Johnson (Inactive) added a comment - Are you just saying for the workflow.xml file? I think this is the default for all non-java resources according to maven.
        Hide
        Eric Westfall added a comment -

        Maven wants all non-java resources in META-INF by default? I wasn't aware of that. I was thinking more along the lines of all of the configuration files we have that we put in that directory.

        Show
        Eric Westfall added a comment - Maven wants all non-java resources in META-INF by default? I wasn't aware of that. I was thinking more along the lines of all of the configuration files we have that we put in that directory.
        Hide
        Nate Johnson (Inactive) added a comment -

        Maybe maven just assume the resources directory, not resources/META-INF... I can't remember right off hand. Were you also thinking of removing the resources directory, or just META-INF where there are no specs?

        Show
        Nate Johnson (Inactive) added a comment - Maybe maven just assume the resources directory, not resources/META-INF... I can't remember right off hand. Were you also thinking of removing the resources directory, or just META-INF where there are no specs?
        Hide
        Eric Westfall added a comment -

        Maven assumes that resources are under <module-name>/src/main/resources so I think we want to keep that but I don't think it prescribes any meaning to the META-INF directory.

        Show
        Eric Westfall added a comment - Maven assumes that resources are under <module-name>/src/main/resources so I think we want to keep that but I don't think it prescribes any meaning to the META-INF directory.
        Hide
        Nate Johnson (Inactive) added a comment -

        That works for me.

        Show
        Nate Johnson (Inactive) added a comment - That works for me.
        Hide
        Jessica Coltrin (Inactive) added a comment -

        no longer an issue.

        Show
        Jessica Coltrin (Inactive) added a comment - no longer an issue.

          People

          • Assignee:
            Unassigned
            Reporter:
            Eric Westfall
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 2 hours
              2h
              Remaining:
              Remaining Estimate - 2 hours
              2h
              Logged:
              Time Spent - Not Specified
              Not Specified

                Structure Helper Panel