Hope someone can help me because i'm getting very frustrated!!

i'm putting a classic range rover v8 3.9 efi in my C reg defender
The engine is in the car and i have done as followed with the original range rover loom:

With the white plug i have put:
orange and black wire to permanent live
big brown wire to permanent live
grey and white wire goes to ignition live
purple and white to fuel pump
2 big black earths to the right head
earth straps front and rear
pump primes up with ignition but doesn't seem to pump when cranking engine
and i don't seem to have a spark

I'd also like to check that i have my flow and return for fuel the right way to the tank

if anyone can help i will be very grateful!!
Thanks norman

coil is a seperate loom with ignition live to + and awire to - and + from dizzy amp,but a wire from efi loom needs to go to - side of coil to let ecu know engines been cranked