MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004649ardourbugspublic2012-01-21 15:442012-01-24 08:18
Reporteranrug 
Assigned Topaul 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version2.8.13Fixed in Version 
Summary0004649: TOC export writes PERFORMER field only when present
DescriptionArdour (2.0 and 3.0) does write the CD text field "PERFORMER" only when it's actually actually present. This may sound reasonable, and for cue sheets it probably is, but for reasons I'm not familiar with, the cdrdao source code wants this field filled in for all tracks and in the global section, so Ardour should probably adjust here, because cdrdao will refuse to burn a CD otherwise.

The same is actually true for the TITLE field, cdrdao handles those two fields differently then any other CD text field. But with TITLE it's not an issue because that will always be filled in by Ardour at the moment.

I've included a patch for review.
Additional InformationThe way CD text fields are written into the R-W subcode streams in the lead-in area of an audio CD requires that each CD text field (TITLE, PERFORMER, SONGWRITER, etc.) is present for all tracks and globally if it's used in at least one track or globally. So in many cases zero-length strings will be stored (a single terminating zero that is), that's how CD text works. And cdrdao hands over that restriction over to it's users. But CD text *does not* require a specific field like PERFORMER to be present at all, that restriction comes from cdrdao.
TagsNo tags attached.
Attached Filespatch file icon toc_performer.patch [^] (1,296 bytes) 2012-01-21 15:44 [Show Content]

- Relationships

-  Notes
(0012612)
cth103 (administrator)
2012-01-22 02:43

Applied to 3.0 SVN 11299. Thanks!
(0012639)
paul (administrator)
2012-01-24 08:18

i believe that this is no longer an issue for either TOC or CUE.

- Issue History
Date Modified Username Field Change
2012-01-21 15:44 anrug New Issue
2012-01-21 15:44 anrug File Added: toc_performer.patch
2012-01-22 02:20 cth103 cost => 0.00
2012-01-22 02:20 cth103 Target Version => 3.0-beta3
2012-01-22 02:43 cth103 Note Added: 0012612
2012-01-22 02:43 cth103 Target Version 3.0-beta3 => 2.8.13
2012-01-24 08:18 paul Note Added: 0012639
2012-01-24 08:18 paul Status new => resolved
2012-01-24 08:18 paul Resolution open => fixed
2012-01-24 08:18 paul Assigned To => paul


Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker