Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The game shouldn't look for files for sequence definitions starting with ^ #16095

Closed
ghost opened this issue Jan 22, 2019 · 4 comments · Fixed by #18514
Closed

The game shouldn't look for files for sequence definitions starting with ^ #16095

ghost opened this issue Jan 22, 2019 · 4 comments · Fixed by #18514

Comments

@ghost
Copy link

ghost commented Jan 22, 2019

OpenRA/raclassic#169 (comment)

@pchote
Copy link
Member

pchote commented Jan 22, 2019

Can you please provide a simple example we can test on the default mods?

@ghost
Copy link
Author

ghost commented Jan 22, 2019

mods\ra\sequences\infantry.yaml

^civilian:
	stand:
		Facings: 8
	panic-stand:
		Start: 8
		Stride: 6
		Facings: 8
	panic-run:
		Start: 8
		Length: 6
		Facings: 8
	shoot:
		Start: 120
		Length: 4
		Facings: 8
	die1:
		Start: 152
		Length: 8
	die2:
		Start: 160
		Length: 8
	die3:
		Start: 168
		Length: 12
	die4:
		Start: 168
		Length: 12
	die5:
		Start: 180
		Length: 18
	die6: electro
		Length: *
		UseTilesetExtension: true
		TilesetOverrides:
			DESERT: TEMPERAT
			INTERIOR: TEMPERAT
	die-crushed: corpse1
		Length: 6
		Tick: 1600
		ZOffset: -511
		UseTilesetExtension: true
		TilesetOverrides:
			DESERT: TEMPERAT
			INTERIOR: TEMPERAT
	run:
		Start: 56
		Length: 6
		Facings: 8
		Tick: 80

c1:
	Inherits: ^civilian

c2:
	Inherits: ^civilian

c3:
	Inherits: ^civilian

@pchote
Copy link
Member

pchote commented Jan 22, 2019

#14859 (specifically: the filtering in Ruleset) can be applied to sequences, and also weapons.

@deleted-user-1
Copy link
Contributor

Seems still valid as the warning is printed to the debug log as explained in comment#1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants