Project

General

Profile

Actions
Copy link

Bug #1400

closed

ruleload.c: correct error message for no default specialist

Added by Alexandr Ignatiev 2 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Marko Lindqvist
Category:
Server
Target version:
3.1.6
Start date:
05/17/2025
Due date:
% Done:

0%

Estimated time:

Description

See patches in #1375, needs to be improved in stable branches. Default specialist must not have any reqs, not only min_size.

Files

Download all files
cities_ruleset_specialists_errors.patch (1.55 KB) cities_ruleset_specialists_errors.patch Alexandr Ignatiev, 05/21/2025 12:16 AM
3_2-cities_ruleset_specialists_errors.patch (1.51 KB) 3_2-cities_ruleset_specialists_errors.patch patch for 3.(1,2) Alexandr Ignatiev, 05/21/2025 10:04 PM

Related issues 1 (0 open — 1 closed)

Blocks Feature #1375: Introduce superspecialistsClosedMarko Lindqvist05/08/2025

Actions
Actions
Copy link
 #1

Updated by Alexandr Ignatiev 2 months ago

Patch ready, likely applied to all branches.

Another thing needed correction, with no specialists at all section list became null and we got sigsegv insto warning.

Actions
Copy link
 #2

Updated by Marko Lindqvist 2 months ago

Actions
Copy link
 #3

Updated by Marko Lindqvist 2 months ago

  • Status changed from New to In Review
  • Assignee set to Marko Lindqvist
Actions
Copy link
 #5

Updated by Alexandr Ignatiev 2 months ago

there is no ruuleload.c in older versions, added similar patch for ruleset.c

Actions
Copy link
 #6

Updated by Marko Lindqvist 2 months ago

Alexandr Ignatiev wrote in #note-5:

there is no ruuleload.c in older versions, added similar patch for ruleset.c

Yeah, but 'git cherry-pick' from branch to branch (I applied the patch to local main-work branch, and cherry-picked to older) was smart enough to know about the file rename.

Actions
Copy link
 #7

Updated by Marko Lindqvist 2 months ago

  • Status changed from In Review to Closed
Actions
Copy link

Also available in: Atom PDF