After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 315068 - use of -nopad for data tracks
use of -nopad for data tracks
Status: RESOLVED FIXED
Product: nautilus-cd-burner
Classification: Deprecated
Component: cd-burner
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Nautilus CD Burner Maintainers
Nautilus CD Burner Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-09-01 20:36 UTC by William Jon McCann
Modified: 2005-10-07 21:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (1.44 KB, patch)
2005-09-02 16:18 UTC, William Jon McCann
committed Details | Review

Description William Jon McCann 2005-09-01 20:36:30 UTC
For data tracks (NAUTILUS_BURN_RECORDER_TRACK_TYPE_DATA) we use the -nopad
option to cdrecord.  It seems that in some cases this prevents a checksum
verification of the resulting disc.

https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=160191
http://www.sun.com/bigadmin/features/articles/burn_iso_images.html
http://www.troubleshooters.com/linux/coasterless.htm
https://bugzilla.ubuntu.com/show_bug.cgi?id=8499

Looks like it originates here:
http://cvs.gnome.org/viewcvs/nautilus-cd-burner/cd-recorder.c?hideattic=0&r1=1.26&r2=1.27

Bastien, do you know why we are using -nopad?
Comment 1 Bastien Nocera 2005-09-02 07:18:33 UTC
Don't remember on top of my head, i must admit. Hence the "TODO" on that line.
Comment 2 William Jon McCann 2005-09-02 13:28:10 UTC
Looks like Ross may have made the change.  I've cc'ed him.  Ross, will we break
anything that you are aware of if we change the -nopad option?
Comment 3 Ross Burton 2005-09-02 13:45:45 UTC
-nopad is the default so it should Just Work.  Adding random padding seems weird
unless someone can produce a definitive reason why though.
Comment 4 William Jon McCann 2005-09-02 16:18:21 UTC
Created attachment 51727 [details] [review]
patch

Seems that by default cdrecord writes in tao mode.  We should be able to
specify dao mode for all writes that start from the n-c-b frontend since we
will always have an ISO or CUE.

With this patch md5sums work reliably for me.
Comment 5 William Jon McCann 2005-10-07 21:28:43 UTC
2005-10-07  William Jon McCann  <mccann@jhu.edu>

	* nautilus-cd-burner.c (burn_cd): Always use
	NAUTILUS_BURN_RECORDER_WRITE_DISC_AT_ONCE for data
	written from burn:// the frontend since that
	can only be ISO and CUE files.  Fixes #315068.