Incidencia #47570

silent noisy warning "Did not find a cm solution..."

Abrir Fecha: 2023-03-13 22:28 Última actualización: 2023-03-26 08:41

Informador:
Propietario:
Tipo:
Estado:
Cerrado
Componente:
Prioridad:
5 - Medium
Gravedad:
5 - Medium
Resolución:
Fixed
Fichero:
2

Details

3.0.6

Attached is patch to completely remove warning, except in the case #ifdef FREECIV_TESMATIC (as someone cares enough about it to #ifdef )

As noted in #44438 , it is not a bug , it is normal , and it generates lot of pollution in logs of large games like WW1 scenario (even if it happens much less than 1% of time)

Ticket History (3/9 Histories)

2023-03-13 22:28 Updated by: alain_bkr
  • New Ticket "silent noisy warning "Did not find a cm solution..."" created
2023-03-13 22:29 Updated by: alain_bkr
  • File S3_0_silent_warning_cm_did_not_find_a_cm_solution.patch (File ID: 11836) is attached
2023-03-13 22:39 Updated by: cazfi
Comentario

I wouldn't remove it completely. How about setting logging level to verbose? (Or if you insist, have it always on debug level only, as currently in testmatic use) If someone wants to find out what's wrong, they should be able to have it logged.

2023-03-13 22:40 Updated by: alain_bkr
  • File S3_0_silent_warning_cm_did_not_find_a_cm_solution.patch (File ID: 11836) is deleted
2023-03-13 22:46 Updated by: alain_bkr
2023-03-24 14:38 Updated by: cazfi
  • Propietario Update from (Ninguno) to cazfi
  • Resolución Update from Ninguno to Accepted
  • Componente Update from Server to General
Comentario

While factually correct that nothing prevents it happening, it's not completely harmless either - cm aborts with non-optimal solution at that point. This message has already made us to increase the cap as its frequency has indicated that cm should try harder.

Anyway, the patches look good to me, functionally. We would prefer 'git format-patch' output for the metadata they contain, though. Most importantly it would give credit directly to you, as the commit author. But I can also use these, and just give the credit in the commit message.

2023-03-26 08:41 Updated by: cazfi
  • Estado Update from Open to Cerrado
  • Resolución Update from Accepted to Fixed

Editar

Please login to add comment to this ticket » Entrar