The circuit doesn't work because a PNP transistor supplied from 12V can't be controlled by 5V logic without a level converter. A source driver like ULN2981 could be used to perform both, level conversion and switching. Also 2803 could basically act as level converter in a suitable dimensioned circuit.
You are apparently intending a per-digit current limiting with R5 - R8. Of course, 100k is an absurd dimensioning for it, please consider ohms law and the specified display opreation current. But you should provide per segment current limiting, to achieve a constant segment current.