On Thu, Nov 29, 2012 at 11:26 AM, Daniel Manzke <[email protected]> wrote: > http://mavenhub.com/mvn/central/com.googlecode.rocoto/rocoto/4.0 > > com.googlecode.rocoto.configuration.ConfigurationReader > > That's why I wrote the upgrade to 5.0 was not done yet. ;)
Sorry to ask, but what is that googlecode repository? Looking in the Rocoto we have from 99soft there is no ConfigurationReader. What does this mean for Onami - does this class come from googlecode (please let us mind the IP) or do we remove the COnfigurationReader interface from Autobind? While IP in mind, I would love to see the project passing through Jenkins Cheers + thanks for your patience Christian > > > 2012/11/29 Christian Grobmeier <[email protected]> > >> Ah btw, do you know what the ConfigurationReader in the autobind class >> org.nnsoft.guice.autobind.configuration.PropertiesReader >> is? I failed to find this class >> >> On Thu, Nov 29, 2012 at 11:14 AM, Christian Grobmeier >> <[email protected]> wrote: >> > OK, thanks Daniel. >> > I already changed a bit and will commit it now. >> > >> > On Thu, Nov 29, 2012 at 11:08 AM, Daniel Manzke >> > <[email protected]> wrote: >> >> +1. Autobind was submitted from my side to nnsoft. The configuration >> module >> >> was not fully updated to the latest rocoto (onami-configuration) >> version. >> >> This is still an open point. >> >> >> >> >> >> >> >> >> >> 2012/11/29 Christian Grobmeier <[email protected]> >> >> >> >>> Hi, >> >>> >> >>> not sure how it worked before, but it seems >> >>> >> >>> Autobind: >> >>> org.nnsoft.guice.autobind.configuration.PropertiesReader >> >>> >> >>> makes use of: >> >>> >> >>> org.apache.onami.configuration.configuration.PropertiesURLReader >> >>> >> >>> Now PropertiesURLReader is package scoped. Even when I correct the >> >>> namespace for autobind it will fail. The solution is to make >> >>> PropertiesReader public. But I guess there was some reason to make it >> >>> package scoped in the past. >> >>> >> >>> Any objections if I would make this class public? >> >>> >> >>> Cheers >> >>> Christian >> >>> >> >>> -- >> >>> http://www.grobmeier.de >> >>> https://www.timeandbill.de >> >>> >> >> >> >> >> >> >> >> -- >> >> Viele Grüße/Best Regards >> >> >> >> Daniel Manzke >> > >> > >> > >> > -- >> > http://www.grobmeier.de >> > https://www.timeandbill.de >> >> >> >> -- >> http://www.grobmeier.de >> https://www.timeandbill.de >> > > > > -- > Viele Grüße/Best Regards > > Daniel Manzke -- http://www.grobmeier.de https://www.timeandbill.de
