On Fri, May 28, 2010 at 7:12 PM, Geoff Lywood <[email protected]> wrote: > Sure. A new patch is attached. > > However: > > $ make bin/e1000e.rom.licence > [LICENCE] bin/e1000e.rom.licence > Unable to determine licence because the following files are missing a licence > declaration: > arch/i386/interface/pcbios/keepsan.c arch/i386/prefix/undiloader.S > config/local/console.h config/local/general.h config/local/ioapi.h > config/local/nap.h config/local/serial.h config/local/timer.h > config/local/umalloc.h core/serial_console.c > make: *** [bin/e1000e.rom.licence] Error 1 > > It's even worse with respect to the EFI headers: > > Unable to determine licence because the following files are missing a licence > declaration: > arch/x86_64/include/bits/byteswap.h arch/x86_64/include/bits/compiler.h > arch/x86_64/include/bits/endian.h arch/x86_64/include/bits/errfile.h > arch/x86_64/include/bits/io.h arch/x86_64/include/bits/nap.h > arch/x86_64/include/bits/smbios.h arch/x86_64/include/bits/stdint.h > arch/x86_64/include/bits/timer.h arch/x86_64/include/bits/uaccess.h > arch/x86_64/include/bits/umalloc.h arch/x86_64/include/limits.h > config/defaults/efi.h config/local/console.h config/local/general.h > config/local/ioapi.h config/local/nap.h config/local/serial.h > config/local/timer.h config/local/umalloc.h core/serial_console.c > include/gpxe/efi/Base.h include/gpxe/efi/Guid/PcAnsi.h > include/gpxe/efi/Guid/SmBios.h include/gpxe/efi/IndustryStandard/PeImage.h > include/gpxe/efi/Pi/PiBootMode.h include/gpxe/efi/Pi/PiDependency.h > include/gpxe/efi/Pi/PiDxeCis.h include/gpxe/efi/Pi/PiFirmwareFile.h > include/gpxe/efi/Pi/PiFirmwareVolume.h include/gpxe/efi/Pi/PiHob.h > include/gpxe/efi/Pi/PiMultiPhase.h include/gpxe/efi/PiDxe.h > include/gpxe/efi/ProcessorBind.h include/gpxe/efi/Protocol/Cpu.h > include/gpxe/efi/Protocol/CpuIo.h include/gpxe/efi/Protocol/DebugSupport.h > include/gpxe/efi/Protocol/DevicePath.h > include/gpxe/efi/Protocol/PciRootBridgeIo.h > include/gpxe/efi/Protocol/SimpleTextIn.h > include/gpxe/efi/Protocol/SimpleTextOut.h include/gpxe/efi/Uefi.h > include/gpxe/efi/Uefi/UefiBaseType.h include/gpxe/efi/Uefi/UefiGpt.h > include/gpxe/efi/Uefi/UefiInternalFormRepresentation.h > include/gpxe/efi/Uefi/UefiMultiPhase.h include/gpxe/efi/Uefi/UefiPxe.h > include/gpxe/efi/Uefi/UefiSpec.h include/gpxe/efi/X64/ProcessorBind.h > include/gpxe/efi/efi.h
Yes. That is why it is important for new files to use FILE_LICENCE(). It is more difficult adding it in later and it hasn't been done, as you saw. Stefan _______________________________________________ gPXE-devel mailing list [email protected] http://etherboot.org/mailman/listinfo/gpxe-devel
