Opened 7 years ago

Closed 4 years ago

Last modified 4 years ago

#4312 closed defect (fixed)

Automatically check for missing references when changing art files

Reported by: fatherbushido Owned by:
Priority: Should Have Milestone: Alpha 24
Component: Build & Packages Keywords:
Cc: Johnathan Good, Niek Patch:

Description (last modified by fatherbushido)

We can track in that ticket missing art files. There is also a ticket about unused art files but I don't want to create confusion between the two tasks.

I would recommend to lauch

perl -I source/tools/entity checkrefs.pl

each time someone changes art stuff.

Change History (11)

comment:1 by fatherbushido, 7 years ago

At r18904:

Missing file 'art/textures/skins/structural/gaul_tile.png' referenced by: 'public/art/actors/props/structures/gauls/fortress_b_roof.xml', 'public/art/actors/props/structures/gauls/scout_tower_b_2_roof.xml', 'public/art/actors/props/structures/gauls/wooden_tower_2_roof.xml' Missing file 'art/textures/skins/structural/gaul_tile_small.png' referenced by: 'public/art/actors/props/structures/gauls/wooden_tower_2_roof.xml' Missing file 'art/textures/skins/structural/gaul_tile_small_norm.png' referenced by: 'public/art/actors/props/structures/gauls/wooden_tower_2_roof.xml' Missing file 'art/textures/skins/structural/gaul_tile_small_spec.png' referenced by: 'public/art/actors/props/structures/gauls/wooden_tower_2_roof.xml'

comment:2 by Itms, 7 years ago

Component: UI & SimulationBuild & Packages

comment:3 by Stan, 7 years ago

Isn't that art and animation ? Btw all those regressions where introduced by lordgood's last commits.

comment:4 by Itms, 7 years ago

The proposal of launching the script automatically should be the main objective of this ticket I believe. Feel free to put Art&Anim but don't close it when the missing files are fixed. :)

comment:5 by Stan, 7 years ago

Yeah sure can someone in the team notify lordgood ?

comment:6 by fatherbushido, 7 years ago

(I informed him, he will fix that after release, but it breaks nothing in the current game)

comment:7 by elexis, 7 years ago

Summary: Missing Art FilesAutomatically check for missing references when changing art files

Reported refs removed in r18908

comment:8 by fatherbushido, 7 years ago

refs #2524

comment:9 by fatherbushido, 7 years ago

Description: modified (diff)

comment:10 by Itms, 4 years ago

Milestone: BacklogAlpha 24
Resolution: fixed
Status: newclosed

This is checked by Jenkins now (and it has been the case for some time). The commit below puts the related Jenkins pipeline under version control.

comment:11 by Itms, 4 years ago

In 23250:

Update the Jenkins scripts to match the current state, which uses Jenkins pipelines, Docker for reproductible build environments, and ZFS for space efficiency and robust incremental patching. refs #4419, refs #4312.

Reviewed By: Angen
Differential Revision: https://code.wildfiregames.com/D1910

Note: See TracTickets for help on using tickets.