-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am 25.07.2012 13:18, wrote Peter Suter:
> On Wed, Jul 25, 2012 at 11:28 AM, Eduard-Cristian Stefan
> <[email protected]> wrote:
>> Can you please test the attached patch? It works for me with the current 
>> trunk
>> of tags plugin, but I can't test the old data migration.
> 
> Thanks.
> Same here. With this patch I can get it to start and it works. I have
> no old data to migrate so I haven't tested that either.
> 
> I'm still not sure if this is a bug in the plugin or in Trac or in
> both. In http://trac.edgewall.org/ticket/10451#comment:1 the idea was
> mentioned that postgresql backend might need automatic rollback on
> error. A workaround was added to the TagsPlugin and the idea was
> dropped / changed because of that. But it still doesn't work...

Obviously the workaround doesn't cope with with non-graceful handling of
failing transactions due to bad SELECT statements.

Thanks for the patch. May I carry it over to the appropriate ticket
against TagsPlugin [1] and use it to patch current trunk, please?

I have no test setup to check the actual upgrade with PostgreSQL too, so
I'd be grateful, if someone could follow-up with his/her experience,
preferably reporting to the ticket. Thanks for taking care.

Sincerely,

Steffen Hoffmann

[1] http://trac-hacks.org/ticket/9521
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAlAQc4wACgkQ31DJeiZFuHck5QCfY7spn7EiAzntcRpPFJ9ifI43
+E4AoNX/CDzMvyaYVFRSn8GKDJa89pHN
=XMSw
-----END PGP SIGNATURE-----

-- 
You received this message because you are subscribed to the Google Groups "Trac 
Development" 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/trac-dev?hl=en.

Reply via email to