Ticket #47570

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

Date d'ouverture: 2023-03-13 22:28 Dernière mise à jour: 2023-03-26 08:41

Rapporteur:
Propriétaire:
Type:
État:
Atteints
Composant:
Priorité:
5 - moyen
Sévérité:
5 - moyen
Résolution:
Fixed
Fichier:
2

Détails

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
Commentaire

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
  • Propriétaire Update from (Aucun) to cazfi
  • Résolution Update from Aucun to Accepted
  • Composant Update from Server to General
Commentaire

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
  • État Update from Ouvert to Atteints
  • Résolution Update from Accepted to Fixed

Modifier

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Connexion