Skip to content

Commit

Permalink
[WFLY-2047] org.jboss.as.connector module needs sun.jdk module depend…
Browse files Browse the repository at this point in the history 
…ency
  • Loading branch information
Jay SenSharma committed Sep 12, 2013
1 parent d6bf657 commit 88756dd
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions build/src/main/resources/modules/system/layers/base/org/jboss/as/connector/main/module.xml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@
</resources>

<dependencies>
<module name="sun.jdk"/>
<module name="javax.api"/>
<module name="javax.resource.api"/>
<module name="javax.transaction.api"/>
Expand Down

5 comments on commit 88756dd

@jesperpedersen
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the real question is why LDAP is leaking into JDBC

@jesperpedersen
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to be reverted

@bstansberry
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this needs to be reverted, someone needs to send a PR doing so.

@kabir
Copy link
Contributor

@kabir kabir commented on 88756dd Sep 12, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jesperpedersen don't bother with the PR, I reverted it already

@dmlloyd
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was this actually causing harm?

Please sign in to comment.