Bug 148318

Summary: Recent versions of p5-DBIx-Class break audio/squeezeboxserver
Product: Ports & Packages Reporter: Mark Knight <markk>
Component: Individual Port(s)Assignee: Brooks Davis <brooks>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch-squeezeboxserver-autocommit.patch none

Description Mark Knight 2010-07-02 14:40:07 UTC
With p5-DBIx-Class-0.8123_1 squeezeboxserver fails to scan my music library correctly.

The scanner stops after one song and /var/log/squeezeboxserver/scanner.log has this error:

 Slim::Schema::Storage::throw_exception (82) Error: DBI Exception: DBD::mysql::db be gin_work failed: Already in a transaction

Backing out to p5-DBIx-Class-0.08120 fixes the issue.
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2010-07-02 20:28:43 UTC
Responsible Changed
From-To: freebsd-ports-bugs->brooks

Fix synopsis and assign.
Comment 2 Christopher Key 2010-07-05 23:59:49 UTC
  SqueezeBox Server turns off auto commit in the scanner to speed up 
scan times.  The problem appears to be that turning off auto commit 
after connecting confuses the transaction nesting logic in DBIx::Class.  
The reason that it only affects 0.08121 and greater is that, according 
the the changelog "Cascading delete/update are now wrapped in a 
transaction for atomicity".

I think that this is technically a bug in DBIx::Class, although whether 
its a documentation or implementation bug (i.e. whether changing 
AutoCommit is prohibited or not) I don't know.

The attached patch solves the problem by specifying AutoCommit when 
connecting to the database.
Comment 3 dfilter service freebsd_committer freebsd_triage 2010-07-08 00:22:34 UTC
brooks      2010-07-07 23:22:25 UTC

  FreeBSD ports repository

  Modified files:
    audio/squeezeboxserver Makefile 
  Added files:
    audio/squeezeboxserver/files patch-dbix-class-bug.diff 
  Log:
  Another attempt to fixing interactions with p5-DBIx-Class.  A few people
  report that this works.
  
  PR:             148318
  Feature safe:   yes
  Submitted by:   Mark Knight
  
  Revision  Changes    Path
  1.76      +1 -1      ports/audio/squeezeboxserver/Makefile
  1.1       +37 -0     ports/audio/squeezeboxserver/files/patch-dbix-class-bug.diff (new)
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 4 Mark Knight 2010-07-08 12:31:46 UTC
  This works a treat, thank you.  Please feel free to close this bug.

-- 
Mark A. R. Knightmarkk@knigma.org
Tel: +44 7880 556751http://www.knigma.org/
Comment 5 Brooks Davis freebsd_committer freebsd_triage 2010-07-08 17:08:14 UTC
State Changed
From-To: open->closed

Committed and reported to work.