Bug 142049 - [PATCH:] make audio/cdparanoia work with new cam/ata code
Summary: [PATCH:] make audio/cdparanoia work with new cam/ata code
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Markus Brueffer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-26 19:40 UTC by Juergen Lock
Modified: 2010-12-04 17:40 UTC (History)
0 users

See Also:


Attachments
file.diff (704 bytes, patch)
2009-12-26 19:40 UTC, Juergen Lock
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Juergen Lock 2009-12-26 19:40:01 UTC
	As noted on -current some time ago, cdparanoia seems to be
	currently broken for optical drives using the new cam/ata code
	i.e. connected via ahci(4)/siis(4):
		http://lists.freebsd.org/pipermail/freebsd-current/2009-August/010359.html

Fix: The patch works for me, only drives conneted on a SAS controller
	(i.e. not native (S)ATA) may still be broken, but that's probably
	still rare so this is at least much better than it was.  (The patch
	can go in e.g. /usr/ports/audio/cdparanoia/files/patch-camata )
How-To-Repeat: 	Make sure the drive is using the new cam/ata code (this thread
	may be useful:
		http://lists.freebsd.org/pipermail/freebsd-current/2009-December/014224.html
	), then:

	Insert audio cd, make sure cdparanoia can access the drive's
	/dev/cdX and /dev/passY devices (you can find the corresponding
	passY device by running
		camcontrol devlist
	), then run:
		cdparanoia -d /dev/cdX 1
	to try and rip the first track.

	If you have mplayer built with cdparanoia support you can
	also test this way:
		mplayer -cache 8192 -cdrom-device /dev/cdX cdda://
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2009-12-26 19:40:17 UTC
Responsible Changed
From-To: freebsd-ports-bugs->markus

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 rnejdl 2010-11-29 14:28:14 UTC

I wanted to check on this PR as it seems to still be highly needed in
order to make cdparanoia work with what is becoming the standard for CD
roms on FreebSD. 

Without the patch, I get  

Verifying CDDA command
set... 
 Could not find any audio tracks on this disk.

Unable to open
disc.
Exit 1 

With the patch, updated slightly for the updated version
of cdparanoia's line numbers, it works perfectly.  

Sincerely,
Rusty
Nejdl 
Comment 3 dfilter service freebsd_committer freebsd_triage 2010-12-04 17:39:19 UTC
nox         2010-12-04 17:39:15 UTC

  FreeBSD ports repository

  Modified files:
    audio/cdparanoia     Makefile 
    audio/cdparanoia/files patch-interface-scsi_interface.c 
  Log:
  - Add fix for the new CAM/ATA code.
  - Bump PORTREVISION.
  
  PR:             ports/142049
  Submitted by:   nox (self)
  Approved by:    maintainer timeout (11 months)
  
  Revision  Changes    Path
  1.17      +1 -1      ports/audio/cdparanoia/Makefile
  1.3       +19 -20    ports/audio/cdparanoia/files/patch-interface-scsi_interface.c
_______________________________________________
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 Juergen Lock freebsd_committer freebsd_triage 2010-12-04 17:39:48 UTC
State Changed
From-To: open->closed

Committed. Thanks!