On Tuesday, July 3, 2012 10:56:54 AM UTC+2, Martin Jansa wrote:
>
> On Thursday, June 28, 2012 2:18:12 AM UTC+2, jberezanski wrote:
>>
>> Hi,
>>
>> I'm afraid the published packages are missing several assemblies,
>> Castle.Transactions.nupkg does not even contain any assembly at all.
>>
>> Coincidentally, I noticed this issue yesterday when I tried to build
>> nuget packages for our internal nuget repo from github sources.
>>
>> In rakefile.rb, the nuspec :tx_nuspec task passes wrong pattern to
>> nuspec_copy (the pattern "*Transaction.{dll,xml,pdb}" matched the old
>> Castle.Services.Transaction name, but the assembly is now named
>> Castle.Transaction*s*). Moreover, new assemblies (Castle.IO,
>> Castle.Transactions.IO and their dependency - Mono.Posix) should also be
>> included.
>>
>> As a side note, good job with NLog removal, please continue :)
>>
>> It would also be great if IO and especially Unix/Posix stuff were
>> optional (separate packages) and not required by AutoTx. I understand,
>> however, that it would probably not be trivial.
>>
>> Best regards,
>>
>> Jakub Berezanski
>>
>
> I've sent pull request to fix those:
> https://github.com/haf/Castle.Transactions/pull/11
>
> I'm preparing another one to improve tx logging and there is problem
> with Transaction(Fork = true) which is caused by changes in Castle.Core
> DynamicProxy. I'll send testcase and bug report to Castle.Core directly.
>
Reported here http://issues.castleproject.org/issue/DYNPROXY-173
Improved logging and added test for this issue:
https://github.com/jansa/Castle.Transactions/commits/fixes
Cheers,
--
You received this message because you are subscribed to the Google Groups
"Castle Project Users" group.
To view this discussion on the web visit
https://groups.google.com/d/msg/castle-project-users/-/SpsHpJXS334J.
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-users?hl=en.