Re: [security-dev] Dependency of idm/impl on config module

On 07/02/13 17:01, Shane Bryzak wrote: > Oops, I didn't mean to commit that before discussing with you, it must > have snuck in with some other changes that I had to make to help out > Bruno with one of the Aerogear examples. In any case, we do need to > remove the dependency on the config module from idm - I'm ok with a > dependency going the other way though. Ok, but why we need to remove it? Is it some xml related conflict? I have same problem today again. I fixed it locally in my env, but i won't commit it until we sort this out.

Thanks, Marek > > On 07/02/13 05:23, Marek Posolda wrote: >> Hi Shane, >> >> I found today that I am unable to build idm/impl module due to >> commented >> dependency on "config" module, which is coming from your commit >> "e1494891b5f906fb5e54511141ee7d661e4941b9 minor dependency fix" . To >> fix >> this, I partially reverted it and uncomment the config module again. >> See >> commit >> https://github.com/mposolda/picketlink/commit/4ff92be99e0ed9ed8898814bddd... >> >> in PR https://github.com/picketlink/picketlink/pull/50 (It's also >> adding >> sorting support in FileBasedIdentityStore). >> >> Wdyt? Are you able to build idm/impl without dependency on config >> module? On my side, I am using "mvn clean install" to build whole >> project and with commented config module, it's failing for me. >> >> Thanks, >> Marek >> _______________________________________________ >> security-dev mailing list >> security-dev(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/security-dev > _______________________________________________ > security-dev mailing list > security-dev(a)lists.jboss.org > https://lists.jboss.org/mailman/listinfo/security-dev