[jboss-jira] [JBoss JIRA] Commented: (AS7-1239) Don't WARN about files in WEB-INF/classes/META-INF

[jaikiran pai (JIRA)]

    [ https://issues.jboss.org/browse/AS7-1239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12613259#comment-12613259 ] 

jaikiran pai commented on AS7-1239:
-----------------------------------

IRC discussion:

{quote}
[10:31:40] <Jaikiran> stuartdouglas: on a different unrelated note, did you happen to see a thread on seam-dev about a seam config issue in AS 7 CR1?
[10:32:08] <Jaikiran> it later turned out to be a discussion on the support for WEB-INF/classes/META-INF as a metadata location for beans.xml
[10:33:10] <stuartdouglas> looking now
[10:33:34] <stuartdouglas> actually I did read that one
[10:34:09] <Jaikiran> currently we log a WARN for any files other than persistence.xml in that location and I guess we ignore beans.xml in that location during parsing too
[10:34:34] <stuartdouglas> I wonder if I do in the beans.xml processor...
[10:34:58] <stuartdouglas> ah, I do
[10:35:27] <Jaikiran> i think we should just get rid of that WARN message altogether
[10:35:56] <stuartdouglas> It was David that originally wanted us to ignore WEB-INF/classes/META-INF
[10:36:47] <Jaikiran> it's now 1 spec (JPA) + CDI (to be more compatible with other servers) which reads metadata from this location, so i think that WARN will no longer be appropriate
[10:36:47] <stuartdouglas> by getting rid of the warn do you mean stop ignoring META-INF?
[10:37:00] <stuartdouglas> CDI doesn't though
[10:37:08] <Jaikiran> i mean, let whichever tech wants to find files there, find them
[10:37:20] <stuartdouglas> I am fine with that
[10:37:30] <stuartdouglas> but you will have to talk to dmlloyd
[10:37:39] <Jaikiran> dmlloyd: ^ when you are up
[10:38:41] <Jaikiran> dmlloyd: here's the thread for reference http://lists.jboss.org/pipermail/seam-dev/2011-July/003953.html
....
[15:53:45] <dmlloyd> incorrect, I don't actually care one way or another about WEB-INF/classes/META-INF
[15:54:18] <dmlloyd> also: good morning!
...
[16:03:29] <Jaikiran> dmlloyd: ok, i'll get rid of that WARN then
{quote}

> Don't WARN about files in WEB-INF/classes/META-INF
> --------------------------------------------------
>
>                 Key: AS7-1239
>                 URL: https://issues.jboss.org/browse/AS7-1239
>             Project: Application Server 7
>          Issue Type: Task
>          Components: EE
>    Affects Versions: 7.0.0.CR1
>            Reporter: jaikiran pai
>            Assignee: jaikiran pai
>
> Currently we log a WARN message if we find any files, other than persistence.xml, in .war/WEB-INF/classes/META-INF folder. We need to get rid of the WARN since Weld looks for beans.xml in that location, although CDI doesn't mandate it, but as per the folks on seam-dev mailing list, all application servers look for beans.xml in this location (including previous version on AS). From what I understand, even AS7 looks for beans.xml in this location currently. So this WARN message is misleading and unnecessary. Let's just allow the individual techs to pick up files from this location, as they wish and get rid of this WARN.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira