9:09 a.m.
This PR fixes it, but Thomas might want to look it over.
https://github.com/jbossas/jboss-as/pull/3166
9:15 a.m.
Tnx,
I already sent info about broken build on windows to Thomas yesteday but he
didn't have time to check it yet.
http://lightning.mw.lab.eng.bos.redhat.com/jenkins/job/as7-master-windows...
--
tomaz
On Wed, Oct 3, 2012 at 3:09 PM, <ssilvert(a)redhat.com> wrote:
This PR fixes it, but Thomas might want to look it over.
https://github.com/jbossas/jboss-as/pull/3166
_______________________________________________
jboss-as7-dev mailing list
jboss-as7-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
9:35 a.m.
Should it not be... ?
moduleId.getName().replace('.', File.separator) + File.separator +
moduleId.getSlot()
--thomas
On 10/03/2012 03:09 PM, ssilvert(a)redhat.com wrote:
This PR fixes it, but Thomas might want to look it over.
https://github.com/jbossas/jboss-as/pull/3166
--
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Thomas Diesler
JBoss OSGi Lead
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
9:46 a.m.
On 10/3/2012 9:35 AM, Thomas Diesler wrote:
Should it not be... ?
moduleId.getName().replace('.', File.separator) + File.separator +
moduleId.getSlot()
--thomas
I don't think so. See my comments on the PR.
On 10/03/2012 03:09 PM, ssilvert(a)redhat.com wrote:
> This PR fixes it, but Thomas might want to look it over.
>
> https://github.com/jbossas/jboss-as/pull/3166
--
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
Thomas Diesler
JBoss OSGi Lead
JBoss, a division of Red Hat
xxxxxxxxxxxxxxxxxxxxxxxxxxxx
4220
days inactive
4220
days old
3 comments
3 participants
participants (3)
-
ssilvert＠redhat.com -
Thomas Diesler -
Tomaž Cerar