I had a look at this package. Here are my notes:

! Not a blocker but a suggestion: The recommended buildroot values are given in
   http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

* Source0 is wrong.

* rpmlint says:
   dc3dd.x86_64: W: wrong-file-end-of-line-encoding /usr/share/doc/dc3dd-6.12.2/README
You can probably correct this with
   sed 's|\r||' README > README.tmp
   touch -r README README.tmp
   mv -f README.tmp README
in %prep.

* Please remove the binary .gmo files in %prep.

* It looks like the files NEWS, Options_Reference.txt, Sample_Commands.txt should go to %doc.

* Each package must consistently use macros. Please make use of the %{name} macro.

* Parallel make must be supported whenever possible. If it is not supported, this should be noted in the SPEC file as a comment.

* The file lib/getdate.c is GPLv2+. That makes the license GPLv2+ and GPLv3+.

? What is the directory build-aux and the executable files in the root of the source tree for?

Spec URL: http://maxamillion.fedorapeople.org/dc3dd.spec
SRPM URL: http://maxamillion.fedorapeople.org/dc3dd-6.12.2-2.src.rpm

I've cleaned up everything listed above. I fixed the licensing, removed the .gmo binaries, fixed source0 (sorry for that one, I based this spec off another spec of mine and mixed that up), added the doc items you listed, and fixed EOF encoding for the current and two added files to the package.

Thank you very much for your feedback and I look forward to further review.

Thanks for the update.

* To clean up the SPEC file you can use a for-loop like: 

for file in README Sample_Commands.txt Options_Reference.txt; do
   sed 's|\r||' $file > $file.tmp
   touch -r $file $file.tmp
   mv -f $file.tmp $file
done

* You probably missed this question I asked:

> ? What is the directory build-aux and the executable files in the root of the
source tree for?

The reason I asked was: If these files run tests to check whether the application is compiled properly, you will need to make a %check section and run these tests.
On the other hand, if these scripts are useful for people who want to use/develop dc3dd, you might want to put them in %{_datadir}/%{name} or inside %doc.

Spec URL: http://maxamillion.fedorapeople.org/dc3dd.spec
SRPM URL: http://maxamillion.fedorapeople.org/dc3dd-6.12.2-3.src.rpm

I applied the cleanup and I inspected the source of a number of the files in the build-aux directory and they appear to just be a cludge of python scripts that provide work arounds for different build environments and/or compilers that don't support certain things.

Thank you again for your comments, looking forward to further review.

So, those files are irrelevant. Then I think everything is fine now. You're welcome

----------------------------------------
This package (dc3dd) is APPROVED by oget
----------------------------------------

New Package CVS Request
=======================
Package Name: dc3dd
Short Description: Patched version of GNU dd for use in computer forensics
Owners: maxamillion
Branches: F-9 F-10 EL-4 EL-5
InitialCC:

cvs done.

dc3dd-6.12.2-3.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/dc3dd-6.12.2-3.fc9

dc3dd-6.12.2-3.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/dc3dd-6.12.2-3.fc10

dc3dd-6.12.2-3.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

dc3dd-6.12.2-3.fc10 has been pushed to the Fedora 10 stable repository.  If problems still persist, please make note of it in this bug report.

Package Change Request
======================
Package Name: dc3dd
New Branches: epel7
Owners: maxamillion rebus

Hello SCM team,
plase can you add epel7 branch for the dc3dd package?
Thank you
Michal Ambroz

Git done (by process-git-requests).

dc3dd-7.1.614-8.el7 has been submitted as an update for Fedora EPEL 7.
https://admin.fedoraproject.org/updates/dc3dd-7.1.614-8.el7

dc3dd-7.1.614-8.el7 has been pushed to the Fedora EPEL 7 stable repository.