URL:
http://gna.org/patch/?4682
Summary: Rule set sanity check: the unit type flag Spy
requires that Diplomat is set
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 02 May 2014 03:59:34 PM UTC
Category: None
URL:
http://gna.org/patch/?4683
Summary: Make the Marines and AttFromNonNative flags
action neutral
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 02 May 2014 04:03:49 PM UTC
Category: None
URL:
http://gna.org/bugs/?22005
Summary: Path finding problem when establishing trade route
from a Transport
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 02 May 2014 04:16:17 PM UTC
Category: None
Update of patch #4671 (project freeciv):
Status: Ready For Test = In Progress
___
Follow-up Comment #6:
Thanks for the deeper expansion of my suggestion
Thanks for making me consider it again. I
Update of bug #22005 (project freeciv):
Depends on: = patch #4682
___
Reply to this item at:
http://gna.org/bugs/?22005
___
Message sent
Update of bug #22005 (project freeciv):
Status: Ready For Test = In Progress
___
Follow-up Comment #2:
I object :-)
I take this an objection to patch #4683 and Alternative 3 in patch #4671 as
well.
Follow-up Comment #3, patch #4682 (project freeciv):
2.4 version. Doesn't break compatibility since ok can be set to false in 2.4
without aborting ruleset loading.
(file #20633)
___
Additional Item Attachment:
File name:
URL:
http://gna.org/bugs/?22011
Summary: Extra ' in Spy must be Diplomat error message
Project: Freeciv
Submitted by: sveinung
Submitted on: Sun 04 May 2014 12:53:56 PM UTC
Category: None
Severity: 3 -
Update of patch #4682 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #22011 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #1, bug #22180 (project freeciv):
Thank you for your bug report, Dennis. It would be nice if you could give some
additional information.
How often do it crash? Unless it always crashes it would be nice if you could
upload a savegame that will crash when pressing turn done. If
Update of bug #22005 (project freeciv):
Status: In Progress = Duplicate
Open/Closed:Open = Closed
___
Follow-up Comment #4:
This was fixed by
Follow-up Comment #11, bug #22180 (project freeciv):
Thank you for the extra information, Dennis.
_One other detail...the same build works fine with an existing/loaded game._
Just a suspicion: Do each new game look different or do you have the same
starting position on the same map etc? If they
URL:
http://gna.org/patch/?4800
Summary: Gtk2 client: Change the way spy action probabilities
are displayed.
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 13 Jun 2014 12:34:56 PM UTC
Category: client-gtk-2.0
URL:
http://gna.org/patch/?4801
Summary: Gtk3 client: Change the way spy action probabilities
are displayed.
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 13 Jun 2014 12:38:25 PM UTC
Category: client-gtk-3.0
Update of patch #4800 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4801 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/bugs/?22204
Summary: Qt 5 headers no longer found on Debian testing
Project: Freeciv
Submitted by: sveinung
Submitted on: Sun 15 Jun 2014 01:50:42 PM UTC
Category: bootstrap
Severity: 3
Update of bug #22204 (project freeciv):
Status: In Progress = Ready For Test
Release: TRUNK, 2.5, 2.4 = TRUNK, 2.5
Planned Release: = 2.6.0, 2.5.0
Update of bug #22204 (project freeciv):
Status: Ready For Test = In Progress
___
Reply to this item at:
http://gna.org/bugs/?22204
___
Message sent
Update of bug #22204 (project freeciv):
Status: Ready For Test = In Progress
Assigned to: cazfi = sveinung
___
Follow-up Comment #6:
New version. Will
Update of bug #22204 (project freeciv):
Status: In Progress = Ready For Test
___
Follow-up Comment #7:
Seems like we had a collision.
_We already use AC_CANONICAL_HOST to determine host system (not
Follow-up Comment #8, bug #22204 (project freeciv):
Cover all possibilities in merged patch.
(file #21222)
___
Additional Item Attachment:
File name: 0001-Also-search-multiarch-folders-when-looking-for-Qt-5-.patch
Size:3 KB
Follow-up Comment #10, bug #22204 (project freeciv):
Thanks for testing and for confirming that I didn't misunderstood the
reference.
If requested I could do a version for 2.4 too.
_Minor nit: the comments have multiarch_tuble rather than
multiarch_tuple._
Fixed.
(file #21228)
URL:
http://gna.org/patch/?4886
Summary: Network protocol: Send each effect rule in a single
packet.
Project: Freeciv
Submitted by: sveinung
Submitted on: Tue 01 Jul 2014 12:33:31 PM UTC
Category: general
URL:
http://gna.org/bugs/?22255
Summary: Nitpick: The generated unit helptext says that a
unit is ubribable twice
Project: Freeciv
Submitted by: sveinung
Submitted on: Tue 01 Jul 2014 02:53:06 PM UTC
Category: rulesets
Update of bug #22204 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Follow-up Comment #13:
_I remain curious
URL:
http://gna.org/patch/?4893
Summary: Make requirement types that check unit state
possible
Project: Freeciv
Submitted by: sveinung
Submitted on: Wed 02 Jul 2014 08:47:39 PM UTC
Category: general
URL:
http://gna.org/patch/?4894
Summary: UnitState requirement type with the test
TransportDependent
Project: Freeciv
Submitted by: sveinung
Submitted on: Wed 02 Jul 2014 08:51:11 PM UTC
Category: None
Update of patch #4894 (project freeciv):
Depends on: = patch #4893
___
Reply to this item at:
http://gna.org/patch/?4894
___
Message sent
Follow-up Comment #1, patch #4894 (project freeciv):
Add a patch that applies on top of this meant for testing. It will enable
sabotage or targeted sabotage in Experimental depending on if the Spy/Diplomat
is on a ship or not.
(file #21248)
Update of patch #4683 (project freeciv):
Status: In Progress = Wont Do
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #2, patch #4893 (project freeciv):
_This looks like it is only an API change._
Correct.
_Would it make sense to add a preface in is_req_active that sets
target_unittype in the event that target_unit != NULL target_unittype ==
NULL?_
Great idea. Added.
(file #21257)
Update of patch #4886 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4893 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/patch/?4902
Summary: Unit veteran level requirment type
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 04 Jul 2014 02:16:07 PM UTC
Category: None
Priority: 5 - Normal
Update of bug #22255 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/patch/?4903
Summary: Coding style: maxTileUnits = max_tile_units
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 04 Jul 2014 05:14:29 PM UTC
Category: None
Priority: 3 - Low
URL:
http://gna.org/bugs/?22286
Summary: Ruledit won't compile because of renamed field
Project: Freeciv
Submitted by: sveinung
Submitted on: Sun 06 Jul 2014 06:59:08 PM UTC
Category: freeciv-ruledit
Update of bug #22286 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #2, bug #22286 (project freeciv):
Note that all done here was to make it compile again. Calculated values are
now written. Since they are stored as an int the calculated values are now
rounded. Should I have commented out the support for writing tech cost in
stead of doing what
URL:
http://gna.org/bugs/?22288
Summary: Ruledit stores (rounded) calculated tech cost
Project: Freeciv
Submitted by: sveinung
Submitted on: Sun 06 Jul 2014 09:46:41 PM UTC
Category: freeciv-ruledit
Update of bug #22288 (project freeciv):
Status:None = Duplicate
Open/Closed:Open = Closed
___
Follow-up Comment #1:
Dupe of bug #22287
Follow-up Comment #1, bug #22287 (project freeciv):
The attached patch fixes the issue for me.
___
Reply to this item at:
http://gna.org/bugs/?22287
___
Message sent via/by Gna!
Update of patch #4903 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4918 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #22349 (project freeciv):
Status:None = Ready For Test
Assigned to:None = sveinung
Release: = TRUNK
Planned Release:
URL:
http://gna.org/patch/?4959
Summary: Make the stub ruleset more minimal by removing all
action enablers.
Project: Freeciv
Submitted by: sveinung
Submitted on: Tue 15 Jul 2014 12:55:56 PM UTC
Category: rulesets
URL:
http://gna.org/patch/?4962
Summary: Remove the now unused fields REQ_RANGE and REQ_TYPE
from packets.def
Project: Freeciv
Submitted by: sveinung
Submitted on: Wed 16 Jul 2014 09:59:03 PM UTC
Category: None
Update of bug #22349 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #1, patch #4959 (project freeciv):
Fix name of commented example action enabler.
(file #21452)
___
Additional Item Attachment:
File name: 0002-Make-the-stub-ruleset-more-minimal-by-removing-all-a.patch
Size:1 KB
Update of patch #4962 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4959 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/bugs/?22377
Summary: Alien ruleset: Savegames with cities are broken
Project: Freeciv
Submitted by: sveinung
Submitted on: Tue 22 Jul 2014 01:51:01 AM UTC
Category: rulesets
Severity: 3
Update of bug #22377 (project freeciv):
Priority: 5 - Normal = 9 - Immediate
___
Follow-up Comment #1:
This fixes data corruption. It is therefore a critical patch. I will therefore
commit it now.
Update of bug #22377 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/bugs/?22381
Summary: Inconsistent nativity limits on source tile for
regular attacks
Project: Freeciv
Submitted by: sveinung
Submitted on: Tue 22 Jul 2014 12:18:46 PM UTC
Category: None
Follow-up Comment #1, bug #22378 (project freeciv):
A check that a specialist rule name is usable in a save game is added in patch
#4972.
_Savegame format freeze probably makes actual fix for this impossible in
stable branches_
What if we replace unacceptable characters in the rulename before
URL:
http://gna.org/patch/?4977
Summary: Add meta knowledge for the building (improvement)
requirement type.
Project: Freeciv
Submitted by: sveinung
Submitted on: Wed 23 Jul 2014 12:51:40 PM UTC
Category: None
Follow-up Comment #1, patch #4977 (project freeciv):
The spell checked version
(file #21519)
___
Additional Item Attachment:
File name: 0001-Add-meta-knowledge-for-the-building-improvement-requ.patch
Size:2 KB
Update of patch #4972 (project freeciv):
Planned Release:2.6.0, 2.5.0 = 2.6.0, 2.5.0, 2.4.3
___
Additional Item Attachment:
File name: 2.4-Check-that-ruleset-supplied-specialist-names-won-t-b.patch
Size:3 KB
URL:
http://gna.org/patch/?4980
Summary: Centralize adding probabilities to the action button
text
Project: Freeciv
Submitted by: sveinung
Submitted on: Wed 23 Jul 2014 09:33:09 PM UTC
Category: None
URL:
http://gna.org/patch/?4983
Summary: Introduce the UnitState requirement type with the
test Transported
Project: Freeciv
Submitted by: sveinung
Submitted on: Thu 24 Jul 2014 12:45:53 PM UTC
Category: None
Follow-up Comment #2, bug #22381 (project freeciv):
_ For the source tile in unit_attack_unit_at_tile_result(), changing to
can_exist_at_tile() would be better._
Another benefit of testing for can exist is that it becomes consistent with
spy actions.
Update of patch #4894 (project freeciv):
Summary: UnitState requirement type with the test
TransportDependent = Add the new test TransportDependent to the UnitState
requirement type
___
Follow-up Comment #3:
Update to
Update of patch #4683 (project freeciv):
Status: Wont Do = In Progress
Open/Closed: Closed = Open
___
Follow-up Comment #3:
_It seems a logical
URL:
http://gna.org/patch/?4984
Summary: What destination tiles should a non AttackNonNative
be able to attack?
Project: Freeciv
Submitted by: sveinung
Submitted on: Thu 24 Jul 2014 01:18:40 PM UTC
Category: None
Follow-up Comment #1, patch #4984 (project freeciv):
* The current limit is that the tile must have a native terrain type or be
native from an extra.
* Should a city where the unit could exist make it native?
_Is there a narrative reason UTYF_TRIREME units should not attack unsafe
terrains?_
A
Follow-up Comment #4, bug #22381 (project freeciv):
_Should the source bug just be fixed, and we can discuss the destination
more?_
I think that is a good idea. I raised patch #4984 for discussing that issue.
I could start working on this bug today (unless you already have started).
_As for
Follow-up Comment #1, bug #22393 (project freeciv):
Looks like its cause by land barbarians in the ocean.
Attaching savegame with barbarians in ocean.
Started with a a set gameseed and mapseed so reproducable:
/ruleset classic
/set minplayers 0
/set timeout -1
/set saveturns 100
/set
Follow-up Comment #3, bug #22393 (project freeciv):
_Do the second patch in patch ##4973 solve your issue?_
Yes.
___
Reply to this item at:
http://gna.org/bugs/?22393
___
Message sent
Update of patch #4977 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4980 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4972 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/patch/?4986
Summary: Make the action UI name getters more usable
Project: Freeciv
Submitted by: sveinung
Submitted on: Fri 25 Jul 2014 03:43:37 PM UTC
Category: None
Priority: 5 -
Update of bug #22381 (project freeciv):
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4983 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4894 (project freeciv):
Status: In Progress = Ready For Test
___
Reply to this item at:
http://gna.org/patch/?4894
___
Message sent
Update of patch #4986 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
URL:
http://gna.org/patch/?4995
Summary: Make the source tile limit on spy actions a hard
action enabler requirement
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 28 Jul 2014 11:04:32 AM UTC
Category: None
URL:
http://gna.org/bugs/?22400
Summary: Stealing random and future tech don't work
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 28 Jul 2014 12:08:40 PM UTC
Category: None
Severity: 3 - Normal
URL:
http://gna.org/bugs/?22401
Summary: Freeciv ruledit stores appended requirements to
action enablers
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 28 Jul 2014 05:24:06 PM UTC
Category: None
Update of bug #22401 (project freeciv):
Status: In Progress = Ready For Test
___
Follow-up Comment #1:
Only append requirements to action enablers when loading the rules to play.
(file #21582)
Update of patch #4995 (project freeciv):
Status: In Progress = Ready For Test
___
Reply to this item at:
http://gna.org/patch/?4995
___
Message sent
Update of patch #4894 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of patch #4996 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Update of bug #22401 (project freeciv):
Category:None = freeciv-ruledit
Status: Ready For Test = Fixed
Open/Closed:Open = Closed
URL:
http://gna.org/bugs/?22407
Summary: Assertion failure while loading classic in Freeciv
ruledit
Project: Freeciv
Submitted by: sveinung
Submitted on: Wed 30 Jul 2014 11:50:40 AM UTC
Category: freeciv-ruledit
Update of bug #22407 (project freeciv):
Status:None = Duplicate
Open/Closed:Open = Closed
___
Follow-up Comment #2:
_This looks like bug
Follow-up Comment #2, bug #22400 (project freeciv):
New version attached. It ignores numbering. It merges the two if statements
that validated tech target.
_Your test looks a bit weak._
Correct. A new renumbering could have broken it.
_What would happen if the tech is A_LAST? Or if the tech is
Update of patch #4997 (project freeciv):
Depends on: = patch #4671
___
Follow-up Comment #1:
Added patch #4671 (move the source tile limits to the ruleset) as a
dependency.
In patch
Update of patch #4995 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Follow-up Comment #9:
_the above is
Update of patch #4671 (project freeciv):
Summary: Move limits on what terrain a spy action can be done
from to action enablers = Move limits on what terrain a spy action can be
done from to the ruleset
___
Reply to this
URL:
http://gna.org/patch/?5017
Summary: Document more hard requirements in
doc/README.actions
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 04 Aug 2014 01:14:01 PM UTC
Category: docs
Priority:
URL:
http://gna.org/patch/?5018
Summary: Rewrite description of action enablers in
doc/README.actions
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 04 Aug 2014 01:28:24 PM UTC
Category: docs
URL:
http://gna.org/patch/?5019
Summary: Document that hard action requiremetns may be
appended
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 04 Aug 2014 01:31:38 PM UTC
Category: docs
Update of patch #5019 (project freeciv):
Depends on: = patch #5018
___
Reply to this item at:
http://gna.org/patch/?5019
___
Message sent
URL:
http://gna.org/bugs/?22442
Summary: embassy is spelled emabassy in README.actions
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 04 Aug 2014 01:08:46 PM UTC
Category: docs
Severity: 3 -
Follow-up Comment #9, patch #4671 (project freeciv):
Patch that moves the limits to the rulesets and updates the documentation. It
isn't Ready For Test since it depends on patch #5020.
(file #21662)
___
Additional Item Attachment:
File
Update of patch #4671 (project freeciv):
Depends on: = patch #5020
___
Reply to this item at:
http://gna.org/patch/?4671
___
Message sent
601 - 700 of 3648 matches
Mail list logo