Skip to content

Commit

Permalink
regress-asc works again. v4 no longer adds gift string to the materia…
Browse files Browse the repository at this point in the history
…l field.

this in turn ensures v5 upgrade doesn't turn the field into a new attribute that results in non-faithful roundtripping.
  • Loading branch information
brlcad committed Jan 10, 2025
1 parent 4623770 commit 9ab1eea
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions regress/asc/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,13 +1,7 @@
if(SH_EXEC)
if(TARGET asc2g)
brlcad_add_test(NAME regress-asc COMMAND ${SH_EXEC} "${CMAKE_CURRENT_SOURCE_DIR}/asc.sh" ${CMAKE_SOURCE_DIR})

# 20220909CSM - temp disable during delayed testing fix

# FIXME: trip through v4 is adding a "material_name" attribute to
# regions because v4 import sets material = "gift#" unnecessarily

# BRLCAD_REGRESSION_TEST(regress-asc "asc2g;g2asc;gdiff;dbupgrade" TEST_DEFINED)
BRLCAD_REGRESSION_TEST(regress-asc "asc2g;g2asc;gdiff;dbupgrade" TEST_DEFINED)
endif(TARGET asc2g)

brlcad_add_test(NAME regress-asc2dsp COMMAND ${SH_EXEC} "${CMAKE_CURRENT_SOURCE_DIR}/asc2dsp.sh" ${CMAKE_SOURCE_DIR})
Expand Down

0 comments on commit 9ab1eea

Please sign in to comment.