[Bug 559117] Review Request: lcdtest - display test pattern generator

bugzilla at redhat.com bugzilla at redhat.com
Wed Jan 27 19:27:04 UTC 2010


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=559117

--- Comment #5 from Eric Smith <eric at brouhaha.com> 2010-01-27 14:27:01 EST ---
Without adding the proposed "--debug=presub", SCons does output the commands
used for compiling and installing to standard out.  I just did a "rpmbuild -ba
lcdtest.spec >lcdtest.log 2>&1", and lcdtest.log contains:

gcc -o build/lcdtest.o -c -O2 -g -pipe -Wall -fexceptions -fstack-protector
--param=ssp-buffer-size=4 -m64 -mtune=generic -Wp,-D_FORTIFY_SOURCE=2
-DRELEASE=1.18 src/lcdtest.c
gcc -o build/lcdtest build/lcdtest.o -lSDL -lSDL_image -lSDL_ttf

That looks like the same kind of output I'd expect from make; do you still want
the --debug=presub added?

I'll make the other changes you're suggested, and post updated spec and SRPM
files for this and levmar once I've done another review.

Thanks!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.



More information about the package-review mailing list