* Ronald G Minnich <rminnich@lanl.gov> [051201 04:59]: > this is the second report of flashrom not compiling. Can you send in > your error messages, and uname -a, and what distro this is?
I have written the following small patch for the flashrom utility Makefile. It will check for pciutils installation before compiling anything and print an explicit error message if pciutils is not found. This is much easier to parse than the half screen gcc errors. Issue 40. Commit? Stefan
Index: Makefile =================================================================== --- Makefile (revision 2116) +++ Makefile (working copy) @@ -17,7 +17,7 @@ sst39sf020.o m29f400bt.o w49f002u.o 82802ab.o msys_doc.o pm49fl004.o \ sst49lf040.o sst_fwhub.o layout.o lbtable.o flashchips.o flash_rom.o -all: dep $(PROGRAM) +all: pciutils dep $(PROGRAM) $(PROGRAM): $(OBJS) $(CC) -o $(PROGRAM) $(OBJS) $(LDFLAGS) @@ -32,7 +32,20 @@ dep: @$(CC) -MM *.c > .dependencies -.PHONY: all clean distclean dep +pciutils: + @echo; echo -n "Checking for pciutils... " + @$(shell ( echo "#include <pci/pci.h>"; \ + echo "struct pci_access *pacc;"; \ + echo "int main(int argc, char **argv)"; \ + echo "{ pacc = pci_alloc(); return 0; }"; ) > .test.c ) + @$(CC) $(CFLAGS) .test.c -o .test $(LDFLAGS) &>/dev/null && \ + echo "found." || ( echo "not found."; echo; \ + echo "Please install pciutils and pciutils-devel."; \ + echo "See README for more information."; echo; \ + rm -f .test.c .test; exit 1) + @rm -f .test.c .test +.PHONY: all clean distclean dep pciutils + -include .dependencies
-- LinuxBIOS mailing list LinuxBIOS@openbios.org http://www.openbios.org/mailman/listinfo/linuxbios