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=652623 --- Comment #3 from Ville-Pekka Vainio <vpivaini@xxxxxxxxxxxxxx> 2011-01-13 09:29:24 EST --- I'd like to discuss a couple of things before going through a "checklist" for the review. (I know very little about erlang, but a few observations anyway.) - I'd like the build system to be even more verbose. For erl compiles it only outputs "Compiled src/bitcask_merge_worker.erl" or so. Could it also output the ecc call? (I'm assuming it's using ecc.) For C compiles it outputs something like "$CC -c $CFLAGS $DRV_CFLAGS c_src/bitcask_nifs.c -o c_src/bitcask_nifs.o" and then you need to figure out yourself from the rest of the output what those environment variables expand to. I'd like to see the exact commands it's running. However, if this is a shortcoming of rebar, then I won't consider this as a blocker for the review. - Another thing about the build. When building the C code, I'm fairly certain your package isn't passing the Fedora build options to cc. I suggest using the following line in %build, or something similar: "CFLAGS=$RPM_OPT_FLAGS rebar compile -v" -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review