Update of bug #21115 (project freeciv):
Depends on: = bugs #21999
___
Reply to this item at:
http://gna.org/bugs/?21115
___
Message sent
Update of bug #21115 (project freeciv):
Depends on: = bugs #21992
___
Reply to this item at:
http://gna.org/bugs/?21115
___
Message sent
Follow-up Comment #18, bug #21115 (project freeciv):
And I've completed the review of my rebase, with the following results:
(1) was mitigated with patch #4451, but has other issues. Bug #21991 raised
for that, but not a dependency of this bug, as the issue is more general than
just
Update of bug #21115 (project freeciv):
Depends on: = bugs #21982
___
Reply to this item at:
http://gna.org/bugs/?21115
___
Message sent
Update of bug #21115 (project freeciv):
Depends on: = patch #4451
___
Reply to this item at:
http://gna.org/bugs/?21115
___
Message sent
Update of bug #21115 (project freeciv):
Planned Release: = 2.6.0
___
Reply to this item at:
http://gna.org/bugs/?21115
___
Message sent
Follow-up Comment #16, bug #21115 (project freeciv):
Recent discussion in patch #4401 highlights that there are still known
outstanding issues in the code, not yet raised as separate tickets.
___
Reply to this item at:
Follow-up Comment #17, bug #21115 (project freeciv):
I rebased an old WIP branch for this cleanup this weekend, and it seems that
vast majority of the issues I had previously identified have been addressed.
The only remaining parts in my branch were:
1) aicity:affected_unit_class(), which needs
Follow-up Comment #11, bug #21115 (project freeciv):
For effect reqs nreqs, I think the plan now is:
S2_4: nreqs must be used, patch ruleset sanity checking to disallow negated =
TRUE reqs
2.6: nreqs are to be removed (or only deprecated?). present = FALSE reqs must
be used.
What about S2_5?
Follow-up Comment #12, bug #21115 (project freeciv):
Oh, if we are still going to release 2.3.5, sanity check patch planned for
S2_4 should go to S2_3 too.
___
Reply to this item at:
http://gna.org/bugs/?21115
Follow-up Comment #13, bug #21115 (project freeciv):
S2_4: nreqs must be used, patch ruleset sanity checking to
disallow negated = TRUE reqs
Oh, if we are still going to release 2.3.5, sanity check patch
planned for S2_4 should go to S2_3 too.
Hm, I'm not wild about potentially disallowing
Follow-up Comment #14, bug #21115 (project freeciv):
Hm, I'm not wild about potentially disallowing rulesets on
stable branches.
True, that would be quite blatantly against datafile format freeze.
log_error() (limited to one client popup even if there's multiple negated
reqs) sounds sensible -
Follow-up Comment #15, bug #21115 (project freeciv):
Just to make sure we don't do duplicate work: have you done
any work to convert our rulesets?
No.
___
Reply to this item at:
http://gna.org/bugs/?21115
Update of bug #21115 (project freeciv):
Summary: [metaticket] Negated requirements for effects
('negated'=TRUE, 'present'=FALSE) do not work = [metaticket] Negated
requirements ('negated'=TRUE, 'present'=FALSE) do not work reliably
Update of bug #21115 (project freeciv):
Depends on: = bugs #21430
___
Reply to this item at:
http://gna.org/bugs/?21115
___
Message sent
Update of bug #21115 (project freeciv):
Depends on: = bugs #21432
___
Reply to this item at:
http://gna.org/bugs/?21115
___
Message sent
16 matches
Mail list logo
