I'm also against the merging of the repositories in a single one. I think the current solution provides the best outcome for individual committers and patchers.
As Seb points out, the current problem is the lack of release process and automation. I've mentioned before that maybe we need some kind of build manager/release manager that releases/builds Castle as a whole, from the individual repositories. -1 for using submodules and/or subtree. Instead we should have a go at automating as much as possible and try OW to deal with inter-dependencies. Cheers John ________________________________ From: Mauricio Scheffer <[email protected]> To: [email protected] Sent: Tuesday, 21 June 2011 6:46 AM Subject: Re: Problem with Windsor null reference exception +1 to this. I wouldn't put the code back together in a single repository. Instead, try creating a super-project / super-repository explicitly for integration purposes, with git submodules (or subtree merging http://progit.org/book/ch6-7.html ) pointing to the individual project repositories. -- Mauricio On Mon, Jun 20, 2011 at 5:02 PM, Sebastien Lambla <[email protected]> wrote: I'd say that one of the issues here is breaking changes being introduced across versions. Even if you reintegrated those components, things would still break for users in the same unexpected way. > >Isn't it an integration testing automation problem rather than a code split >one? > > >-----Original Message----- >From: [email protected] >[mailto:[email protected]] On Behalf Of hammett >Sent: 20 June 2011 17:03 >To: [email protected] >Subject: Re: Problem with Windsor null reference exception > >I guess this testifies that the separation of codebases brought up some pain >for users. I'd say that we need to bring them back together.. > >On Mon, Jun 20, 2011 at 1:45 AM, Berke Sokhan <[email protected]> wrote: >> I also tried to gather all Castle projects to one solution to see what >> is failing but w/o success. >> >> Got all sources from git/castleproject source and >> git/haf/castle.facilities.nhibernate built them. But NH facility uses >> Windsor 2.5.1, and I dont want to downgrade to it. I already spent too >> much time try to integrate the new nh facility. I also noticed old >> NHIntegration facility uses an old version of Castle. >> >> So may I ask you guys, to achive session-per-call/tx-per-call >> nhibernate WCF application server (that is not using IIS or >> httpbinding), excluding nhfacilities and including wcf facility... >> >> Can you guide me to a best practice? >> >> >> >> 2011/6/18 Henrik Feldt <[email protected]> >>> >>> Hmm, I don’t know how to fix this properly... >>> >>> >>> >>> The only work-around so far for me is to downgrade Windsor to 2.5.1. >>> That works. I suggested that some of the improvements from v3 might >>> be back-ported to change some of the bits in Windsor and make it go away. >>> >>> >>> >>> Henrik >>> >>> >>> >>> From: [email protected] >>> [mailto:[email protected]] On Behalf Of Berke >>> Sokhan >>> Sent: den 16 juni 2011 14:11 >>> To: [email protected] >>> >>> Subject: Re: Problem with Windsor null reference exception >>> >>> >>> >>> My stack looks same with Henrik: >>> >>> at System.Collections.ObjectModel.Collection`1.Add(T item) >>> at >>> Castle.Facilities.AutoTx.TransactionalComponentInspector.AddIntercept >>> or(ComponentModel >>> model) in >>> d:\Builds\Castle.Transactions-beta\src\Castle.Facilities.AutoTx\Trans >>> actionalComponentInspector.cs:line >>> 78 >>> at >>> Castle.Facilities.AutoTx.TransactionalComponentInspector.ProcessModel >>> (IKernel >>> kernel, ComponentModel model) in >>> d:\Builds\Castle.Transactions-beta\src\Castle.Facilities.AutoTx\Trans >>> actionalComponentInspector.cs:line >>> 46 >>> at >>> Castle.MicroKernel.ModelBuilder.DefaultComponentModelBuilder.BuildMod >>> el(String key, Type service, Type classType, IDictionary >>> extendedProperties) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\MicroKernel\ModelBuilde >>> r\DefaultComponentModelBuilder.cs:line >>> 67 >>> at >>> Castle.MicroKernel.Registration.ComponentRegistration`1.Castle.MicroK >>> ernel.Registration.IRegistration.Register(IKernel >>> kernel) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\MicroKernel\Registratio >>> n\ComponentRegistration.cs:line >>> 904 >>> at Castle.MicroKernel.DefaultKernel.Register(IRegistration[] >>> registrations) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\MicroKernel\DefaultKern >>> el.cs:line >>> 595 >>> at >>> Castle.MicroKernel.Registration.BasedOnDescriptor.TryRegister(Type >>> type, IKernel kernel) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\MicroKernel\Registratio >>> n\BasedOnDescriptor.cs:line >>> 208 >>> at >>> Castle.MicroKernel.Registration.FromDescriptor.Castle.MicroKernel.Reg >>> istration.IRegistration.Register(IKernel >>> kernel) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\MicroKernel\Registratio > >>> n\FromDescriptor.cs:line >>> 96 >>> at >>> Castle.MicroKernel.Registration.BasedOnDescriptor.Castle.MicroKernel. >>> Registration.IRegistration.Register(IKernel >>> kernel) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\MicroKernel\Registratio >>> n\BasedOnDescriptor.cs:line >>> 325 >>> at Castle.MicroKernel.DefaultKernel.Register(IRegistration[] >>> registrations) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\MicroKernel\DefaultKern >>> el.cs:line >>> 595 >>> at Castle.Windsor.WindsorContainer.Register(IRegistration[] >>> registrations) in >>> e:\OSS.Code\Castle.Windsor\src\Castle.Windsor\Windsor\WindsorContaine >>> r.cs:line >>> 689 >>> at PayFlex.Vpos.Server.Container.Bootstrapper.Initialize() in >>> D:\SVN >>> Repositories\iPayVPOS\trunk\PayFlex.Vpos\PayFlex.Vpos.Server.Containe >>> r\Bootstrapper.cs:line >>> 32 >>> at PayFlex.Vpos.Server.Application.Program.Main() in D:\SVN >>> Repositories\iPayVPOS\trunk\PayFlex.Vpos\PayFlex.Vpos.Server.Applicat >>> ion\Program.cs:line >>> 19 >>> at System.AppDomain._nExecuteAssembly(RuntimeAssembly assembly, >>> String[] args) >>> at System.AppDomain.ExecuteAssembly(String assemblyFile, Evidence >>> assemblySecurity, String[] args) >>> at >>> Microsoft.VisualStudio.HostingProcess.HostProc.RunUsersAssembly() >>> at System.Threading.ThreadHelper.ThreadStart_Context(Object state) >>> at System.Threading.ExecutionContext.Run(ExecutionContext >>> executionContext, ContextCallback callback, Object state, Boolean >>> ignoreSyncCtx) >>> at System.Threading.ExecutionContext.Run(ExecutionContext >>> executionContext, ContextCallback callback, Object state) >>> at System.Threading.ThreadHelper.ThreadStart() >>> >>> I am using Henrik's version of Transactions/AutoTx lib coming with >>> new NH Facility. >>> >>> And I get this exception when registering some POCO service class >>> which have "Transactional" attribute on it (removing attr. prevents >>> exception, but nh facility need it to open injected session). >>> >>> My early impression was AutoTx has some bug in inspecting attributes, >>> maybe a null check before adding to interceptors collection obviously ... >>> >>> Will look at source and try to point exact location. >>> >>> >>> >>> 2011/6/16 Krzysztof Koźmic <[email protected]> >>> >>> If you can reproduce it in a way that points to some reasonable bug >>> in Windsor or elsewhere... >>> >>> I looked at it when Henrik originally reported it but other than >>> "that's bizarre it can't be happening" I didn't find any good explanation >>> for that. >>> >>> Krzysztof >>> >>> On 16/06/2011 8:47 AM, Berke Sokhan wrote: >>> >>> Any news on this one? I am getting it too. >>> >>> Without a resolution to this, whole new Castle.Facilities.AutoTx / >>> Castle.Services.Transactions / Castle.Facilities.NHibernate trilogy >>> become useless :( >>> >>> 2011/5/6 Henrik <[email protected]> >>> >>> I was wondering if I could get some help debugging this exception? >>> >>> TearDown : System.NullReferenceException : Object reference not set >>> to an instance of an object. >>> at System.Collections.ObjectModel.Collection`1.Add(T item) >>> >>> at >>> >>> Castle.Facilities.AutoTx.TxComponentInspector.AddInterceptor(Componen >>> tModel >>> model) >>> at >>> Castle.Facilities.AutoTx.TxComponentInspector.ProcessModel(IKernel >>> kernel, ComponentModel model) >>> >>> at >>> >>> Castle.MicroKernel.ModelBuilder.DefaultComponentModelBuilder.BuildMod >>> el(String key, Type service, Type classType, IDictionary >>> extendedProperties) >>> at >>> >>> Castle.MicroKernel.Registration.ComponentRegistration`1.Castle.MicroK >>> ernel.Registration.IRegistration.Register(IKernel >>> kernel) >>> >>> at Castle.MicroKernel.DefaultKernel.Register(IRegistration[] >>> registrations) >>> at Castle.Windsor.WindsorContainer.Register(IRegistration[] >>> registrations) >>> NHibernateFacility_ValidationError_OnSave.cs(75,0): at >>> Castle.Facilities.NHibernate.Tests.Container..ctor() >>> NHibernateFacility_ValidationError_OnSave.cs(48,0): at >>> >>> Castle.Facilities.NHibernate.Tests.NHibernateFacility_ValidationError >>> _OnSave.SetUp() >>> --TearDown >>> NHibernateFacility_ValidationError_OnSave.cs(54,0): at >>> >>> Castle.Facilities.NHibernate.Tests.NHibernateFacility_ValidationError >>> _OnSave.TearDown() >>> >>> A repro is here: >>> >>> https://github.com/haf/Castle.Facilities.NHibernate/commit/bc2257cd4e >>> 0992f1d2bde70de14da383910927b9 >>> >>> Windsor 2.5.2, 2.5.3 are affected. Not 2.5.1.xxxx >>> >>> -- >>> You received this message because you are subscribed to the Google >>> Groups "Castle Project Development List" group. >>> To post to this group, send email to >>> [email protected]. >>> To unsubscribe from this group, send email to >>> [email protected]. >>> For more options, visit this group at >>> http://groups.google.com/group/castle-project-devel?hl=en. >>> >>> >>> -- >>> Berke SOKHAN. >>> >>> http://twitter.com/berkesokhan >>> http://blog.berkesokhan.com >>> http://www.birliktegelistir.com/editors.aspx >>> >>> -- >>> >>> You received this message because you are subscribed to the Google >>> Groups "Castle Project Development List" group. >>> To post to this group, send email to >>> [email protected]. >>> To unsubscribe from this group, send email to >>> [email protected]. >>> For more options, visit this group at >>> http://groups.google.com/group/castle-project-devel?hl=en. >>> >>> >>> >>> -- >>> You received this message because you are subscribed to the Google >>> Groups "Castle Project Development List" group. >>> To post to this group, send email to >>> [email protected]. >>> To unsubscribe from this group, send email to >>> [email protected]. >>> For more options, visit this group at >>> http://groups.google.com/group/castle-project-devel?hl=en. >>> >>> >>> -- >>> Berke SOKHAN. >>> >>> http://twitter.com/berkesokhan >>> http://blog.berkesokhan.com >>> http://www.birliktegelistir.com/editors.aspx >>> >>> -- >>> You received this message because you are subscribed to the Google >>> Groups "Castle Project Development List" group. >>> To post to this group, send email to >>> [email protected]. >>> To unsubscribe from this group, send email to >>> [email protected]. >>> For more options, visit this group at >>> http://groups.google.com/group/castle-project-devel?hl=en. >>> >>> -- >>> You received this message because you are subscribed to the Google >>> Groups "Castle Project Development List" group. >>> To post to this group, send email to >>> [email protected]. >>> To unsubscribe from this group, send email to >>> [email protected]. >>> For more options, visit this group at >>> http://groups.google.com/group/castle-project-devel?hl=en. >> >> >> >> -- >> Berke SOKHAN. >> >> http://twitter.com/berkesokhan >> http://blog.berkesokhan.com >> http://www.birliktegelistir.com/editors.aspx >> >> -- >> You received this message because you are subscribed to the Google >> Groups "Castle Project Development List" group. >> To post to this group, send email to [email protected]. >> To unsubscribe from this group, send email to >> [email protected]. >> For more options, visit this group at >> http://groups.google.com/group/castle-project-devel?hl=en. >> > > > >-- >Cheers, >hammett >http://hammett.castleproject.org/ > >-- >You received this message because you are subscribed to the Google Groups >"Castle Project Development List" group. >To post to this group, send email to [email protected]. >To unsubscribe from this group, send email to >[email protected]. >For more options, visit this group at >http://groups.google.com/group/castle-project-devel?hl=en. > >-- > >You received this message because you are subscribed to the Google Groups >"Castle Project Development List" group. >To post to this group, send email to [email protected]. >To unsubscribe from this group, send email to >[email protected]. >For more options, visit this group at >http://groups.google.com/group/castle-project-devel?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Castle Project Development List" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en. -- You received this message because you are subscribed to the Google Groups "Castle Project Development List" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/castle-project-devel?hl=en.
