Principalmente o break
está em lugar errado, e sem condição alguma, e se vai sair um esse comando não tem porque complicar, e duplicar a condição. E o código pode ser muito mais simples sem duplicações.
Ele só precisa pedir o dado dentro do laço, verificar se é negativo, e se for deve encerrar, caso contrário continua o fluxo imprimindo e e repetindo. Não tem porque fazer diferente do que o enunciado pede, é uma questão de interpretação de texto.
A pergunta fala que tem que imprimir o negativo, mas o enunciado é ambíguo quanto a isso, geralmente não é o desejado.
Não fiz a verificação se o dado foi digitado errado, se algo que não seja um número for digitado a aplicação quebrará.
while True:
n1 = int(input())
if n1 < 0:
break
print(n1)
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Se realmente deseja que o negativo deve ser impresso é só uma questão de ordem que as coisas devem ser executadas, então se o if
estabelece a saída do laço e quer que seja impresso antes de sair, é só colocar o print()
antes do if
.
while True:
n1 = int(input())
print(n1)
if n1 < 0:
break
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.